Adding support for ng_repeat, data-ng-repeat, and x-ng-repeat.

Author: juliemr

protractor.js

@@ -80,10 +80,14 @@ clientSideScripts.findBindings = function() {
   var index = arguments[1];
 
   var rows = [];
-  var repeatElems = document.querySelectorAll('[ng-repeat]');
-  for (var i = 0; i < repeatElems.length; ++i) {
-    if (repeatElems[i].getAttribute('ng-repeat').indexOf(repeater) != -1) {
-      rows.push(repeatElems[i]);
+  var prefixes = ['ng-', 'ng_', 'data-ng-', 'x-ng-'];
+  for (var p = 0; p < prefixes.length; ++p) {
+    var attr = prefixes[p] + 'repeat';
+    var repeatElems = document.querySelectorAll('[' + attr + ']');
+    for (var i = 0; i < repeatElems.length; ++i) {
+      if (repeatElems[i].getAttribute(attr).indexOf(repeater) != -1) {
+        rows.push(repeatElems[i]);
+      }
     }
   }
   return rows[index - 1];
@@ -105,10 +109,14 @@ clientSideScripts.findRepeaterElement = function() {
   var binding = arguments[2];
 
   var rows = [];
-  var repeatElems = document.querySelectorAll('[ng-repeat]');
-  for (var i = 0; i < repeatElems.length; ++i) {
-    if (repeatElems[i].getAttribute('ng-repeat').indexOf(repeater) != -1) {
-      rows.push(repeatElems[i]);
+  var prefixes = ['ng-', 'ng_', 'data-ng-', 'x-ng-'];
+  for (var p = 0; p < prefixes.length; ++p) {
+    var attr = prefixes[p] + 'repeat';
+    var repeatElems = document.querySelectorAll('[' + attr + ']');
+    for (var i = 0; i < repeatElems.length; ++i) {
+      if (repeatElems[i].getAttribute(attr).indexOf(repeater) != -1) {
+        rows.push(repeatElems[i]);
+      }
     }
   }
   var row = rows[index - 1];
@@ -145,10 +153,14 @@ clientSideScripts.findRepeaterElement = function() {
   var binding = arguments[1];
 
   var rows = [];
-  var repeatElems = document.querySelectorAll('[ng-repeat]');
-  for (var i = 0; i < repeatElems.length; ++i) {
-    if (repeatElems[i].getAttribute('ng-repeat').indexOf(repeater) != -1) {
-      rows.push(repeatElems[i]);
+  var prefixes = ['ng-', 'ng_', 'data-ng-', 'x-ng-'];
+  for (var p = 0; p < prefixes.length; ++p) {
+    var attr = prefixes[p] + 'repeat';
+    var repeatElems = document.querySelectorAll('[' + attr + ']');
+    for (var i = 0; i < repeatElems.length; ++i) {
+      if (repeatElems[i].getAttribute(attr).indexOf(repeater) != -1) {
+        rows.push(repeatElems[i]);
+      }
     }
   }
   for (var i = 0; i < rows.length; ++i) {

spec/testAppSpec.js

@@ -110,7 +110,7 @@ describe('test application', function() {
           });
     });
 
-    xit('should find a repeater using data-ng-repeat', function(done) {
+    it('should find a repeater using data-ng-repeat', function(done) {
       catchPromiseErrors(done);
       ptor.findElement(protractor.By.repeater('day in days').row(3)).
           getText().then(function(text) {
@@ -139,28 +139,28 @@ describe('test application', function() {
           });
     });
 
-    xit('should find a repeater using ng_repeat', function(done) {
+    it('should find a repeater using ng_repeat', function(done) {
       catchPromiseErrors(done);
       ptor.findElement(protractor.By.repeater('foo in days').row(3)).
           getText().then(function(text) {
             expect(text).toEqual('Wed');
           });
-      ptor.findElement(protractor.By.repeater('foo in days').row(3)).
-          column('foo').
+      ptor.findElement(protractor.By.repeater('foo in days').row(3).
+          column('foo')).
           getText().then(function(text) {
             expect(text).toEqual('Wed');
             done();
           });
     });
 
-    xit('should find a repeater using x-ng-repeat', function(done) {
+    it('should find a repeater using x-ng-repeat', function(done) {
       catchPromiseErrors(done);
       ptor.findElement(protractor.By.repeater('qux in days').row(3)).
           getText().then(function(text) {
             expect(text).toEqual('Wed');
           });
-      ptor.findElement(protractor.By.repeater('qux in days').row(3)).
-          column('quz').
+      ptor.findElement(protractor.By.repeater('qux in days').row(3).
+          column('qux')).
           getText().then(function(text) {
             expect(text).toEqual('Wed');
             done();