Throw an error on invalid passthrough copy modes #3573

Author: zachleat

src/UserConfig.js

@@ -792,7 +792,12 @@ class UserConfig {
 	 * @returns {any} a reference to the `EleventyConfig` object.
 	 */
 	addPassthroughCopy(fileOrDir, copyOptions = {}) {
-		if (copyOptions.mode === "html-relative") {
+		if (copyOptions.mode) {
+			if (copyOptions.mode !== "html-relative") {
+				throw new Error(
+					"Invalid `mode` option for `addPassthroughCopy`. Received: '" + copyOptions.mode + "'",
+				);
+			}
 			if (isPlainObject(fileOrDir)) {
 				throw new Error(
 					"mode: 'html-relative' does not yet support passthrough copy objects (input -> output mapping). Use a string glob or an Array of string globs.",

test/HtmlRelativeCopyTest.js

@@ -592,8 +592,30 @@ test("Input -> output remapping not yet supported (throws error)", async (t) =>
 	await t.throwsAsync(async () => {
 		await elev.write();
 	}, {
-		message: `mode: \'html-relative\' does not yet support passthrough copy objects (input -> output mapping). Use a string glob or an Array of string globs.`
+		message: `mode: 'html-relative' does not yet support passthrough copy objects (input -> output mapping). Use a string glob or an Array of string globs.`
 	});
 
 	t.is(fs.existsSync("test/stubs-autocopy/_site12/test/index.html"), false);
 });
+
+test("Invalid copy mode throws error", async (t) => {
+	let elev = new Eleventy("./test/stubs-autocopy/", "./test/stubs-autocopy/_site13", {
+		configPath: false,
+		config: function (eleventyConfig) {
+			// not yet supported
+			eleventyConfig.addPassthroughCopy({"**/*.png": "yo"}, {
+				mode: "throw-an-error"
+			});
+		},
+	});
+
+	elev.disableLogger();
+
+	await t.throwsAsync(async () => {
+		await elev.write();
+	}, {
+		message: `Invalid \`mode\` option for \`addPassthroughCopy\`. Received: 'throw-an-error'`
+	});
+
+	t.is(fs.existsSync("test/stubs-autocopy/_site13/test/index.html"), false);
+});