fix(ng:repeat): support repeating over array with null

typeof null == 'object', but it doesn't behave like an object
because its properties can't be dereferenced, so we need
to special-case it.

Closes #702

2 files changed, 27 insertions, 0 deletions

diff --git a/test/ApiSpecs.js b/test/ApiSpecs.js
index a0833fba..35a85bd4 100644
--- a/test/ApiSpecs.js
+++ b/test/ApiSpecs.js
@@ -36,6 +36,25 @@ describe('api', function() {
       expect(map.shift('key')).toEqual(undefined);
       expect(map[hashKey('key')]).toEqual(undefined);
     });
+
+    it('should support primitive and object keys', function() {
+      var obj1 = {},
+          obj2 = {};
+
+      var map = new HashQueueMap();
+      map.push(obj1, 'a1');
+      map.push(obj1, 'a2');
+      map.push(obj2, 'b');
+      map.push(1, 'c');
+      map.push(undefined, 'd');
+      map.push(null, 'e');
+
+      expect(map[hashKey(obj1)]).toEqual(['a1', 'a2']);
+      expect(map[hashKey(obj2)]).toEqual(['b']);
+      expect(map[hashKey(1)]).toEqual(['c']);
+      expect(map[hashKey(undefined)]).toEqual(['d']);
+      expect(map[hashKey(null)]).toEqual(['e']);
+    });
   });
 });
 
diff --git a/test/widgetsSpec.js b/test/widgetsSpec.js
index d2867d09..3b53c7d2 100644
--- a/test/widgetsSpec.js
+++ b/test/widgetsSpec.js
@@ -429,6 +429,14 @@ describe('widget', function() {
       expect(element.text()).toBe('a|b|||c||d|');
     }));
 
+    it('should iterate over all kinds of types', inject(function($rootScope, $compile) {
+      var element = $compile('<ul><li ng:repeat="item in array">{{item}}|</li></ul>')($rootScope);
+      $rootScope.array = ['a', 1, null, undefined, {}];
+      $rootScope.$digest();
+
+      expect(element.text()).toMatch(/a\|1\|\|\|\{\s*\}\|/);
+    }));
+
 
     describe('stability', function() {
       var a, b, c, d, lis, element;