1berry/nodejs
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

build: streamline JS test suites in Makefile

Browse Source 
* alphabetize the JS test suites
* Use CI_JS_SUITES variable instead of listing all tests

PR-URL: https://github.com/nodejs/node/pull/13340
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
This commit is contained in:
Rich Trott 2017-05-31 10:14:34 -07:00
parent 1609899142
commit 7b56fd8cf0
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Makefile
View File

@ -197,8 +197,8 @@ test: all
$(MAKE) build-addons-napi $(MAKE) build-addons-napi
$(MAKE) cctest $(MAKE) cctest
$(PYTHON) tools/test.py --mode=release -J \ $(PYTHON) tools/test.py --mode=release -J \
doctool inspector known_issues message pseudo-tty parallel sequential \ $(CI_JS_SUITES) \
async-hooks $(CI_NATIVE_SUITES) $(CI_NATIVE_SUITES)
$(MAKE) lint $(MAKE) lint
test-parallel: all test-parallel: all
@ -328,7 +328,7 @@ test-all-valgrind: test-build
$(PYTHON) tools/test.py --mode=debug,release --valgrind $(PYTHON) tools/test.py --mode=debug,release --valgrind
CI_NATIVE_SUITES := addons addons-napi CI_NATIVE_SUITES := addons addons-napi
CI_JS_SUITES := doctool inspector known_issues message parallel pseudo-tty sequential async-hooks CI_JS_SUITES := async-hooks doctool inspector known_issues message parallel pseudo-tty sequential
# Build and test addons without building anything else # Build and test addons without building anything else
test-ci-native: LOGLEVEL := info test-ci-native: LOGLEVEL := info