Skip to content

Commit

Permalink
test: fix vm assertion actual and expected order
Browse files Browse the repository at this point in the history
PR-URL: #50371
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Vinícius Lourenço Claro Cardoso <contact@viniciusl.com.br>
  • Loading branch information
legendecas authored and UlisesGascon committed Dec 11, 2023
1 parent 59f7316 commit 099f5cf
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions test/parallel/test-vm-module-dynamic-import.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ async function test() {
});

const result = s.runInThisContext();
assert.strictEqual(foo.namespace, await result);
assert.strictEqual(await result, foo.namespace);
}

{
Expand All @@ -53,7 +53,7 @@ async function test() {
});
await m.link(common.mustNotCall());
await m.evaluate();
assert.strictEqual(foo.namespace, await globalThis.fooResult);
assert.strictEqual(await globalThis.fooResult, foo.namespace);
delete globalThis.fooResult;
}

Expand All @@ -68,7 +68,7 @@ async function test() {
});

const result = s.runInThisContext();
assert.strictEqual(foo.namespace, await result);
assert.strictEqual(await result, foo.namespace);
}
}

Expand Down

0 comments on commit 099f5cf

Please sign in to comment.