From fff74ab7594adf5db0165660375c68549fd5e177 Mon Sep 17 00:00:00 2001 From: Georgios Kalpakas Date: Wed, 15 Oct 2014 10:35:21 +0300 Subject: [PATCH] test($resource): enhance test-case to verify correct behaviour Previously, the test-case verified that calling `toJson()`, would remove the `$promise` and `$resolved`, but not that other `$`-prefixed properties would not be removed. --- test/ngResource/resourceSpec.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/test/ngResource/resourceSpec.js b/test/ngResource/resourceSpec.js index 39232b40773b..36b8c4c47363 100644 --- a/test/ngResource/resourceSpec.js +++ b/test/ngResource/resourceSpec.js @@ -658,13 +658,15 @@ describe("resource", function() { var cc = CreditCard.get({id: 123}); $httpBackend.flush(); + cc.$myProp = 'still here'; + expect(cc.$promise).toBeDefined(); expect(cc.$resolved).toBe(true); var json = JSON.parse(angular.toJson(cc)); expect(json.$promise).not.toBeDefined(); expect(json.$resolved).not.toBeDefined(); - expect(json).toEqual({id: 123, number: '9876'}); + expect(json).toEqual({id: 123, number: '9876', $myProp: 'still here'}); }); describe('promise api', function() {