mirror of
https://github.com/nodejs/node.git
synced 2025-04-28 21:46:48 +00:00

This commit introduces store as the first argument in AsyncLocalStorage's run methods. The change is motivated by the following expectation: most users are going to use a custom object as the store and an extra Map created by the previous implementation is an overhead for their use case. Important note. This is a backwards incompatible change. It was discussed and agreed an incompatible change is ok since the API is still experimental and the modified methods were only added within the last week so usage will be minimal to none. PR-URL: https://github.com/nodejs/node/pull/31930 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Vladimir de Turckheim <vlad2t@hotmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
28 lines
766 B
JavaScript
28 lines
766 B
JavaScript
'use strict';
|
|
require('../common');
|
|
const assert = require('assert');
|
|
const { AsyncLocalStorage } = require('async_hooks');
|
|
|
|
async function foo() {}
|
|
|
|
const asyncLocalStorage = new AsyncLocalStorage();
|
|
|
|
async function testOut() {
|
|
await foo();
|
|
assert.strictEqual(asyncLocalStorage.getStore(), undefined);
|
|
}
|
|
|
|
async function testAwait() {
|
|
await foo();
|
|
assert.notStrictEqual(asyncLocalStorage.getStore(), undefined);
|
|
assert.strictEqual(asyncLocalStorage.getStore().get('key'), 'value');
|
|
await asyncLocalStorage.exitSyncAndReturn(testOut);
|
|
}
|
|
|
|
asyncLocalStorage.run(new Map(), () => {
|
|
const store = asyncLocalStorage.getStore();
|
|
store.set('key', 'value');
|
|
testAwait(); // should not reject
|
|
});
|
|
assert.strictEqual(asyncLocalStorage.getStore(), undefined);
|