diff options
| author | Oren Avissar | 2013-04-22 21:00:15 -0700 |
|---|---|---|
| committer | Pete Bacon Darwin | 2013-04-29 16:57:33 +0100 |
| commit | c3235db9ee89041f4c9c9b39fbf1a41ab866820c (patch) | |
| tree | ec73af293da81cb47f289e5829cdd3c64a9f1068 | |
| parent | 738113bac88b549000d071059d1eea501e9c9db2 (diff) | |
| download | angular.js-c3235db9ee89041f4c9c9b39fbf1a41ab866820c.tar.bz2 | |
test(browser/compile): fix calls to Jasmine fail()
The fail() function in Jasmine expects an Error object parameter.
Also, there is no global alias for fail() so it must be accessed using
`this.fail(new Error())`.
| -rwxr-xr-x[-rw-r--r--] | test/ng/browserSpecs.js | 4 | ||||
| -rwxr-xr-x[-rw-r--r--] | test/ng/compileSpec.js | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/test/ng/browserSpecs.js b/test/ng/browserSpecs.js index e989330b..eba9bd02 100644..100755 --- a/test/ng/browserSpecs.js +++ b/test/ng/browserSpecs.js @@ -271,8 +271,8 @@ describe('browser', function() { browser.cookies('x', longVal + longVal + longVal); //should be too long for all browsers if (document.cookie !== cookieStr) { - fail("browser didn't drop long cookie when it was expected. make the cookie in this " + - "test longer"); + this.fail(new Error("browser didn't drop long cookie when it was expected. make the " + + "cookie in this test longer")); } expect(browser.cookies().x).toEqual('shortVal'); diff --git a/test/ng/compileSpec.js b/test/ng/compileSpec.js index d24a9dac..50aa30ef 100644..100755 --- a/test/ng/compileSpec.js +++ b/test/ng/compileSpec.js @@ -540,8 +540,8 @@ describe('$compile', function() { it('should prevent multiple templates per element', inject(function($compile) { try { - $compile('<div><span replace class="replace"></span></div>') - fail(); + $compile('<div><span replace class="replace"></span></div>'); + this.fail(new Error('should have thrown Multiple directives error')); } catch(e) { expect(e.message).toMatch(/Multiple directives .* asking for template/); } |
