diff options
| author | Greg Thornton | 2013-06-23 14:04:59 -0500 | 
|---|---|---|
| committer | Pawel Kozlowski | 2013-07-12 20:32:40 +0200 | 
| commit | 5a294c8646452d6e49339d145faeae4f31dcd0fc (patch) | |
| tree | 1f4599ce8591fb93e358e8036dccbe12267bb8d8 /test/AngularSpec.js | |
| parent | ccda0f35090b1a6b3d5cd2288987ba77fcd1d4d7 (diff) | |
| download | angular.js-5a294c8646452d6e49339d145faeae4f31dcd0fc.tar.bz2 | |
feat(Angular.js): skip JSON.stringify for undefined
Return early in `angular.toJson` if the object to be stringified is `undefined`.
IE8 stringifies `undefined` to `'undefined'` whereas other browsers return
`undefined`. This normalizes behavior and passes currently broken unit tests
in IE8.
Diffstat (limited to 'test/AngularSpec.js')
| -rw-r--r-- | test/AngularSpec.js | 4 | 
1 files changed, 4 insertions, 0 deletions
| diff --git a/test/AngularSpec.js b/test/AngularSpec.js index bf952249..da70aef7 100644 --- a/test/AngularSpec.js +++ b/test/AngularSpec.js @@ -909,6 +909,10 @@ describe('angular', function() {      it('should not serialize scope instances', inject(function($rootScope) {        expect(toJson({key: $rootScope})).toEqual('{"key":"$SCOPE"}');      })); + +    it('should serialize undefined as undefined', function() { +      expect(toJson(undefined)).toEqual(undefined); +    });    });  }); | 
