Hopiu/angular.js
1
0
Fork 0
mirror of https://github.com/Hopiu/angular.js.git synced 2026-05-04 21:24:42 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix(test): improve $cookie service test to work with Safari 5.1

Browse source 
the max size for safari cookies has changed sligtly so I had to adjust
the test to make cookie creation fail on this browser
This commit is contained in:
Igor Minar 2011-08-26 10:54:38 -07:00 committed by Igor Minar
parent 90ca6f983e
commit ea8952177e
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
test/BrowserSpecs.js
View file

@ -377,17 +377,17 @@ describe('browser', function(){
it('should log warnings when 4kb per cookie storage limit is reached', function() { it('should log warnings when 4kb per cookie storage limit is reached', function() {
var i, longVal = '', cookieStr; var i, longVal = '', cookieStr;
for(i=0; i<4092; i++) { for(i=0; i<4091; i++) {
longVal += '+'; longVal += '+';
} }
cookieStr = document.cookie; cookieStr = document.cookie;
browser.cookies('x', longVal); //total size 4094-4096, so it should go through browser.cookies('x', longVal); //total size 4093-4096, so it should go through
expect(document.cookie).not.toEqual(cookieStr); expect(document.cookie).not.toEqual(cookieStr);
expect(browser.cookies()['x']).toEqual(longVal); expect(browser.cookies()['x']).toEqual(longVal);
expect(logs.warn).toEqual([]); expect(logs.warn).toEqual([]);
browser.cookies('x', longVal + 'xxx'); //total size 4097-4099, a warning should be logged browser.cookies('x', longVal + 'xxxx'); //total size 4097-4099, a warning should be logged
expect(logs.warn).toEqual( expect(logs.warn).toEqual(
[[ "Cookie 'x' possibly not set or overflowed because it was too large (4097 > 4096 " + [[ "Cookie 'x' possibly not set or overflowed because it was too large (4097 > 4096 " +
"bytes)!" ]]); "bytes)!" ]]);