Commit 0127712
build: introduce make jstest
Add a `make jstest` target that runs tests written in JavaScript
(excluding documentation tests which are run in `make test-doc`).
PR-URL: #19324
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Matheus Marchini <matheus@sthima.com>1 parent 92e9ed0 commit 0127712
1 file changed
Lines changed: 16 additions & 12 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -236,36 +236,40 @@ v8: | |||
| 236 | 236 | tools/make-v8.sh | |
| 237 | 237 | $(MAKE) -C deps/v8 $(V8_ARCH).$(BUILDTYPE_LOWER) $(V8_BUILD_OPTIONS) | |
| 238 | 238 | ||
| 239 | + .PHONY: jstest | ||
| 240 | + jstest: build-addons build-addons-napi ## Runs addon tests and JS tests | ||
| 241 | + $(PYTHON) tools/test.py --mode=release -J \ | ||
| 242 | + $(CI_JS_SUITES) \ | ||
| 243 | + $(CI_NATIVE_SUITES) | ||
| 244 | + | ||
| 239 | 245 | .PHONY: test | |
| 240 | 246 | # This does not run tests of third-party libraries inside deps. | |
| 241 | 247 | test: all ## Runs default tests, linters, and builds docs. | |
| 248 | + # Build the addons before running the tests so the test results | ||
| 249 | + # can be displayed together | ||
| 242 | 250 | $(MAKE) -s build-addons | |
| 243 | 251 | $(MAKE) -s build-addons-napi | |
| 244 | - $(MAKE) -s doc-only | ||
| 245 | - $(MAKE) -s lint | ||
| 252 | + $(MAKE) -s test-doc | ||
| 246 | 253 | $(MAKE) -s cctest | |
| 247 | - $(PYTHON) tools/test.py --mode=release -J \ | ||
| 248 | - $(CI_JS_SUITES) \ | ||
| 249 | - $(CI_NATIVE_SUITES) \ | ||
| 250 | - $(CI_DOC) | ||
| 254 | + $(MAKE) -s jstest | ||
| 251 | 255 | ||
| 252 | 256 | .PHONY: test-only | |
| 253 | 257 | test-only: all ## For a quick test, does not run linter or build docs. | |
| 258 | + # Build the addons before running the tests so the test results | ||
| 259 | + # can be displayed together | ||
| 254 | 260 | $(MAKE) build-addons | |
| 255 | 261 | $(MAKE) build-addons-napi | |
| 256 | 262 | $(MAKE) cctest | |
| 257 | - $(PYTHON) tools/test.py --mode=release -J \ | ||
| 258 | - $(CI_JS_SUITES) \ | ||
| 259 | - $(CI_NATIVE_SUITES) | ||
| 263 | + $(MAKE) jstest | ||
| 260 | 264 | ||
| 261 | 265 | # Used by `make coverage-test` | |
| 262 | 266 | test-cov: all | |
| 267 | + # Build the addons before running the tests so the test results | ||
| 268 | + # can be displayed together | ||
| 263 | 269 | $(MAKE) build-addons | |
| 264 | 270 | $(MAKE) build-addons-napi | |
| 265 | 271 | # $(MAKE) cctest | |
| 266 | - $(PYTHON) tools/test.py --mode=release -J \ | ||
| 267 | - $(CI_JS_SUITES) \ | ||
| 268 | - $(CI_NATIVE_SUITES) | ||
| 272 | + $(MAKE) jstest | ||
| 269 | 273 | $(MAKE) lint | |
| 270 | 274 | ||
| 271 | 275 | test-parallel: all | |
0 commit comments