Skip to content
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.

Commit 7cb691e

Browse files
committedJan 13, 2017
AJ proof ie9 bundle test
1 parent 4ce9df0 commit 7cb691e

File tree

2 files changed

+9
-4
lines changed

2 files changed

+9
-4
lines changed
 

‎test/jasmine/bundle_tests/ie9_test.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,9 +21,11 @@ describe('Bundle with IE9 supported trace types:', function() {
2121

2222
afterEach(destroyGraphDiv);
2323

24-
it('[wip] check that ie9_mock.js did its job', function() {
25-
expect(window.ArrayBuffer).toBeUndefined();
26-
expect(window.Uint8Array).toBeUndefined();
24+
it(' check that ie9_mock.js did its job', function() {
25+
expect(function() { return ArrayBuffer; })
26+
.toThrow(new ReferenceError('ArrayBuffer is not defined'));
27+
expect(function() { return Uint8Array; })
28+
.toThrow(new ReferenceError('Uint8Array is not defined'));
2729
});
2830

2931
it('heatmaps with smoothing should work', function(done) {

‎test/jasmine/karma.conf.js

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,9 +129,12 @@ else if(isRequireJSTest) {
129129
];
130130
}
131131
else if(isIE9Test) {
132+
// load ie9_mock.js before plotly.js+test bundle
133+
// to catch reference errors that could occur
134+
// when plotly.js is first loaded.
135+
132136
func.defaultConfig.files = [
133137
'./assets/ie9_mock.js',
134-
// '../../dist/extras/typedarray.min.js',
135138
testFileGlob
136139
];
137140

0 commit comments

Comments
 (0)
Please sign in to comment.