Skip to content

Commit 2e91b63

Browse files
committed
Modifying the test code to make it less misleading
1 parent 2c36e1f commit 2e91b63

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

packages/npm-packages/ruby-wasm-wasi/test-e2e/integrations/js-require-remote.spec.ts

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ if (!process.env.RUBY_NPM_PACKAGE_ROOT) {
116116
expect(await resolve()).toBe("Hello from RecursiveRequire::B");
117117
});
118118

119-
test("JS::RequireRemote#load loads a file with a relative path from base_url option", async ({
119+
test("JS::RequireRemote#load loads the file with a path relative to the base_url specified by the base_url property.", async ({
120120
page
121121
}) => {
122122
const resolve = await resolveBinding(page, "checkResolved");
@@ -127,12 +127,13 @@ if (!process.env.RUBY_NPM_PACKAGE_ROOT) {
127127
<script src="browser.script.iife.js"></script>
128128
<script type="text/ruby" data-eval="async">
129129
require 'js/require_remote'
130-
JS::RequireRemote.instance.base_url = 'fixtures'
131-
JS.global.checkResolved JS::RequireRemote.instance.load 'error_on_load_twice'
130+
JS::RequireRemote.instance.base_url = 'fixtures/recursive_require'
131+
JS::RequireRemote.instance.load 'b'
132+
JS.global.checkResolved RecursiveRequire::B.new.message
132133
</script>
133134
`);
134135

135-
expect(await resolve()).toBe(true);
136+
expect(await resolve()).toBe("Hello from RecursiveRequire::B");
136137
});
137138
});
138139
}

0 commit comments

Comments
 (0)