Use correct require instance and enable tests

rollup · Jun 20, 2022 · 4e6e721 · 4e6e721
1 parent 44fbb21
commit 4e6e721
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 5 deletions.
diff --git a/cli/run/commandPlugins.ts b/cli/run/commandPlugins.ts
@@ -73,7 +73,9 @@ async function loadAndRegisterPlugin(
 			try {
 				if (pluginText[0] == '.') pluginText = resolve(pluginText);
 				// Windows absolute paths must be specified as file:// protocol URL
-				else if (pluginText.match(/^[A-Za-z]:\\/)) pluginText = "file://" + require$$0.resolve(pluginText);
+				else if (pluginText.match(/^[A-Za-z]:\\/)) {
+					pluginText = 'file://' + require.resolve(pluginText);
+				}
 				plugin = await requireOrImport(pluginText);
 			} catch (err: any) {
 				throw new Error(`Cannot load plugin "${pluginText}": ${err.message}.`);

diff --git a/test/cli/samples/plugin/absolute-esm/_config.js b/test/cli/samples/plugin/absolute-esm/_config.js
@@ -1,6 +1,5 @@
 module.exports = {
 	description: 'ESM CLI --plugin /absolute/path',
 	minNodeVersion: 12,
-	skipIfWindows: true,
-	command: `echo 'console.log(1 ? 2 : 3);' | rollup -p "\`pwd\`/my-esm-plugin.mjs={comment: 'Absolute ESM'}"`
+	command: `rollup main.js -p "\`pwd\`/my-esm-plugin.mjs={comment: 'Absolute ESM'}"`
 };
diff --git a/test/cli/samples/plugin/absolute-esm/main.js b/test/cli/samples/plugin/absolute-esm/main.js
@@ -0,0 +1 @@
+console.log(1 ? 2 : 3);
diff --git a/test/cli/samples/plugin/absolute/_config.js b/test/cli/samples/plugin/absolute/_config.js
@@ -1,5 +1,4 @@
 module.exports = {
 	description: 'CLI --plugin /absolute/path',
-	skipIfWindows: true,
-	command: `echo 'console.log(VALUE);' | rollup -p "\`pwd\`/my-plugin={VALUE: 'absolute', ZZZ: 1}"`
+	command: `rollup main.js -p "\`pwd\`/my-plugin={VALUE: 'absolute', ZZZ: 1}"`
 };
diff --git a/test/cli/samples/plugin/absolute/main.js b/test/cli/samples/plugin/absolute/main.js
@@ -0,0 +1 @@
+console.log(VALUE);