Skip to content

Commit aa86c78

Browse files
legendecastargos
authored andcommitted
test: fix vm assertion actual and expected order
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>
1 parent 893024c commit aa86c78

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/parallel/test-vm-module-dynamic-import.js

+3-3
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ async function test() {
4040
});
4141

4242
const result = s.runInThisContext();
43-
assert.strictEqual(foo.namespace, await result);
43+
assert.strictEqual(await result, foo.namespace);
4444
}
4545

4646
{
@@ -53,7 +53,7 @@ async function test() {
5353
});
5454
await m.link(common.mustNotCall());
5555
await m.evaluate();
56-
assert.strictEqual(foo.namespace, await globalThis.fooResult);
56+
assert.strictEqual(await globalThis.fooResult, foo.namespace);
5757
delete globalThis.fooResult;
5858
}
5959

@@ -68,7 +68,7 @@ async function test() {
6868
});
6969

7070
const result = s.runInThisContext();
71-
assert.strictEqual(foo.namespace, await result);
71+
assert.strictEqual(await result, foo.namespace);
7272
}
7373
}
7474

0 commit comments

Comments
 (0)