Skip to content

Array edits #1403

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 18 commits into from
Feb 24, 2017
Merged

Array edits #1403

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
cc45189
fix basic annotation reference change bug
alexcjohnson Jan 6, 2017
13a87ce
support RegExp in Lib.pushUnique
alexcjohnson Feb 18, 2017
61f250c
editContainerArray: consistent handling of container array edits
alexcjohnson Jan 10, 2017
a9526bf
test fixin
alexcjohnson Feb 21, 2017
649a831
robustify click_test
alexcjohnson Feb 21, 2017
d55568c
:hocho: obsolete getShapeLayer
alexcjohnson Feb 21, 2017
2e6c030
break up new circular deps
alexcjohnson Feb 22, 2017
f7e60fb
relinkPrivateKeys inside arrayContainerDefaults
alexcjohnson Feb 22, 2017
d7cdc0a
revert layout_image.json - keep #1390 separate for now
alexcjohnson Feb 22, 2017
7ed9eb9
fix for components using category names as coordinates
alexcjohnson Feb 22, 2017
9f89dd6
fix for autorange with bad axis refs from a component
alexcjohnson Feb 22, 2017
46962af
point to open issue #1111
alexcjohnson Feb 22, 2017
1af058a
allow all `info_array`s to be pruned and command args to be optional
alexcjohnson Feb 23, 2017
c6378b1
exclude `args` from `nestedProperty` pruning
alexcjohnson Feb 23, 2017
70c882b
keep _fullLayout container arrays in sync with splices in layout
alexcjohnson Feb 23, 2017
5fb462f
comments and tests for plot_api/helpers.hasParent
alexcjohnson Feb 24, 2017
d29cbae
editContainerArray => applyContainerArrayChanges
alexcjohnson Feb 24, 2017
3ef1c9b
test errors from parent/child simultaneous edits
alexcjohnson Feb 24, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
78 changes: 76 additions & 2 deletions test/jasmine/tests/plot_api_test.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,6 +180,42 @@ describe('Test plot api', function() {
.then(done);
});

it('errors if child and parent are edited together', function(done) {
var edit1 = {rando: [{a: 1}, {b: 2}]};
var edit2 = {'rando[1]': {c: 3}};
var edit3 = {'rando[1].d': 4};

Plotly.plot(gd, [{ x: [1, 2, 3], y: [1, 2, 3] }])
.then(function() {
return Plotly.relayout(gd, edit1);
})
.then(function() {
expect(gd.layout.rando).toEqual([{a: 1}, {b: 2}]);
return Plotly.relayout(gd, edit2);
})
.then(function() {
expect(gd.layout.rando).toEqual([{a: 1}, {c: 3}]);
return Plotly.relayout(gd, edit3);
})
.then(function() {
expect(gd.layout.rando).toEqual([{a: 1}, {c: 3, d: 4}]);

// OK, setup is done - test the failing combinations
[[edit1, edit2], [edit1, edit3], [edit2, edit3]].forEach(function(v) {
// combine properties in both orders - which results in the same object
// but the properties are iterated in opposite orders
expect(function() {
return Plotly.relayout(gd, Lib.extendFlat({}, v[0], v[1]));
}).toThrow();
expect(function() {
return Plotly.relayout(gd, Lib.extendFlat({}, v[1], v[0]));
}).toThrow();
});
})
.catch(fail)
.then(done);
});

it('can set empty text nodes', function(done) {
var data = [{
x: [1, 2, 3],
Expand Down Expand Up @@ -333,7 +369,7 @@ describe('Test plot api', function() {
destroyGraphDiv();
});

it('should redo auto z/contour when editing z array', function() {
it('should redo auto z/contour when editing z array', function(done) {
Plotly.plot(gd, [{type: 'contour', z: [[1, 2], [3, 4]]}]).then(function() {
expect(gd.data[0].zauto).toBe(true, gd.data[0]);
expect(gd.data[0].zmin).toBe(1);
Expand All @@ -348,7 +384,45 @@ describe('Test plot api', function() {
expect(gd.data[0].zmax).toBe(10);

expect(gd.data[0].contours).toEqual({start: 3, end: 9, size: 1});
});
})
.catch(fail)
.then(done);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hah, this test wasn't actually running. More precisely, it was running but not being used. Who wrote that sloppy thing? 🙈

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

Not me 😄

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The biggest irony is my commit message there:

fix the tests I must have only thought I had run successfully

});

it('errors if child and parent are edited together', function(done) {
var edit1 = {rando: [[{a: 1}, {b: 2}]]};
var edit2 = {'rando[1]': {c: 3}};
var edit3 = {'rando[1].d': 4};

Plotly.plot(gd, [{x: [1, 2, 3], y: [1, 2, 3], type: 'scatter'}])
.then(function() {
return Plotly.restyle(gd, edit1);
})
.then(function() {
expect(gd.data[0].rando).toEqual([{a: 1}, {b: 2}]);
return Plotly.restyle(gd, edit2);
})
.then(function() {
expect(gd.data[0].rando).toEqual([{a: 1}, {c: 3}]);
return Plotly.restyle(gd, edit3);
})
.then(function() {
expect(gd.data[0].rando).toEqual([{a: 1}, {c: 3, d: 4}]);

// OK, setup is done - test the failing combinations
[[edit1, edit2], [edit1, edit3], [edit2, edit3]].forEach(function(v) {
// combine properties in both orders - which results in the same object
// but the properties are iterated in opposite orders
expect(function() {
return Plotly.restyle(gd, Lib.extendFlat({}, v[0], v[1]));
}).toThrow();
expect(function() {
return Plotly.restyle(gd, Lib.extendFlat({}, v[1], v[0]));
}).toThrow();
});
})
.catch(fail)
.then(done);
});
});

Expand Down