Skip to content

Commit 4d4b0b7

Browse files
committed
categorical 2dmap: fix flaky tests
1 parent 793c2a3 commit 4d4b0b7

File tree

3 files changed

+8
-6
lines changed

3 files changed

+8
-6
lines changed

src/traces/heatmap/clean_2d_array.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ module.exports = function clean2dArray(zOld, trace, xa, ya) {
2020
return +v;
2121
}
2222

23-
if(trace.transpose) {
23+
if(trace && trace.transpose) {
2424
rowlen = 0;
2525
for(i = 0; i < zOld.length; i++) rowlen = Math.max(rowlen, zOld[i].length);
2626
if(rowlen === 0) return false;

test/jasmine/tests/contour_test.js

+6-4
Original file line numberDiff line numberDiff line change
@@ -358,9 +358,10 @@ describe('contour calc', function() {
358358
['contour'].forEach(function(traceType) {
359359
it('should sort z data based on axis categoryorder for ' + traceType, function() {
360360
var mock = require('@mocks/heatmap_categoryorder');
361-
var data = mock.data[0];
361+
var mockCopy = Lib.extendDeep({}, mock);
362+
var data = mockCopy.data[0];
362363
data.type = traceType;
363-
var layout = mock.layout;
364+
var layout = mockCopy.layout;
364365

365366
// sort x axis categories
366367
var mockLayout = Lib.extendDeep({}, layout);
@@ -387,9 +388,10 @@ describe('contour calc', function() {
387388

388389
it('should sort z data based on axis categoryarray ' + traceType, function() {
389390
var mock = require('@mocks/heatmap_categoryorder');
390-
var data = mock.data[0];
391+
var mockCopy = Lib.extendDeep({}, mock);
392+
var data = mockCopy.data[0];
391393
data.type = traceType;
392-
var layout = mock.layout;
394+
var layout = mockCopy.layout;
393395

394396
layout.xaxis.categoryorder = 'array';
395397
layout.xaxis.categoryarray = ['x', 'z', 'y', 'w'];

test/jasmine/tests/histogram2d_test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ describe('Test histogram2d', function() {
172172
});
173173

174174
['histogram2d', 'histogram2dcontour'].forEach(function(traceType) {
175-
it('@flaky should sort z data based on axis categoryorder for ' + traceType, function() {
175+
it('should sort z data based on axis categoryorder for ' + traceType, function() {
176176
var mock = require('@mocks/heatmap_categoryorder');
177177
var mockCopy = Lib.extendDeep({}, mock);
178178
var data = mockCopy.data[0];

0 commit comments

Comments
 (0)