Skip to content

Commit 4b8cdf2

Browse files
ShMcKShMcK
committed
remove handleLog
1 parent f64f02e commit 4b8cdf2

File tree

6 files changed

+154
-154
lines changed

6 files changed

+154
-154
lines changed

‎lib/create-runner.js‎

Lines changed: 43 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -1,43 +1,43 @@
1-
"use strict";
2-
var path = require('path');
3-
var exists_1 = require('./exists');
4-
var spawn = require('child_process').spawn;
5-
var node = null;
6-
if (process.platform === 'darwin' && process.resourcesPath) {
7-
node = path.resolve(process.resourcesPath, '..', 'Frameworks', 'Atom Helper.app', 'Contents', 'MacOS', 'Atom Helper');
8-
}
9-
else if (process.platform.match(/win/)) {
10-
node = 'node';
11-
}
12-
else {
13-
node = process.execPath;
14-
}
15-
var mocha = path.join(__dirname, '..', '..', 'mocha', 'bin', 'mocha');
16-
if (!exists_1.fileExists(mocha)) {
17-
mocha = path.join(__dirname, '..', 'node_modules', 'mocha', 'bin', 'mocha');
18-
if (!exists_1.fileExists(mocha)) {
19-
var error = 'Error finding mocha';
20-
throw (error);
21-
}
22-
}
23-
function createRunner(config, testFile) {
24-
var options = {
25-
cwd: config.dir
26-
};
27-
if (options.env == null) {
28-
options.env = Object.create(process.env);
29-
}
30-
options.env.ATOM_SHELL_INTERNAL_RUN_AS_NODE = 1;
31-
options.env.DIR = config.dir;
32-
options.env.TUTORIAL_DIR = config.tutorialDir;
33-
options.env.TASK_POSITION = config.taskPosition;
34-
return spawn(node, [
35-
mocha,
36-
'--bail',
37-
'--harmony',
38-
'--no-colors',
39-
("--reporter=" + path.join(__dirname, 'reporter.js')),
40-
testFile
41-
], options);
42-
}
43-
exports.createRunner = createRunner;
1+
"use strict";
2+
var path = require('path');
3+
var exists_1 = require('./exists');
4+
var spawn = require('child_process').spawn;
5+
var node = null;
6+
if (process.platform === 'darwin' && process.resourcesPath) {
7+
node = path.resolve(process.resourcesPath, '..', 'Frameworks', 'Atom Helper.app', 'Contents', 'MacOS', 'Atom Helper');
8+
}
9+
else if (process.platform.match(/win/)) {
10+
node = 'node';
11+
}
12+
else {
13+
node = process.execPath;
14+
}
15+
var mocha = path.join(__dirname, '..', '..', 'mocha', 'bin', 'mocha');
16+
if (!exists_1.fileExists(mocha)) {
17+
mocha = path.join(__dirname, '..', 'node_modules', 'mocha', 'bin', 'mocha');
18+
if (!exists_1.fileExists(mocha)) {
19+
var error = 'Error finding mocha';
20+
throw (error);
21+
}
22+
}
23+
function createRunner(config, testFile) {
24+
var options = {
25+
cwd: config.dir
26+
};
27+
if (options.env == null) {
28+
options.env = Object.create(process.env);
29+
}
30+
options.env.ATOM_SHELL_INTERNAL_RUN_AS_NODE = 1;
31+
options.env.DIR = config.dir;
32+
options.env.TUTORIAL_DIR = config.tutorialDir;
33+
options.env.TASK_POSITION = config.taskPosition;
34+
return spawn(node, [
35+
mocha,
36+
'--bail',
37+
'--harmony',
38+
'--no-colors',
39+
("--reporter=" + path.join(__dirname, 'reporter.js')),
40+
testFile
41+
], options);
42+
}
43+
exports.createRunner = createRunner;

‎lib/exists.js‎

