fix: make methods reenrant

arlac77 · May 17, 2022 · 3d08a97 · 3d08a97
1 parent d108d3c
commit 3d08a97
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/one-time-execution-method.mjs b/src/one-time-execution-method.mjs
@@ -29,6 +29,9 @@ export function replaceWithOneTimeExecutionMethod(object, name) {
   Object.defineProperty(object, name, {
     value: function (...args) {
       if (this[transitionState] === undefined) {
+        // make method reentrant
+        this[transitionState] = Promise.resolve();
+
         this[transitionState] = func.apply(this, ...args);
       }
 

diff --git a/tests/one-time-execution-method-test.mjs b/tests/one-time-execution-method-test.mjs
@@ -44,7 +44,7 @@ test("replaceWithOneTimeExecutionMethod parallel", async t => {
 
 replaceWithOneTimeExecutionMethod(MyClass.prototype, "reentrantInitialize");
 
-test.skip("replaceWithOneTimeExecutionMethod reentrant", async t => {
+test("replaceWithOneTimeExecutionMethod reentrant", async t => {
   const object = new MyClass();
 
   t.is(object.executions, 0);