Skip to content

Commit 374e131

Browse files
committed
Make parallel.js browser ready.
1 parent 055f5ef commit 374e131

File tree

6 files changed

+15
-33
lines changed

6 files changed

+15
-33
lines changed

lib/parallel.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,7 @@
33
var setImmediate = setImmediate || function (cb) {
44
setTimeout(cb, 0);
55
};
6-
if (isNode) {
7-
var Worker = require('./Worker.js');
8-
}
6+
var Worker = isNode ? require(__dirname + '/Worker.js') : self.Worker;
97

108
function extend(from, to) {
119
if (!to) to = {};

test/api.spec.js

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,22 @@
11
describe('API', function () {
2+
var isNode = typeof module !== 'undefined' && module.exports;
3+
var Parallel = isNode ? require('../lib/parallel.js') : self.Parallel;
4+
25
it('should be a constructor', function () {
3-
var Parallel = require('../lib/parallel.js');
46
expect(Parallel).toEqual(jasmine.any(Function));
57
});
68

79
it('should define a .then(cb) function', function () {
8-
var Parallel = require('../lib/parallel.js');
910
var p = new Parallel([1, 2, 3]);
1011
expect(p.then).toEqual(jasmine.any(Function));
1112
});
1213

1314
it('should define a .map(cb) function', function () {
14-
var Parallel = require('../lib/parallel.js');
1515
var p = new Parallel([1, 2, 3]);
1616
expect(p.map).toEqual(jasmine.any(Function));
1717
});
1818

1919
it('should execute a .then function without an operation immediately', function () {
20-
var Parallel = require('../lib/parallel.js');
2120
var p = new Parallel([1, 2, 3]);
2221
expect(p.then).toEqual(jasmine.any(Function));
2322

@@ -33,7 +32,6 @@
3332
});
3433

3534
it('should execute .spawn() correctly', function () {
36-
var Parallel = require('../lib/parallel.js');
3735
var p = new Parallel([1, 2, 3]);
3836

3937
var done = false;
@@ -58,7 +56,6 @@
5856
});
5957

6058
it('should .map() correctly', function () {
61-
var Parallel = require('../lib/parallel.js');
6259
var p = new Parallel([1, 2, 3]);
6360

6461
var done = false;
@@ -83,7 +80,6 @@
8380
});
8481

8582
it('should queue map work correctly', function () {
86-
var Parallel = require('../lib/parallel.js');
8783
var p = new Parallel([1, 2, 3], {maxWorkers: 2});
8884

8985
var done = false;
@@ -108,7 +104,6 @@
108104
});
109105

110106
it('should chain .map() correctly', function () {
111-
var Parallel = require('../lib/parallel.js');
112107
var p = new Parallel([1, 2, 3]);
113108

114109
var done = false;
@@ -135,7 +130,6 @@
135130
});
136131

137132
it('should mix .spawn and .map() correctly', function () {
138-
var Parallel = require('../lib/parallel.js');
139133
var p = new Parallel([1, 2, 3]);
140134

141135
var done = false;
@@ -166,7 +160,6 @@
166160
});
167161

168162
it('should execute .reduce() correctly', function () {
169-
var Parallel = require('../lib/parallel.js');
170163
var p = new Parallel([1, 2, 3]);
171164
var done = false;
172165
var result = null;
@@ -190,7 +183,6 @@
190183
});
191184

192185
it('should process data returned from .then()', function () {
193-
var Parallel = require('../lib/parallel.js');
194186
var p = new Parallel([1, 2, 3]);
195187

196188
var done = false;

test/eval.spec.js

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,9 @@
1-
var isNode = module && module.exports;
2-
3-
if (isNode) {
4-
var Worker = require(__dirname + '/../lib/Worker.js');
5-
}
6-
7-
describe('eval.js', function () {
1+
describe('eval.js', function () {
2+
var isNode = typeof module !== 'undefined' && module.exports;
3+
var Worker = isNode ? require(__dirname + '/../lib/Worker.js') : self.Worker;
84

95
it('should eval the given code', function () {
10-
var wrk = new Worker(__dirname + '/../lib/eval.js');
6+
var wrk = new Worker(isNode ? __dirname + '/../lib/eval.js' : 'lib/eval.js');
117

128
var result = null;
139
var done = false;
@@ -17,7 +13,7 @@ describe('eval.js', function () {
1713
done = true;
1814
wrk.terminate();
1915
};
20-
wrk.postMessage('process.send(JSON.stringify("abc"))');
16+
wrk.postMessage(isNode ? 'process.send(JSON.stringify("abc"))' : 'self.postMessage("abc")');
2117
});
2218

2319
waitsFor(function () {

test/performance.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
describe('Performance', function () {
22
var isNode = typeof module !== 'undefined' && module.exports;
3+
var Parallel = isNode ? require('../lib/parallel.js') : self.Parallel;
34

45
it('.map() should be using multi-threading (could fail on single-core)', function () {
56
var slowSquare = function (n) {
@@ -8,7 +9,6 @@
89
return i;
910
};
1011

11-
var Parallel = require('../lib/parallel.js');
1212
var p = new Parallel([10000, 20000, 30000]);
1313
var p2 = new Parallel([10000, 20000, 30000]);
1414

test/q-api.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
describe('Q-API', function () {
2-
var isNode = module && module.exports;
2+
var isNode = typeof module !== 'undefined' && module.exports;
33

44
if (isNode) {
55
var Q = require('q');

test/worker.spec.js

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,10 @@
1-
var isNode = module && module.exports;
2-
3-
if (isNode) {
4-
var Worker = require(__dirname + '/../lib/Worker.js');
5-
}
6-
7-
describe('WebWorker-API', function () {
1+
describe('WebWorker-API', function () {
2+
var isNode = typeof module !== 'undefined' && module.exports;
3+
var Worker = isNode ? require(__dirname + '/../lib/Worker.js') : self.Worker;
84

95
it('should define the used API', function () {
106
expect(Worker).toEqual(jasmine.any(Function));
11-
var wrk = new Worker(__dirname + '/../lib/eval.js');
7+
var wrk = new Worker(isNode ? __dirname + '/../lib/eval.js' : 'lib/eval.js');
128
expect(wrk.postMessage).toEqual(jasmine.any(Function));
139
expect(wrk.terminate).toEqual(jasmine.any(Function));
1410
wrk.terminate();

0 commit comments

Comments
 (0)