Lines changed: 17 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
1-
"use strict";
2-
var fs = require('fs');
3-
function fileExists(path) {
4-
try {
5-
fs.accessSync(path, fs.F_OK);
6-
}
7-
catch (e) {
8-
if (e) {
9-
if (e.code !== 'ENOENT') {
10-
console.log(e);
11-
}
12-
return false;
13-
}
14-
}
15-
return true;
16-
}
17-
exports.fileExists = fileExists;
1+
"use strict";
2+
var fs = require('fs');
3+
function fileExists(path) {
4+
try {
5+
fs.accessSync(path, fs.F_OK);
6+
}
7+
catch (e) {
8+
if (e) {
9+
if (e.code !== 'ENOENT') {
10+
console.log(e);
11+
}
12+
return false;
13+
}
14+
}
15+
return true;
16+
}
17+
exports.fileExists = fileExists;

‎lib/reporter.js‎

Lines changed: 41 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -1,41 +1,41 @@
1-
"use strict";
2-
var utils_1 = require('./utils');
3-
exports = module.exports = reporter;
4-
function reporter(runner) {
5-
var result = {
6-
passes: [],
7-
failures: [],
8-
pass: true
9-
};
10-
runner.on('pass', function (test) {
11-
var title = test.fullTitle();
12-
var obj = getIndexAndTitle(title);
13-
result.passes.push({
14-
msg: "Task " + obj.index + " Complete",
15-
taskPosition: obj.index
16-
});
17-
});
18-
runner.on('fail', function (test, err) {
19-
var title = test.fullTitle();
20-
var obj = getIndexAndTitle(title);
21-
result.failures.push({
22-
msg: obj.msg,
23-
taskPosition: obj.index - 1,
24-
timedOut: test.timedOut
25-
});
26-
result.pass = false;
27-
});
28-
runner.on('end', function () {
29-
process.stdout.write(utils_1.signal + JSON.stringify(result, null, 2));
30-
});
31-
function getIndexAndTitle(title) {
32-
var indexString = title.match(/^[0-9]+/);
33-
if (!indexString) {
34-
throw 'Tests should begin with a number, indicating the task number';
35-
}
36-
return {
37-
index: parseInt(indexString[0], 10),
38-
msg: title.slice(indexString[0].length + 1)
39-
};
40-
}
41-
}
1+
"use strict";
2+
var utils_1 = require('./utils');
3+
exports = module.exports = reporter;
4+
function reporter(runner) {
5+
var result = {
6+
passes: [],
7+
failures: [],
8+
pass: true
9+
};
10+
runner.on('pass', function (test) {
11+
var title = test.fullTitle();
12+
var obj = getIndexAndTitle(title);
13+
result.passes.push({
14+
msg: "Task " + obj.index + " Complete",
15+
taskPosition: obj.index
16+
});
17+
});
18+
runner.on('fail', function (test, err) {
19+
var title = test.fullTitle();
20+
var obj = getIndexAndTitle(title);
21+
result.failures.push({
22+
msg: obj.msg,
23+
taskPosition: obj.index - 1,
24+
timedOut: test.timedOut
25+
});
26+
result.pass = false;
27+
});
28+
runner.on('end', function () {
29+
process.stdout.write(utils_1.signal + JSON.stringify(result, null, 2));
30+
});
31+
function getIndexAndTitle(title) {
32+
var indexString = title.match(/^[0-9]+/);
33+
if (!indexString) {
34+
throw 'Tests should begin with a number, indicating the task number';
35+
}
36+
return {
37+
index: parseInt(indexString[0], 10),
38+
msg: title.slice(indexString[0].length + 1)
39+
};
40+
}
41+
}

‎lib/runner.js‎

