public/cordova-android
public/cordova-android
9
0
Fork 0
forked from github/cordova-android
Code Pull Requests Activity

Do not clobber process properties with test mocks (#783)

Browse Source
This commit is contained in:
Raphael von der Grün 2019-07-17 03:01:56 +02:00 committed by GitHub
parent 47c6048d53
commit 997943a194
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
spec/unit/run.spec.js
View File

@ -201,14 +201,7 @@ describe('run', () => {
describe('help', () => {
it('should print out usage and help', () => {
spyOn(console, 'log');
// Rewiring the process object in entirety does not work on NodeJS 12.
// Rewiring members of process however does work
// https://github.com/apache/cordova-android/issues/768
// https://github.com/jhnns/rewire/issues/167
run.__set__('process.exit', _ => null);
run.__set__('process.cwd', _ => '');
run.__set__('process.argv', ['', '']);
spyOn(process, 'exit');
run.help();
expect(console.log).toHaveBeenCalledWith(jasmine.stringMatching(/^Usage:/));