fix($browser.addJs): make addJs jQuery compatible

Change addJs implementation to avoid use of jQuery because of issues
that affect angular-ie-compat.js. See inlined comment for more info.
This commit is contained in:
Igor Minar
2011-07-12 00:44:18 -07:00
parent c52e749a6e
commit 47efe44a1d
2 changed files with 41 additions and 7 deletions

View File

@@ -505,4 +505,28 @@ describe('browser', function(){
});
});
});
describe('addJs', function() {
it('should append a script tag to body', function() {
browser.addJs('http://localhost/bar.js');
expect(scripts.length).toBe(1);
expect(scripts[0].src).toBe('http://localhost/bar.js');
expect(scripts[0].id).toBe('');
});
it('should append a script with an id to body', function() {
browser.addJs('http://localhost/bar.js', 'foo-id');
expect(scripts.length).toBe(1);
expect(scripts[0].src).toBe('http://localhost/bar.js');
expect(scripts[0].id).toBe('foo-id');
});
it('should return the appended script element', function() {
var script = browser.addJs('http://localhost/bar.js');
expect(script).toBe(scripts[0]);
});
});
});