Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

wp-scripts test-unit-js node debugging #21631

Merged
merged 8 commits into from
Apr 24, 2020
Merged

wp-scripts test-unit-js node debugging #21631

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
8167689
Pass node inspect flags to wp-scripts
ajlende Apr 16, 2020
b09a3d4
Add debugging instructions in wp-scripts README
ajlende Apr 16, 2020
cd26cac
Add tests for passing node args to spawnScript
ajlende Apr 16, 2020
6ac725d
Add a test-unit:debug to npm scripts
ajlende Apr 16, 2020
3b4d85e
Add section in testing overview for debugging jest tests
ajlende Apr 16, 2020
823366d
Allow all node options
ajlende Apr 23, 2020
c60a060
Add clarification for debugging with other browsers/IDEs
ajlende Apr 23, 2020
a79138a
Update packages/scripts/README.md
gziolo Apr 24, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add tests for passing node args to spawnScript
  • Loading branch information
@ajlende
ajlende committed Apr 16, 2020
commit cd26cac2d5b1fcca5e9617eac005b77ab1fdedf7
26 changes: 26 additions & 0 deletions packages/scripts/utils/test/index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,32 @@ describe( 'utils', () => {
expect( console ).toHaveLogged();
} );

test( 'should pass inspect args to node', () => {
crossSpawnMock.mockReturnValueOnce( { status: 0 } );

expect( () =>
spawnScript( scriptName, [], [ '--inspect-brk' ] )
).toThrow( 'Exit code: 0.' );
expect( crossSpawnMock ).toHaveBeenCalledWith(
'node',
[ '--inspect-brk', expect.stringContaining( scriptName ) ],
{ stdio: 'inherit' }
);
} );

test( 'should pass script args to the script', () => {
crossSpawnMock.mockReturnValueOnce( { status: 0 } );

expect( () =>
spawnScript( scriptName, [ '--runInBand' ] )
).toThrow( 'Exit code: 0.' );
expect( crossSpawnMock ).toHaveBeenCalledWith(
'node',
[ expect.stringContaining( scriptName ), '--runInBand' ],
{ stdio: 'inherit' }
);
} );

test( 'should finish successfully when the script properly executed', () => {
crossSpawnMock.mockReturnValueOnce( { status: 0 } );

Expand Down