Tree: a290ddfdc9

Commit Graph

8 Commits (a290ddfdc9cd9453886d433060c5132095d916c4)

Author	SHA1	Message	Date
Sakthipriyan Vairamani	89ede72fca	tools: remove unnecessary imports and assignments PR-URL: https://github.com/nodejs/node/pull/7483 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net>	9 years ago
Sakthipriyan Vairamani	52f84add77	test: fix default value for additional param In Python, the default values of parameters are evaluated only once during their declaration. So, whenever the default parameter is used the same object will be used. Since we use a list, which is a mutable object, this could lead to unexpected results. PR-URL: https://github.com/nodejs/node/pull/2553 Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>	10 years ago
Rich Trott	a6b8ee19b8	test: create temp dir in common.js Move creation of temporary directories for tests out of the Python harness and into common.js. This allows all tests to be run reliably outside of the Python wrapper. PR-URL: https://github.com/nodejs/io.js/pull/1877 Reviewed-By: Rod Vagg <rod@vagg.org> Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>	10 years ago
Ben Noordhuis	dd260d2152	test: fix `make test-addons` target I broke the python script in commit edaf7af but because test-all does not run test-addons, it slipped under the radar. Mea culpa. Fixes the following test runner error: Traceback (most recent call last): File "tools/test.py", line 1522, in <module> sys.exit(Main()) File "tools/test.py", line 1454, in Main test_list = root.ListTests([], path, context, arch, mode) File "tools/test.py", line 720, in ListTests test.AddTestsToList(result, full_path, path, context, arch, mode) File "tools/test.py", line 690, in AddTestsToList arch, mode) File "/home/bnoordhuis/src/v1.x/test/gc/../testpy/__init__.py", line 176, in ListTests result.append(SimpleTestCase(test, file_path, mode, self.context, self)) TypeError: __init__() takes at least 7 arguments (6 given) PR-URL: https://github.com/iojs/io.js/pull/313 Reviewed-By: Chris Dickinson <christopher.s.dickinson@gmail.com>	10 years ago
Fedor Indutny	b7d247856e	test: run tests in parallel, common improvements * Allow running tests in mixed parallel/sequential modes * Add -J flag for running tests on all available CPUs * Support TEST_THREAD_ID in test/common.js and use it for tmpDir and PORT * make: use -J flag Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> PR-URL: https://github.com/iojs/io.js/pull/172 Fix: iojs/io.js#139	10 years ago
Ben Noordhuis	edaf7af30b	test: make test runner multi-arch/mode compatible Make `python tools/test.py --arch=ia32,x64 --mode=debug,release` work. The test runner looks for the `node` binary in `out/${arch}.${mode}/`. Running tools/test.py without --arch makes it use `out/Release/node` or `out/Debug/node` like before. This commit removes `test/simple/test-executable-path.js` because the assumptions it makes about the locations of the debug and release binaries are now outdated. PR-URL: https://github.com/node-forward/node/pull/24 Reviewed-By: Fedor Indutny <fedor@indutny.com>	11 years ago
Fedor Indutny	1442c1c6de	addons: build and test examples fix #6910	11 years ago
Timothy J Fontaine	6cc95b06ea	test: refactor to use common testcfg	11 years ago