Skip to content

Commit 8b3d663

Browse files
committed
Remove debug code from scatter3d test
1 parent 64f0a6a commit 8b3d663

File tree

2 files changed

+2
-29
lines changed

2 files changed

+2
-29
lines changed

.github/workflows/ci.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,6 +114,7 @@ jobs:
114114
strategy:
115115
fail-fast: false
116116
matrix:
117+
# There's a race condition with one of the tests in shard 6, so run it first to avoid it
117118
shard: [6, 0, 1, 2, 3, 4, 5, 7]
118119
env:
119120
TZ: 'America/Anchorage'

test/jasmine/tests/scatter3d_test.js

Lines changed: 1 addition & 29 deletions
Original file line numberDiff line numberDiff line change
@@ -178,16 +178,6 @@ describe('Test scatter3d interactions:', function() {
178178
});
179179

180180
it('@gl should be able to toggle visibility', function(done) {
181-
var originalOnerror = window.onerror;
182-
window.onerror = function(msg, src, line, col, err) {
183-
console.error('Uncaught error during toggle visibility test:',
184-
msg, 'at', src + ':' + line + ':' + col,
185-
err && err.stack ? '\nStack: ' + err.stack : ''
186-
);
187-
return false;
188-
};
189-
var cleanup = function(x) { window.onerror = originalOnerror; return x; };
190-
191181
var _mock = Lib.extendDeep({}, mock2);
192182
_mock.data[0].x = [0, 1, 3];
193183
_mock.data[0].y = [0, 1, 2];
@@ -205,24 +195,7 @@ describe('Test scatter3d interactions:', function() {
205195
var order0 = [0, 0, 0, 0, 0, 1, 2];
206196

207197
function assertObjects(expected) {
208-
var fullLayout = gd._fullLayout;
209-
var scene = fullLayout && fullLayout.scene;
210-
var _scene = scene && scene._scene;
211-
var glplot = _scene && _scene.glplot;
212-
var objects = glplot && glplot.objects;
213-
console.log('assertObjects debug:',
214-
'fullLayout:', !!fullLayout,
215-
'scene:', !!scene,
216-
'_scene:', !!_scene,
217-
'glplot:', !!glplot,
218-
'objects:', objects,
219-
'objects type:', Object.prototype.toString.call(objects)
220-
);
221-
if(!objects) {
222-
fail('assertObjects: glplot.objects is not available (fullLayout=' + !!fullLayout +
223-
', scene=' + !!scene + ', _scene=' + !!_scene + ', glplot=' + !!glplot + ')');
224-
return;
225-
}
198+
var objects = gd._fullLayout.scene._scene.glplot.objects;
226199
var actual = objects.map(function(o) {
227200
return o._trace.data.index;
228201
});
@@ -265,7 +238,6 @@ describe('Test scatter3d interactions:', function() {
265238
.then(function() {
266239
assertObjects(order0);
267240
})
268-
.then(cleanup, cleanup)
269241
.then(done, done.fail);
270242
});
271243

0 commit comments

Comments
 (0)