Skip to content

Commit 0158f45

Browse files
committed
test: move user-defined transform test to transform_multi
1 parent f6eb2f4 commit 0158f45

File tree

2 files changed

+45
-39
lines changed

2 files changed

+45
-39
lines changed

test/jasmine/tests/transform_filter_test.js

-39
Original file line numberDiff line numberDiff line change
@@ -95,45 +95,6 @@ describe('one-to-one transforms:', function() {
9595
}, '- trace second');
9696
});
9797

98-
it('should pass correctly arguments to transform methods', function() {
99-
var transformIn = { type: 'fake' };
100-
var transformOut = {};
101-
102-
var dataIn = [{
103-
transforms: [transformIn]
104-
}];
105-
106-
var layout = {};
107-
108-
function assertSupplyDefaultsArgs(_transformIn, traceOut, _layout) {
109-
expect(_transformIn).toBe(transformIn);
110-
expect(_layout).toBe(layout);
111-
112-
return transformOut;
113-
}
114-
115-
function assertTransformArgs(dataOut, opts) {
116-
expect(dataOut[0]._input).toBe(dataIn[0]);
117-
expect(opts.transform).toBe(transformOut);
118-
expect(opts.fullTrace._input).toBe(dataIn[0]);
119-
expect(opts.layout).toBe(layout);
120-
121-
return dataOut;
122-
}
123-
124-
var fakeTransformModule = {
125-
moduleType: 'transform',
126-
name: 'fake',
127-
attributes: {},
128-
supplyDefaults: assertSupplyDefaultsArgs,
129-
transform: assertTransformArgs
130-
};
131-
132-
Plotly.register(fakeTransformModule);
133-
Plots.supplyDataDefaults(dataIn, [], layout);
134-
delete Plots.transformsRegistry.fake;
135-
});
136-
13798
it('supplyDataDefaults should apply the transform while', function() {
13899
var dataIn = [{
139100
x: [-2, -2, 1, 2, 3],

test/jasmine/tests/transform_multi_test.js

+45
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
var Plotly = require('@lib/index');
2+
var Plots = require('@src/plots/plots');
23
var Lib = require('@src/lib');
34

45
var createGraphDiv = require('../assets/create_graph_div');
@@ -7,6 +8,50 @@ var assertDims = require('../assets/assert_dims');
78
var assertStyle = require('../assets/assert_style');
89

910

11+
describe('user-defined transforms:', function() {
12+
'use strict';
13+
14+
it('should pass correctly arguments to transform methods', function() {
15+
var transformIn = { type: 'fake' };
16+
var transformOut = {};
17+
18+
var dataIn = [{
19+
transforms: [transformIn]
20+
}];
21+
22+
var layout = {};
23+
24+
function assertSupplyDefaultsArgs(_transformIn, traceOut, _layout) {
25+
expect(_transformIn).toBe(transformIn);
26+
expect(_layout).toBe(layout);
27+
28+
return transformOut;
29+
}
30+
31+
function assertTransformArgs(dataOut, opts) {
32+
expect(dataOut[0]._input).toBe(dataIn[0]);
33+
expect(opts.transform).toBe(transformOut);
34+
expect(opts.fullTrace._input).toBe(dataIn[0]);
35+
expect(opts.layout).toBe(layout);
36+
37+
return dataOut;
38+
}
39+
40+
var fakeTransformModule = {
41+
moduleType: 'transform',
42+
name: 'fake',
43+
attributes: {},
44+
supplyDefaults: assertSupplyDefaultsArgs,
45+
transform: assertTransformArgs
46+
};
47+
48+
Plotly.register(fakeTransformModule);
49+
Plots.supplyDataDefaults(dataIn, [], layout);
50+
delete Plots.transformsRegistry.fake;
51+
});
52+
53+
});
54+
1055
describe('multiple transforms:', function() {
1156
'use strict';
1257

0 commit comments

Comments
 (0)