Skip to content

Commit

Permalink
Merge pull request #124 from toddy15/main
Browse files Browse the repository at this point in the history
Fix test failure due to missing atom command
  • Loading branch information
DeeDeeG authored Feb 1, 2024
2 parents b084b11 + 96e33d0 commit 7237881
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions spec/test-spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,8 +30,8 @@ describe('apm test', () => {
waitsFor('waiting for test to complete', () => atomSpawn.callCount === 1);

runs(() => {
// On Windows, there's a suffix (atom.cmd), so we only check that atom is _included_ in the path
expect(atomSpawn.mostRecentCall.args[0].indexOf('atom')).not.toBe(-1);
// On Windows, there's a suffix (pulsar.cmd), so we only check that pulsar is _included_ in the path
expect(atomSpawn.mostRecentCall.args[0].indexOf('pulsar')).not.toBe(-1);
expect(atomSpawn.mostRecentCall.args[1][0]).toEqual('--dev');
expect(atomSpawn.mostRecentCall.args[1][1]).toEqual('--test');
expect(atomSpawn.mostRecentCall.args[1][2]).toEqual(specPath);
Expand Down
2 changes: 1 addition & 1 deletion src/test.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ to the current working directory).\

if (options.argv.path) { atomCommand = options.argv.path; }
if (!fs.existsSync(atomCommand)) {
atomCommand = 'atom';
atomCommand = 'pulsar';
if (process.platform === 'win32') { atomCommand += '.cmd'; }
}

Expand Down

0 comments on commit 7237881

Please sign in to comment.