GeezerVison is showing github.com — test: add trailing commas in addons test (#45548) · nodejs/node@d5b0844
File treeExpand file treeCollapse file tree Open diff view settings openssl-client-cert-engine stringbytes-external-exceed-max
Expand file treeCollapse file tree Open diff view settings Collapse file Copy file name to clipboardExpand all lines: test/.eslintrc.yaml -1Lines changed: 0 additions & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -72,7 +72,6 @@ globals: | 72 | 72 |
| 73 | 73 | overrides: | 74 | 74 | - files: | 75 | | - - addons/*/*.js | 76 | 75 | - async-hooks/*.js | 77 | 76 | - es-module/*.js | 78 | 77 | - es-module/*.mjs | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/async-hooks-id/test.js +4-4Lines changed: 4 additions & 4 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -7,20 +7,20 @@ const async_hooks = require('async_hooks'); | 7 | 7 |
| 8 | 8 | assert.strictEqual( | 9 | 9 | binding.getExecutionAsyncId(), | 10 | | - async_hooks.executionAsyncId() | | 10 | + async_hooks.executionAsyncId(), | 11 | 11 | ); | 12 | 12 | assert.strictEqual( | 13 | 13 | binding.getTriggerAsyncId(), | 14 | | - async_hooks.triggerAsyncId() | | 14 | + async_hooks.triggerAsyncId(), | 15 | 15 | ); | 16 | 16 |
| 17 | 17 | process.nextTick(common.mustCall(() => { | 18 | 18 | assert.strictEqual( | 19 | 19 | binding.getExecutionAsyncId(), | 20 | | - async_hooks.executionAsyncId() | | 20 | + async_hooks.executionAsyncId(), | 21 | 21 | ); | 22 | 22 | assert.strictEqual( | 23 | 23 | binding.getTriggerAsyncId(), | 24 | | - async_hooks.triggerAsyncId() | | 24 | + async_hooks.triggerAsyncId(), | 25 | 25 | ); | 26 | 26 | })); |
Collapse file Copy file name to clipboardExpand all lines: test/addons/async-resource/test.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -38,7 +38,7 @@ async_hooks.createHook({ | 38 | 38 |
| 39 | 39 | destroy(id) { | 40 | 40 | if (bindingUids.includes(id)) destroy++; | 41 | | - } | | 41 | + }, | 42 | 42 | }).enable(); | 43 | 43 |
| 44 | 44 | for (const call of [binding.callViaFunction, | @@ -53,7 +53,7 @@ for (const call of [binding.callViaFunction, | 53 | 53 | assert.strictEqual(async_hooks.executionAsyncId(), uid); | 54 | 54 | return 'baz'; | 55 | 55 | }, | 56 | | - kObjectTag | | 56 | + kObjectTag, | 57 | 57 | }; | 58 | 58 |
| 59 | 59 | if (passedTriggerId === undefined) | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/callback-scope/test-async-hooks.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -14,7 +14,7 @@ async_hooks.createHook({ | 14 | 14 | after: common.mustCall((id) => { | 15 | 15 | assert.strictEqual(id, 1000); | 16 | 16 | insideHook = false; | 17 | | - }) | | 17 | + }), | 18 | 18 | }).enable(); | 19 | 19 |
| 20 | 20 | runInCallbackScope({}, 1000, 1000, () => { | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/no-addons/test-worker.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -13,7 +13,7 @@ const assertError = (error) => { | 13 | 13 | assert.strictEqual(error.code, 'ERR_DLOPEN_DISABLED'); | 14 | 14 | assert.strictEqual( | 15 | 15 | error.message, | 16 | | - 'Cannot load native addon because loading addons is disabled.' | | 16 | + 'Cannot load native addon because loading addons is disabled.', | 17 | 17 | ); | 18 | 18 | }; | 19 | 19 |
| @@ -32,7 +32,7 @@ const assertError = (error) => { | 32 | 32 | `process.dlopen({ exports: {} }, ${JSON.stringify(binding)});`, | 33 | 33 | { | 34 | 34 | eval: true, | 35 | | - } | | 35 | + }, | 36 | 36 | ); | 37 | 37 |
| 38 | 38 | worker.on('error', common.mustCall(assertError)); | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/no-addons/test.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -12,7 +12,7 @@ const assertError = (error) => { | 12 | 12 | assert.strictEqual(error.code, 'ERR_DLOPEN_DISABLED'); | 13 | 13 | assert.strictEqual( | 14 | 14 | error.message, | 15 | | - 'Cannot load native addon because loading addons is disabled.' | | 15 | + 'Cannot load native addon because loading addons is disabled.', | 16 | 16 | ); | 17 | 17 | }; | 18 | 18 |
| |
Collapse file Copy file name to clipboardExpand all lines: test/addons/non-node-context/test-make-buffer.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
| 3 | 3 | const common = require('../../common'); | 4 | 4 | const assert = require('assert'); | 5 | 5 | const { | 6 | | - makeBufferInNewContext | | 6 | + makeBufferInNewContext, | 7 | 7 | } = require(`./build/${common.buildType}/binding`); | 8 | 8 |
| 9 | 9 | // Because the `Buffer` function and its protoype property only (currently) | @@ -19,5 +19,5 @@ assert.throws( | 19 | 19 | assert.strictEqual(exception.message, | 20 | 20 | 'Buffer is not available for the current Context'); | 21 | 21 | return true; | 22 | | - } | | 22 | + }, | 23 | 23 | ); |
Collapse file Copy file name to clipboardExpand all lines: test/addons/openssl-client-cert-engine/test.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -26,7 +26,7 @@ const serverOptions = { | 26 | 26 | cert: agentCert, | 27 | 27 | ca: agentCa, | 28 | 28 | requestCert: true, | 29 | | - rejectUnauthorized: true | | 29 | + rejectUnauthorized: true, | 30 | 30 | }; | 31 | 31 |
| 32 | 32 | const server = https.createServer(serverOptions, common.mustCall((req, res) => { | @@ -40,7 +40,7 @@ const server = https.createServer(serverOptions, common.mustCall((req, res) => { | 40 | 40 | path: '/test', | 41 | 41 | clientCertEngine: engine, // `engine` will provide key+cert | 42 | 42 | rejectUnauthorized: false, // Prevent failing on self-signed certificates | 43 | | - headers: {} | | 43 | + headers: {}, | 44 | 44 | }; | 45 | 45 |
| 46 | 46 | const req = https.request(clientOptions, common.mustCall((response) => { | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/openssl-key-engine/test.js +2-2Lines changed: 2 additions & 2 deletions | Original file line number | Diff line number | Diff line change |
|---|
@@ -26,7 +26,7 @@ const serverOptions = { | 26 | 26 | cert: agentCert, | 27 | 27 | ca: agentCa, | 28 | 28 | requestCert: true, | 29 | | - rejectUnauthorized: true | | 29 | + rejectUnauthorized: true, | 30 | 30 | }; | 31 | 31 |
| 32 | 32 | const server = https.createServer(serverOptions, common.mustCall((req, res) => { | @@ -42,7 +42,7 @@ const server = https.createServer(serverOptions, common.mustCall((req, res) => { | 42 | 42 | privateKeyIdentifier: 'dummykey', | 43 | 43 | cert: agentCert, | 44 | 44 | rejectUnauthorized: false, // Prevent failing on self-signed certificates | 45 | | - headers: {} | | 45 | + headers: {}, | 46 | 46 | }; | 47 | 47 |
| 48 | 48 | const req = https.request(clientOptions, common.mustCall((response) => { | |
Collapse file Copy file name to clipboardExpand all lines: test/addons/repl-domain-abort/test.js +1-1Lines changed: 1 addition & 1 deletion | Original file line number | Diff line number | Diff line change |
|---|
@@ -63,7 +63,7 @@ const options = { | 63 | 63 | input: dInput, | 64 | 64 | output: dOutput, | 65 | 65 | terminal: false, | 66 | | - ignoreUndefined: true | | 66 | + ignoreUndefined: true, | 67 | 67 | }; | 68 | 68 |
| 69 | 69 | // Run commands from fake REPL. | |
|