delta/node-new.git/test/testpy/init.py, branch Remove-Python-3-tests

tools: python: ignore instead of select flake8 rules

2019-04-14T00:33:06+00:00

PR-URL: https://github.com/nodejs/node/pull/25614
Reviewed-By: Sakthipriyan Vairamani

test: remove Files: comment processing from Python test runner

2018-12-23T17:46:14+00:00

We don't use any Files: comments in our tests so remove the Python code
for it from test/testpy/__init__.py.

PR-URL: https://github.com/nodejs/node/pull/25183
Reviewed-By: Richard Lau 
Reviewed-By: Michaël Zasso 
Reviewed-By: Luigi Pinca 
Reviewed-By: Trivikram Kamat

build: add a space to clarify skipping crypto msg

2018-12-17T09:35:24+00:00

This commit adds a space to the message that is displayed for tests that
are skipped when node was built --without-ssl. For example, this is what
is currently displayed:
"release test-https-agent-additional-optionsSkipping as node was
compiled without crypto support"

After this change this will be:
"release test-https-agent-additional-options: Skipping as node was
compiled without crypto support"

PR-URL: https://github.com/nodejs/node/pull/25011
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Richard Lau 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Luigi Pinca

test: from functools import reduce in test/testpy/init.py

2018-12-14T01:20:12+00:00

$ __make lint-py__  # When run on Python 3
```
PYTHONPATH=tools/pip python -m flake8 . \
		--count --show-source --statistics --select=E901,E999,F821,F822,F823 \
		--exclude=.git,deps,lib,src,tools/*_macros.py,tools/gyp,tools/inspector_protocol,tools/jinja2,tools/markupsafe,tools/pip
./test/testpy/__init__.py:119:37: F821 undefined name 'reduce'
        file_path = join(self.root, reduce(join, test[1:], ""))
                                    ^
./test/testpy/__init__.py:161:37: F821 undefined name 'reduce'
        file_path = join(self.root, reduce(join, test[1:], "") + ".js")
                                    ^
2     F821 undefined name 'reduce'
2
make: *** [lint-py] Error 1
```

PR-URL: https://github.com/nodejs/node/pull/24954
Reviewed-By: Richard Lau 
Reviewed-By: Rich Trott 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Anna Henningsen

test: skip test that use --tls-v1.x flags

2018-11-18T12:26:08+00:00

Currently, configuring --without-ssl will cause the following test to
fail:

=== release test-https-agent-additional-options ===
Path: parallel/test-https-agent-additional-options
out/Release/node: bad option: --tls-v1.1
Command: out/Release/node --tls-v1.1
  /node/test/parallel/test-https-agent-additional-options.js

=== release test-https-agent-session-eviction ===
Path: parallel/test-https-agent-session-eviction
out/Release/node: bad option: --tls-v1.0

Command: out/Release/node --tls-v1.0
  /node/test/parallel/test-https-agent-session-eviction.js

This commit adds a check for the --tls-v.x flags and skips them if node
was built without crypto support.

PR-URL: https://github.com/nodejs/node/pull/24376
Reviewed-By: Refael Ackermann 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Sakthipriyan Vairamani 
Reviewed-By: Franziska Hinkelmann 
Reviewed-By: Rod Vagg

tools,test: cleanup and dedup code

2018-10-16T15:09:53+00:00

* Hoist common code to base class
  (`GetTestStatus`, and the `section` property to `TestConfiguration`)
* Replace ListSet with the built in set
* Remove ClassifiedTest
* Inline PrintReport
* How cases_to_run are filtered

PR-URL: https://github.com/nodejs/node/pull/23251
Reviewed-By: Rich Trott

test: remove custom AsyncHooksTestConfiguration

2018-07-31T03:50:17+00:00

Has the same behavior as `ParallelTestConfiguration`

PR-URL: https://github.com/nodejs/node/pull/22008
Reviewed-By: Rich Trott 
Reviewed-By: Ruben Bridgewater

test: simplify common.PORT code

2017-12-12T03:49:30+00:00

common.PORT is no longer used in parallelized tests and should not be.
Remove code that accommodates parallelized tests.

PR-URL: https://github.com/nodejs/node/pull/17559
Reviewed-By: Luigi Pinca 
Reviewed-By: Gibson Fahnestock

module: Allow runMain to be ESM

2017-09-07T20:18:32+00:00

This follows the EPS an allows the node CLI to have ESM as an entry point.
`node ./example.mjs`. A newer V8 is needed for `import()` so that is not
included. `import.meta` is still in specification stage so that also is not
included.

PR-URL: https://github.com/nodejs/node/pull/14369
Author: Bradley Farias 
Author: Guy Bedford 
Author: Jan Krems 
Author: Timothy Gu 
Author: Michaël Zasso 
Author: Anna Henningsen 
Reviewed-By: James M Snell 
Reviewed-By: Jeremiah Senkpiel

test: run abort tests

2017-08-26T23:30:35+00:00

Currently, tests in test/abort do not run in CI.

This change configures the test runner to not write core files for abort
tests and to run them.

PR-URL: https://github.com/nodejs/node/pull/14013
Fixes: https://github.com/nodejs/node/issues/14012
Reviewed-By: Anna Henningsen 
Reviewed-By: Refael Ackermann 
Reviewed-By: Ben Noordhuis 
Reviewed-By: Michael Dawson 
Reviewed-By: Colin Ihrig 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Yuta Hiroto 
Reviewed-By: Sakthipriyan Vairamani

delta/node-new.git/test/testpy/__init__.py, branch Remove-Python-3-tests