Lines changed: 49 additions & 49 deletions
Original file line numberDiff line numberDiff line change
@@ -1,49 +1,49 @@
1-
"use strict";
2-
var utils_1 = require('./utils');
3-
var create_runner_1 = require('./create-runner');
4-
function runner(testFile, config, handleResult, handleLog) {
5-
var runner = create_runner_1.createRunner(config, testFile);
6-
var final = null;
7-
var signalMatch = new RegExp(utils_1.signal);
8-
return new Promise(function (resolve, reject) {
9-
runner.stdout.on('data', function (data) {
10-
data = data.toString();
11-
var match = signalMatch.exec(data);
12-
if (!match) {
13-
handleLog(data);
14-
return;
15-
}
16-
var resultString = data.substring(match.index + utils_1.signal.length);
17-
var result = JSON.parse(JSON.stringify(resultString));
18-
if (typeof result === 'string') {
19-
result = JSON.parse(result);
20-
}
21-
if (result.pass) {
22-
final = result.passes[result.passes.length - 1];
23-
}
24-
else if (result.pass === false) {
25-
final = result.failures[0];
26-
}
27-
else {
28-
console.log('error processing result: ', result);
29-
}
30-
final.change = final.taskPosition - config.taskPosition;
31-
final.pass = final.change > 0;
32-
final.completed = result.pass;
33-
handleResult(final);
34-
});
35-
runner.stderr.on('data', function (data) {
36-
console.log('test error', data.toString());
37-
});
38-
runner.on('close', function (code) {
39-
if (code === 0) {
40-
resolve(final);
41-
}
42-
else {
43-
resolve(final);
44-
}
45-
});
46-
});
47-
}
48-
Object.defineProperty(exports, "__esModule", { value: true });
49-
exports.default = runner;
1+
"use strict";
2+
var utils_1 = require('./utils');
3+
var create_runner_1 = require('./create-runner');
4+
function runner(testFile, config, handleResult) {
5+
var runner = create_runner_1.createRunner(config, testFile);
6+
var final = null;
7+
var signalMatch = new RegExp(utils_1.signal);
8+
return new Promise(function (resolve, reject) {
9+
runner.stdout.on('data', function (data) {
10+
data = data.toString();
11+
var match = signalMatch.exec(data);
12+
if (!match) {
13+
console.log(data);
14+
return;
15+
}
16+
var resultString = data.substring(match.index + utils_1.signal.length);
17+
var result = JSON.parse(JSON.stringify(resultString));
18+
if (typeof result === 'string') {
19+
result = JSON.parse(result);
20+
}
21+
if (result.pass) {
22+
final = result.passes[result.passes.length - 1];
23+
}
24+
else if (result.pass === false) {
25+
final = result.failures[0];
26+
}
27+
else {
28+
console.log('error processing result: ', result);
29+
}
30+
final.change = final.taskPosition - config.taskPosition;
31+
final.pass = final.change > 0;
32+
final.completed = result.pass;
33+
handleResult(final);
34+
});
35+
runner.stderr.on('data', function (data) {
36+
console.log('test error', data.toString());
37+
});
38+
runner.on('close', function (code) {
39+
if (code === 0) {
40+
resolve(final);
41+
}
42+
else {
43+
resolve(final);
44+
}
45+
});
46+
});
47+
}
48+
Object.defineProperty(exports, "__esModule", { value: true });
49+
exports.default = runner;

‎lib/utils.js‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
1-
"use strict";
2-
exports.signal = '@@@CodeRoad Results@@@';
1+
"use strict";
2+
exports.signal = '@@@CodeRoad Results@@@';

‎src/runner.ts‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import {signal} from './utils';
22
import {createRunner} from './create-runner';
33

44
export default function runner(testFile: string, config: CR.Config,
5-
handleResult: (result) => CR.TestResult, handleLog: (log) => string) {
5+
handleResult: (result) => CR.TestResult) {
66

77
let runner = createRunner(config, testFile);
88
var final = null;
@@ -16,7 +16,7 @@ export default function runner(testFile: string, config: CR.Config,
1616
let match = signalMatch.exec(data); // 0
1717

1818
if (!match) {
19-
handleLog(data);
19+
console.log(data);
2020
return;
2121
}
2222

0 commit comments

Comments
 (0)