Skip to content

Commit 7611baf

Browse files
committed
Adjust tests
1 parent 4b845ce commit 7611baf

File tree

2 files changed

+2
-28
lines changed

2 files changed

+2
-28
lines changed

src/lib/__tests__/multiValued-test.js

+1-27
Original file line numberDiff line numberDiff line change
@@ -1,34 +1,10 @@
1-
import NumericInput from '../../components/widgets/NumericInput';
21
import React from 'react';
3-
import {MULTI_VALUED_PLACEHOLDER} from '../constants';
42
import {Numeric} from '../../components';
53
import {TestEditor, fixtures, plotly} from '../test-utils';
64
import {connectLayoutToPlot, connectAxesToLayout} from '..';
75
import {mount} from 'enzyme';
86

97
describe('multiValued Numeric', () => {
10-
it('uses placeholder and empty string value', () => {
11-
const fixtureProps = fixtures.scatter({
12-
layout: {xaxis: {range: [0, 1]}, yaxis: {range: [-1, 1]}},
13-
});
14-
const AxesNumeric = connectLayoutToPlot(connectAxesToLayout(Numeric));
15-
16-
const rangeInputs = mount(
17-
<TestEditor {...{...fixtureProps, onUpdate: jest.fn(), plotly}}>
18-
<AxesNumeric attr="range[0]" />
19-
</TestEditor>
20-
).find(NumericInput);
21-
22-
expect(rangeInputs.length).toBe(1);
23-
24-
const allaxisRangeInput = rangeInputs.at(0);
25-
26-
expect(allaxisRangeInput.prop('value')).toBe('');
27-
expect(allaxisRangeInput.prop('placeholder')).toBe(
28-
MULTI_VALUED_PLACEHOLDER
29-
);
30-
});
31-
328
it('uses multiValued defaultContainer as default increment value', () => {
339
const beforeUpdateLayout = jest.fn();
3410
const xaxisLowerRange = -30;
@@ -48,9 +24,7 @@ describe('multiValued Numeric', () => {
4824
expect(beforeUpdateLayout).toBeCalled();
4925
const payload = beforeUpdateLayout.mock.calls[0][0];
5026
expect(payload.update).toEqual({
51-
'xaxis.range[0]': xaxisLowerRange,
52-
'yaxis.range[0]': xaxisLowerRange,
53-
'yaxis2.range[0]': xaxisLowerRange,
27+
'xaxis.range[0]': xaxisLowerRange + 1,
5428
});
5529
});
5630
});

src/lib/__tests__/nestedContainerConnections-test.js

+1-1
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ describe('Plot Connection', () => {
3131
expect(beforeUpdateLayout).toBeCalled();
3232
const payload = beforeUpdateLayout.mock.calls[0][0];
3333
expect(payload).toEqual({
34-
update: {'xaxis.range[0]': 0, 'yaxis.range[0]': 0, 'yaxis2.range[0]': 0},
34+
update: {'xaxis.range[0]': 1},
3535
});
3636
});
3737

0 commit comments

Comments
 (0)