diff options
| author | Brenton | 2013-06-01 23:29:18 -0700 | 
|---|---|---|
| committer | Jeff Cross | 2013-07-24 10:58:56 -0700 | 
| commit | 1dcafd18afed4465ee13db91cedc8fecc3aa2c96 (patch) | |
| tree | 5f6cb7c53c4606072637a529df44046c4369821a /test/AngularSpec.js | |
| parent | 4a7b6a4555a76b19dd217171fc0ddce6707bca95 (diff) | |
| download | angular.js-1dcafd18afed4465ee13db91cedc8fecc3aa2c96.tar.bz2 | |
fix(equals): {} and [] should not be considered equivalent
angular.equals was returning inconsistent values for the comparison between
{} and []:
    angular.equals({}, []) // true
    angular.equals([], {}]) // false
Since these object are not of the same type, they should not be considered
equivalent.
Diffstat (limited to 'test/AngularSpec.js')
| -rw-r--r-- | test/AngularSpec.js | 5 | 
1 files changed, 5 insertions, 0 deletions
| diff --git a/test/AngularSpec.js b/test/AngularSpec.js index b43edf1c..a58b8109 100644 --- a/test/AngularSpec.js +++ b/test/AngularSpec.js @@ -289,6 +289,11 @@ describe('angular', function() {        expect(equals(/^abc/, /abc/)).toBe(false);        expect(equals(/^abc/, '/^abc/')).toBe(false);      }); + +    it('should return false when comparing an object and an array', function() { +      expect(equals({}, [])).toBe(false); +      expect(equals([], {})).toBe(false); +    });    });    describe('size', function() { | 
