diff options
| author | Daniel Bevenius <daniel.bevenius@gmail.com> | 2019-08-19 14:09:12 +0200 |
|---|---|---|
| committer | Daniel Bevenius <daniel.bevenius@gmail.com> | 2019-08-23 05:41:54 +0200 |
| commit | db3fdfbcb68bb83fab9c0f3c9cb282e41297fa70 (patch) | |
| tree | 42c0f0aa87129ff14edc431cb954d212e17c532a /Makefile | |
| parent | 775048d54c6f190cbc8c0b55c0b53d2ba9d4d028 (diff) | |
| download | android-node-v8-db3fdfbcb68bb83fab9c0f3c9cb282e41297fa70.tar.gz android-node-v8-db3fdfbcb68bb83fab9c0f3c9cb282e41297fa70.tar.bz2 android-node-v8-db3fdfbcb68bb83fab9c0f3c9cb282e41297fa70.zip | |
build: move tooltest to before jstest target
This commit moves the tooltest target to come before the jstest target
to make the console output consistent with it was before I introduced
the tooltest target.
Currently the output looks like this which is might give the impression
that only one test was run:
/Applications/Xcode.app/Contents/Developer/usr/bin/make -s tooltest
.
----------------------------------------------------------------------
Ran 1 test in 0.000s
Compared to the usual:
[03:58|% 100|+ 2739|- 0]: Done
PR-URL: https://github.com/nodejs/node/pull/29220
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>
Diffstat (limited to 'Makefile')
| -rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -313,13 +313,13 @@ coverage-run-js: .PHONY: test # This does not run tests of third-party libraries inside deps. test: all ## Runs default tests, linters, and builds docs. + $(MAKE) -s tooltest $(MAKE) -s test-doc $(MAKE) -s build-addons $(MAKE) -s build-js-native-api-tests $(MAKE) -s build-node-api-tests $(MAKE) -s cctest $(MAKE) -s jstest - $(MAKE) -s tooltest .PHONY: test-only test-only: all ## For a quick test, does not run linter or build docs. |