Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

Commit 821da26

Browse files
test($location): fix use of browserTrigger
You must now pass `keys` to the function in a config object. This bug in the test became apparent because in newer browsers, arrays have a function called `keys()` and this was causing browserTrigger to fail. Previously it was quietly passing this test despite being wrong.
1 parent 300bffc commit 821da26

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

test/ng/locationSpec.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1239,7 +1239,7 @@ describe('$location', function() {
12391239
initBrowser(),
12401240
initLocation(),
12411241
function($browser) {
1242-
browserTrigger(link, 'click', ['ctrl']);
1242+
browserTrigger(link, 'click', { keys: ['ctrl'] });
12431243
expectNoRewrite($browser);
12441244
}
12451245
);
@@ -1252,7 +1252,7 @@ describe('$location', function() {
12521252
initBrowser(),
12531253
initLocation(),
12541254
function($browser) {
1255-
browserTrigger(link, 'click', ['meta']);
1255+
browserTrigger(link, 'click', { keys: ['meta'] });
12561256
expectNoRewrite($browser);
12571257
}
12581258
);

0 commit comments

Comments
 (0)