GeezerVison is showing github.com — test,crypto: update WebCryptoAPI WPT · nodejs/node@bb4aa7a
File treeExpand file treeCollapse file tree Open diff view settings Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/README.md +1-1Lines changed: 1 addition & 1 deletion - Display the source diff
- Display the rich diff
| Original file line number | Diff line number | Diff line change |
|---|
@@ -32,7 +32,7 @@ Last update: | 32 | 32 | - user-timing: https://github.com/web-platform-tests/wpt/tree/5ae85bf826/user-timing | 33 | 33 | - wasm/jsapi: https://github.com/web-platform-tests/wpt/tree/cde25e7e3c/wasm/jsapi | 34 | 34 | - wasm/webapi: https://github.com/web-platform-tests/wpt/tree/fd1b23eeaa/wasm/webapi | 35 | | -- WebCryptoAPI: https://github.com/web-platform-tests/wpt/tree/b81831169b/WebCryptoAPI | | 35 | +- WebCryptoAPI: https://github.com/web-platform-tests/wpt/tree/ecf39b605f/WebCryptoAPI | 36 | 36 | - webidl/ecmascript-binding/es-exceptions: https://github.com/web-platform-tests/wpt/tree/a370aad338/webidl/ecmascript-binding/es-exceptions | 37 | 37 | - webmessaging/broadcastchannel: https://github.com/web-platform-tests/wpt/tree/6495c91853/webmessaging/broadcastchannel | 38 | 38 | - webstorage: https://github.com/web-platform-tests/wpt/tree/9dafa89214/webstorage | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/WebCryptoAPI/digest/digest.https.any.js +14Lines changed: 14 additions & 0 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 118 | 118 | }); | 119 | 119 | }); | 120 | 120 |
| | 121 | + // Call digest() with empty algorithm object | | 122 | + Object.keys(sourceData).forEach(function(size) { | | 123 | + promise_test(function(test) { | | 124 | + var promise = subtle.digest({}, sourceData[size]) | | 125 | + .then(function(result) { | | 126 | + assert_unreached("digest() with missing algorithm name should have thrown a TypeError"); | | 127 | + }, function(err) { | | 128 | + assert_equals(err.name, "TypeError", "Missing algorithm name should cause TypeError") | | 129 | + }); | | 130 | + | | 131 | + return promise; | | 132 | + }, "empty algorithm object with " + size); | | 133 | + }); | | 134 | + | 121 | 135 |
| 122 | 136 | done(); | 123 | 137 |
| |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/WebCryptoAPI/generateKey/failures.js +8Lines changed: 8 additions & 0 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -166,6 +166,14 @@ function run_test(algorithmNames) { | 166 | 166 | }); | 167 | 167 | }); | 168 | 168 |
| | 169 | + // Empty algorithm should fail with TypeError | | 170 | + allValidUsages(["decrypt", "sign", "deriveBits"], true, []) // Small search space, shouldn't matter because should fail before used | | 171 | + .forEach(function(usages) { | | 172 | + [false, true, "RED", 7].forEach(function(extractable){ | | 173 | + testError({}, extractable, usages, "TypeError", "Empty algorithm"); | | 174 | + }); | | 175 | + }); | | 176 | + | 169 | 177 |
| 170 | 178 | // Algorithms normalize okay, but usages bad (though not empty). | 171 | 179 | // It shouldn't matter what other extractable is. Should fail | |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/WebCryptoAPI/import_export/importKey_failures.js +15Lines changed: 15 additions & 0 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -192,4 +192,19 @@ function run_test(algorithmNames) { | 192 | 192 | }); | 193 | 193 | }); | 194 | 194 | }); | | 195 | + | | 196 | + // Missing mandatory "name" field on algorithm | | 197 | + testVectors.forEach(function(vector) { | | 198 | + var name = vector.name; | | 199 | + // We just need *some* valid keydata, so pick the first available algorithm. | | 200 | + var algorithm = allAlgorithmSpecifiersFor(name)[0]; | | 201 | + getValidKeyData(algorithm).forEach(function(test) { | | 202 | + validUsages(vector, test.format, test.data).forEach(function(usages) { | | 203 | + [true, false].forEach(function(extractable) { | | 204 | + testError(test.format, {}, test.data, name, usages, extractable, "TypeError", "Missing algorithm name"); | | 205 | + }); | | 206 | + }); | | 207 | + }); | | 208 | + }); | | 209 | + | 195 | 210 | } |
Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/WebCryptoAPI/wrapKey_unwrapKey/wrapKey_unwrapKey.https.any.js +267-266Lines changed: 267 additions & 266 deletions Large diffs are not rendered by default. Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/WebCryptoAPI/wrapKey_unwrapKey/wrapKey_unwrapKey_vectors.js +114Lines changed: 114 additions & 0 deletions Collapse file Copy file name to clipboardExpand all lines: test/fixtures/wpt/versions.json +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
| 88 | 88 | "path": "wasm/webapi" | 89 | 89 | }, | 90 | 90 | "WebCryptoAPI": { | 91 | | - "commit": "b81831169b8527a6c569a4ad92cf8a1baf4a7118", | | 91 | + "commit": "ecf39b605f4d2b5d7ef4fe344063c2856c9f105c", | 92 | 92 | "path": "WebCryptoAPI" | 93 | 93 | }, | 94 | 94 | "webidl/ecmascript-binding/es-exceptions": { | |
|