Commit f3c3b1d
test: add coverage for ERR_TLS_INVALID_PROTOCOL_VERSION
There is currently no test that confirms that an invalid TLS protocol
results in ERR_TLS_INVALID_PROTOCOL_VERSION. Add tests to check this for
the `minVersion` and `maxVersion` options in `createSecureContext()`.
Refs: https://codecov.io/gh/nodejs/node/src/c14c476614e3134867ddb997bdfe5a41ba668175/lib/_tls_common.js#L56
Refs: https://coverage.nodejs.org/coverage-c14c476614e31348/lib/_tls_common.js.html#L56
PR-URL: #30741
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Sam Roberts <vieuxtech@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 197b616 commit f3c3b1d
1 file changed
Lines changed: 12 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -114,3 +114,15 @@ common.expectsInternalAssertion( | |||
| 114 | 114 | } | |
| 115 | 115 | ); | |
| 116 | 116 | } | |
| 117 | + | ||
| 118 | + assert.throws(() => { tls.createSecureContext({ minVersion: 'fhqwhgads' }); }, | ||
| 119 | + { | ||
| 120 | + code: 'ERR_TLS_INVALID_PROTOCOL_VERSION', | ||
| 121 | + name: 'TypeError' | ||
| 122 | + }); | ||
| 123 | + | ||
| 124 | + assert.throws(() => { tls.createSecureContext({ maxVersion: 'fhqwhgads' }); }, | ||
| 125 | + { | ||
| 126 | + code: 'ERR_TLS_INVALID_PROTOCOL_VERSION', | ||
| 127 | + name: 'TypeError' | ||
| 128 | + }); | ||
0 commit comments