Skip to content

Commit 45717b1

Browse files
committed
Add note about test failure
1 parent e5a80ee commit 45717b1

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

test/jasmine/tests/command_test.js

+8
Original file line numberDiff line numberDiff line change
@@ -328,7 +328,15 @@ describe('Plots.computeAPICommandBindings', function() {
328328
expect(result).toEqual([
329329
{type: 'data', prop: 'y', traces: [1], value: [[3, 4, 5]]},
330330
{type: 'data', prop: 'marker.size', traces: [1, 0], value: [10, 20]},
331+
332+
// This result is actually not quite correct. Setting `line` should override
333+
// this—or actually it's technically undefined since the iteration order of
334+
// objects is not strictly defined but is at least consistent across browsers.
335+
// The worst-case scenario right now isn't too bad though since it's an obscure
336+
// case that will definitely cause bailout anyway before any bindings would
337+
// happen.
331338
{type: 'data', prop: 'line.color', traces: [1, 0], value: ['red', 'red']},
339+
332340
{type: 'data', prop: 'line.width', traces: [1, 0], value: [2, 8]}
333341
]);
334342
});

0 commit comments

Comments
 (0)