Skip to content

Commit 6f6cd47

Browse files
fix: Add missing methods, and shorten how they are defined
1 parent e6a56b9 commit 6f6cd47

File tree

3 files changed

+94
-37
lines changed

3 files changed

+94
-37
lines changed

Diff for: index.js

+31-34
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,11 @@ function apiDecorate(kernel) {
9797
kernel.context = flag;
9898
return kernel;
9999
};
100+
kernel.destroy = () => {};
101+
kernel.validateSettings = () => {};
102+
if (kernel.graphical && kernel.output) {
103+
setupGraphical(kernel);
104+
}
100105
kernel.exec = function() {
101106
return new Promise((resolve, reject) => {
102107
try {
@@ -136,40 +141,32 @@ function apiDecorate(kernel) {
136141
};
137142

138143
// these are added for api compatibility, but have no affect
139-
kernel.setWarnVarUsage = () => {
140-
return kernel;
141-
};
142-
kernel.setOptimizeFloatMemory = () => {
143-
return kernel;
144-
};
145-
kernel.setArgumentTypes = () => {
146-
return kernel;
147-
};
148-
kernel.setDebug = () => {
149-
return kernel;
150-
};
151-
kernel.setLoopMaxIterations = () => {
152-
return kernel;
153-
};
154-
kernel.setPipeline = () => {
155-
return kernel;
156-
};
157-
kernel.setPrecision = () => {
158-
return kernel;
159-
};
160-
kernel.setImmutable = () => {
161-
return kernel;
162-
};
163-
kernel.setFunctions = () => {
164-
return kernel;
165-
};
166-
kernel.addSubKernel = () => {
167-
return kernel;
168-
};
169-
kernel.destroy = () => {};
170-
kernel.validateSettings = () => {};
171-
if (kernel.graphical && kernel.output) {
172-
setupGraphical(kernel);
144+
const mockMethod = () => kernel;
145+
const methods = [
146+
'setWarnVarUsage',
147+
'setArgumentTypes',
148+
'setTactic',
149+
'setOptimizeFloatMemory',
150+
'setDebug',
151+
'setLoopMaxIterations',
152+
'setConstantTypes',
153+
'setFunctions',
154+
'setNativeFunctions',
155+
'setInjectedNative',
156+
'setPipeline',
157+
'setPrecision',
158+
'setOutputToTexture',
159+
'setImmutable',
160+
'setStrictIntegers',
161+
'setDynamicOutput',
162+
'setHardcodeConstants',
163+
'setDynamicArguments',
164+
'setUseLegacyEncoder',
165+
'setWarnVarUsage',
166+
'addSubKernel',
167+
];
168+
for (let i = 0; i < methods.length; i++) {
169+
kernel[methods[i]] = mockMethod;
173170
}
174171
return kernel;
175172
}

Diff for: package.json

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "gpu-mock.js",
3-
"version": "1.1.0",
3+
"version": "1.1.1",
44
"description": "A simple mocker for testing threaded development",
55
"files": [
66
"index.js"

Diff for: test/index.js

+62-2
Original file line numberDiff line numberDiff line change
@@ -163,7 +163,7 @@ describe('gpuMock', () => {
163163
it('sets value', () => {
164164
const constants = {};
165165
const kernel = gpuMock(function() {});
166-
kernel.setConstants(constants);
166+
assert.equal(kernel.setConstants(constants), kernel);
167167
assert.equal(kernel.constants, constants);
168168
});
169169
});
@@ -187,7 +187,7 @@ describe('gpuMock', () => {
187187
it('sets value', () => {
188188
const context = {};
189189
const kernel = gpuMock(function() {});
190-
kernel.setContext(context);
190+
assert.equal(kernel.setContext(context), kernel);
191191
assert.equal(kernel.context, context);
192192
});
193193
});
@@ -228,6 +228,48 @@ describe('gpuMock', () => {
228228
assert.equal(kernel.setArgumentTypes(), kernel);
229229
});
230230
});
231+
describe('.setConstantTypes()', () => {
232+
it('can be called and return kernel', () => {
233+
const kernel = gpuMock(function() {});
234+
assert.equal(kernel.setConstantTypes(), kernel);
235+
});
236+
});
237+
describe('.setStrictIntegers()', () => {
238+
it('can be called and return kernel', () => {
239+
const kernel = gpuMock(function() {});
240+
assert.equal(kernel.setStrictIntegers(), kernel);
241+
});
242+
});
243+
describe('.setDynamicOutput()', () => {
244+
it('can be called and return kernel', () => {
245+
const kernel = gpuMock(function() {});
246+
assert.equal(kernel.setDynamicOutput(), kernel);
247+
});
248+
});
249+
describe('.setDynamicArguments()', () => {
250+
it('can be called and return kernel', () => {
251+
const kernel = gpuMock(function() {});
252+
assert.equal(kernel.setDynamicArguments(), kernel);
253+
});
254+
});
255+
describe('.setUseLegacyEncoder()', () => {
256+
it('can be called and return kernel', () => {
257+
const kernel = gpuMock(function() {});
258+
assert.equal(kernel.setUseLegacyEncoder(), kernel);
259+
});
260+
});
261+
describe('.setHardcodeConstants()', () => {
262+
it('can be called and return kernel', () => {
263+
const kernel = gpuMock(function() {});
264+
assert.equal(kernel.setHardcodeConstants(), kernel);
265+
});
266+
});
267+
describe('.setTactic()', () => {
268+
it('can be called and return kernel', () => {
269+
const kernel = gpuMock(function() {});
270+
assert.equal(kernel.setTactic(), kernel);
271+
});
272+
});
231273
describe('.setDebug()', () => {
232274
it('can be called and return kernel', () => {
233275
const kernel = gpuMock(function() {});
@@ -252,6 +294,12 @@ describe('gpuMock', () => {
252294
assert.equal(kernel.setPrecision(), kernel);
253295
});
254296
});
297+
describe('.setOutputToTexture()', () => {
298+
it('can be called and return kernel', () => {
299+
const kernel = gpuMock(function() {});
300+
assert.equal(kernel.setOutputToTexture(), kernel);
301+
});
302+
});
255303
describe('.setImmutable()', () => {
256304
it('can be called and return kernel', () => {
257305
const kernel = gpuMock(function() {});
@@ -264,6 +312,18 @@ describe('gpuMock', () => {
264312
assert.equal(kernel.setFunctions(), kernel);
265313
});
266314
});
315+
describe('.setNativeFunctions()', () => {
316+
it('can be called and return kernel', () => {
317+
const kernel = gpuMock(function() {});
318+
assert.equal(kernel.setNativeFunctions(), kernel);
319+
});
320+
});
321+
describe('.setInjectedNative()', () => {
322+
it('can be called and return kernel', () => {
323+
const kernel = gpuMock(function() {});
324+
assert.equal(kernel.setInjectedNative(), kernel);
325+
});
326+
});
267327
describe('.addSubKernel()', () => {
268328
it('can be called and return kernel', () => {
269329
const kernel = gpuMock(function() {});

0 commit comments

Comments
 (0)