Skip to content

Commit ca9ce37

Browse files
Krinkletrentmwillis
authored andcommitted
CLI: Move non-bin files from bin/ to src/cli/
Follows-up #1271.
1 parent b54e732 commit ca9ce37

File tree

9 files changed

+22
-17
lines changed

9 files changed

+22
-17
lines changed

bin/qunit

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@
33
"use strict";
44

55
const program = require( "commander" );
6-
const run = require( "./run" );
7-
const FindReporter = require( "./find-reporter" );
6+
const run = require( "../src/cli/run" );
7+
const FindReporter = require( "../src/cli/find-reporter" );
88
const pkg = require( "../package.json" );
99

1010
const findReporter = FindReporter.findReporter;

src/cli/.eslintrc.json

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,5 @@
1+
{
2+
"env": {
3+
"node": true
4+
}
5+
}

bin/find-reporter.js renamed to src/cli/find-reporter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ const JSReporters = require( "js-reporters" );
44
const path = require( "path" );
55
const findup = require( "findup-sync" );
66
const utils = require( "./utils" );
7-
const pkg = require( "../package.json" );
7+
const pkg = require( "../../package.json" );
88

99
function findReporter( reporterName ) {
1010
if ( !reporterName ) {
File renamed without changes.

bin/require-qunit.js renamed to src/cli/require-qunit.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,14 +13,14 @@ module.exports = function requireQUnit() {
1313
try {
1414

1515
// Second, we use the globally installed QUnit
16-
delete require.cache[ resolve.sync( "../qunit/qunit" ) ];
17-
return require( "../qunit/qunit" );
16+
delete require.cache[ resolve.sync( "../../qunit/qunit" ) ];
17+
return require( "../../qunit/qunit" );
1818
} catch ( e ) {
1919
if ( e.code === "MODULE_NOT_FOUND" ) {
2020

2121
// Finally, we use the local development version of QUnit
22-
delete require.cache[ resolve.sync( "../dist/qunit" ) ];
23-
return require( "../dist/qunit" );
22+
delete require.cache[ resolve.sync( "../../dist/qunit" ) ];
23+
return require( "../../dist/qunit" );
2424
}
2525

2626
throw e;
File renamed without changes.
File renamed without changes.

test/cli/custom-reporter.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ const co = require( "co" );
22
const JSReporters = require( "js-reporters" );
33
const NPMReporter = require( "npm-reporter" );
44

5-
const findReporter = require( "../../bin/find-reporter" ).findReporter;
5+
const findReporter = require( "../../src/cli/find-reporter" ).findReporter;
66

77
const expectedOutput = require( "./fixtures/expected/tap-outputs" );
88
const execute = require( "./helpers/execute" );

test/cli/require-qunit-test.js

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ QUnit.module( "requireQUnit", function() {
1010
const localQUnit = {
1111
"@noCallThru": true
1212
};
13-
const requireQUnit = proxyquire( "../../bin/require-qunit", {
13+
const requireQUnit = proxyquire( "../../src/cli/require-qunit", {
1414
"resolve": resolveStub,
1515
"qunit": localQUnit
1616
} );
@@ -22,10 +22,10 @@ QUnit.module( "requireQUnit", function() {
2222
const globalQUnit = {
2323
"@noCallThru": true
2424
};
25-
const requireQUnit = proxyquire( "../../bin/require-qunit", {
25+
const requireQUnit = proxyquire( "../../src/cli/require-qunit", {
2626
"resolve": resolveStub,
2727
"qunit": null,
28-
"../qunit/qunit": globalQUnit
28+
"../../qunit/qunit": globalQUnit
2929
} );
3030

3131
assert.strictEqual( requireQUnit(), globalQUnit );
@@ -35,22 +35,22 @@ QUnit.module( "requireQUnit", function() {
3535
const devQUnit = {
3636
"@noCallThru": true
3737
};
38-
const requireQUnit = proxyquire( "../../bin/require-qunit", {
38+
const requireQUnit = proxyquire( "../../src/cli/require-qunit", {
3939
"resolve": resolveStub,
4040
"qunit": null,
41-
"../qunit/qunit": null,
42-
"../dist/qunit": devQUnit
41+
"../../qunit/qunit": null,
42+
"../../dist/qunit": devQUnit
4343
} );
4444

4545
assert.strictEqual( requireQUnit(), devQUnit );
4646
} );
4747

4848
QUnit.test( "throws error if none of the modules are found", function( assert ) {
49-
const requireQUnit = proxyquire( "../../bin/require-qunit", {
49+
const requireQUnit = proxyquire( "../../src/cli/require-qunit", {
5050
"resolve": resolveStub,
5151
"qunit": null,
52-
"../qunit/qunit": null,
53-
"../dist/qunit": null
52+
"../../qunit/qunit": null,
53+
"../../dist/qunit": null
5454
} );
5555

5656
assert.throws( requireQUnit, /Cannot find module/ );

0 commit comments

Comments
 (0)