Skip to content

Commit

Permalink
fix(tests): incorrect test (should use existing result)
Browse files Browse the repository at this point in the history
Since `readFileSync` is only stubbed `onFirstCall` we get a different answer
the second time around which is probably Not What You Want.  But also we
*already checked that config = false*.  So you could just remove this
test, it does nothing useful.
  • Loading branch information
dhdaines committed Aug 14, 2024
1 parent 6d8e90d commit f1fe344
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion test/node-unit/cli/options.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -318,7 +318,7 @@ describe('options', function () {
});

it('should set config = false', function () {
expect(loadOptions(), 'to have property', 'config', false);
expect(result, 'to have property', 'config', false);
});
});

Expand Down

0 comments on commit f1fe344

Please sign in to comment.