Skip to content

Commit 24caa5e

Browse files
committed
improve comments
1 parent 1970855 commit 24caa5e

File tree

3 files changed

+19
-5
lines changed

3 files changed

+19
-5
lines changed

test/image/compare_pixels_test.js

Lines changed: 12 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -28,9 +28,18 @@ function runAll () {
2828

2929
var files = fs.readdirSync(constants.pathToTestImageMocks);
3030

31-
// -1 for font-wishlist and
32-
// -38 for the gl2d mocks
33-
// -1 for gl3d_bunny-hull
31+
/*
32+
* Some test cases exhibit run-to-run randomness;
33+
* skip over a few test cases for now
34+
*
35+
* More info:
36+
* https://github.com/plotly/plotly.js/issues/62
37+
*
38+
* 40 test case are removed:
39+
* - font-wishlist (1 test case)
40+
* - all gl2d (38)
41+
* - gl2d_bunny-hull (1)
42+
*/
3443
t.plan(files.length - 40);
3544

3645
for (var i = 0; i < files.length; i ++) {

test/jasmine/karma.ciconf.js

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,12 @@ function func(config) {
88

99
// Continuous Integration mode
1010

11-
// exclude the WebGL interaction test on circle runs
11+
/*
12+
* WebGL interaction test cases fail on the CircleCI
13+
* most likely due to a WebGL/driver issue;
14+
* exclude them from the CircleCI test bundle.
15+
*
16+
*/
1217
func.defaultConfig.exclude = ['tests/gl_plot_interact_test.js'];
1318

1419
// if true, Karma captures browsers, runs the tests and exits

test/jasmine/tests/gl_plot_interact_test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ var d3 = require('d3');
33
var Plotly = require('@src/index');
44

55
/*
6-
* The following test cases fail on the CircleCI
6+
* WebGL interaction test cases fail on the CircleCI
77
* most likely due to a WebGL/driver issue
88
*
99
*/

0 commit comments

Comments
 (0)