delta/node-new.git/test/parallel/test-crypto-fips.js, branch Remove-Python-3-tests

test: use fipsMode in test-crypto-fips

2019-01-30T11:37:38+00:00

This commit updates test/parallel/test-crypto-fips.js to use fipsMode
instead of process.config.variables.openssl_fips.

The motivation for this is that since the addition of the
--openssl-is-fips configuration flag, it is possible to dynamically
link with a FIPS compliant OpenSSL library. Using fipsMode allows for
determining if there is FIPS support when statically or dynamically
linking against a FIPS compatible OpenSSL library.

PR-URL: https://github.com/nodejs/node/pull/25563
Reviewed-By: Michael Dawson 
Reviewed-By: Richard Lau 
Reviewed-By: Colin Ihrig 
Reviewed-By: James M Snell 
Reviewed-By: Luigi Pinca

test: fix arguments order in assert.strictEqual

2018-11-25T03:50:42+00:00

PR-URL: https://github.com/nodejs/node/pull/24591
Reviewed-By: Ron Korving 
Reviewed-By: Colin Ihrig 
Reviewed-By: Gireesh Punathil 
Reviewed-By: Masashi Hirano

tools: add 'spaced-comment' into eslint rules

2018-04-01T14:33:13+00:00

PR-URL: https://github.com/nodejs/node/pull/19596
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Trivikram Kamat 
Reviewed-By: Sakthipriyan Vairamani

crypto: docs-only deprecate crypto.fips, replace

2018-02-02T15:50:31+00:00

Docs-only deprecate the getter/setter crypto.fips and replace
with crypto.setFips() and crypto.getFips()

This is specifically in preparation for ESM module support

PR-URL: https://github.com/nodejs/node/pull/18335
Refs: https://github.com/nodejs/node/pull/18131
Reviewed-By: Anna Henningsen 
Reviewed-By: Colin Ihrig 
Reviewed-By: Matteo Collina 
Reviewed-By: Guy Bedford 
Reviewed-By: Jon Moss 
Reviewed-By: Michael Dawson 
Reviewed-By: Ruben Bridgewater

test: fix spelling in test case comments

2018-01-11T15:49:38+00:00

PR-URL: https://github.com/nodejs/node/pull/18018
Reviewed-By: Jon Moss 
Reviewed-By: Anatoli Papirovski 
Reviewed-By: Weijia Wang 
Reviewed-By: Luigi Pinca 
Reviewed-By: James M Snell 
Reviewed-By: Colin Ihrig 
Reviewed-By: Yuta Hiroto

crypto: migrate setFipsCrypto to internal/errors

2017-10-27T15:13:01+00:00

With the exception of ThrowCryptoError, use internal/errors
to report fips unavailable or forced

PR-URL: https://github.com/nodejs/node/pull/16428
Reviewed-By: Luigi Pinca 
Reviewed-By: Michaël Zasso 
Reviewed-By: Joyee Cheung 
Reviewed-By: Matteo Collina

crypto: only try to set FIPS mode if different

2017-09-26T22:49:31+00:00

Turning FIPS mode on (or off) when it's already on (or off) should be a
no-op, not an error.

PR-URL: https://github.com/nodejs/node/pull/12210
Fixes: https://github.com/nodejs/node/issues/11849
Reviewed-By: Richard Lau 
Reviewed-By: Michael Dawson 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: James M Snell

test: remove envPlus, use Object.assign everywhere

2017-09-03T20:28:50+00:00

PR-URL: https://github.com/nodejs/node/pull/14845
Fixes: https://github.com/nodejs/node/issues/14823
Refs: https://github.com/nodejs/node/pull/14822
Reviewed-By: Refael Ackermann 
Reviewed-By: James M Snell 
Reviewed-By: Yuta Hiroto 
Reviewed-By: Ruben Bridgewater 
Reviewed-By: Tobias Nießen 
Reviewed-By: Rich Trott

test: begin normalizing fixtures use

2017-08-08T01:00:57+00:00

Adds a new `../common/fixtures' module to begin normalizing
`test/fixtures` use. Our test code is a bit inconsistent with
regards to use of the fixtures directory. Some code uses
`path.join()`, some code uses string concats, some other
code uses template strings, etc. In mnay cases, significant
duplication of code is seen when accessing fixture files, etc.

This updates many (but by no means all) of the tests in the
test suite to use the new consistent API. There are still
many more to update, which would make an excelent Code-n-Learn
exercise.

PR-URL: https://github.com/nodejs/node/pull/14332
Reviewed-By: Anna Henningsen 
Reviewed-By: Gibson Fahnestock 
Reviewed-By: Colin Ihrig 
Reviewed-By: Tobias Nießen 
Reviewed-By: Michaël Zasso

test: adjust indentation for stricter linting

2017-07-27T16:24:20+00:00

ESLint 4.x has stricter linting than previous versions. We are currently
using the legacy indentation rules in the test directory. This commit
changes the indentation of files to comply with the stricter 4.x linting
and enable stricter linting in the test directory.

PR-URL: https://github.com/nodejs/node/pull/14431
Reviewed-By: Refael Ackermann 
Reviewed-By: Vse Mozhet Byt 
Reviewed-By: Trevor Norris