From fb09faed3badf149ec9c38bb69c4edb2679d7bbf Mon Sep 17 00:00:00 2001
From: Ben Holloway <github@bholloway.com>
Date: Wed, 9 Jun 2021 23:01:47 +1000
Subject: [PATCH] remove the engine option harder, ensure deprecation warning
 is tested

---
 packages/resolve-url-loader/index.js |  2 +-
 test/cases/misconfiguration.js       | 30 ++++++++++++++++++++++++++++
 2 files changed, 31 insertions(+), 1 deletion(-)

diff --git a/packages/resolve-url-loader/index.js b/packages/resolve-url-loader/index.js
index 0705f01..4738b9b 100644
--- a/packages/resolve-url-loader/index.js
+++ b/packages/resolve-url-loader/index.js
@@ -20,7 +20,7 @@ const logToTestHarness = require('./lib/log-to-test-harness');
 const DEPRECATED_OPTIONS = {
   engine: [
     'DEP_RESOLVE_URL_LOADER_OPTION_ENGINE',
-    'the "engine" option has been removed, "postcss" is the only available parser'
+    '"engine" option has been removed, postcss is the only parser used'
   ],
   keepQuery: [
     'DEP_RESOLVE_URL_LOADER_OPTION_KEEP_QUERY',
diff --git a/test/cases/misconfiguration.js b/test/cases/misconfiguration.js
index ceed3f4..c0821fa 100644
--- a/test/cases/misconfiguration.js
+++ b/test/cases/misconfiguration.js
@@ -53,6 +53,36 @@ module.exports = test(
     env({
       ENTRY: join('src', 'index.scss')
     }),
+    testWithOption({ engine: 'postcss' })(
+      testDefault(
+        buildDevNormal(
+          assertWebpackOk,
+          assertNoErrors,
+          assertDeprecationWarning('"engine" option has been removed'),
+          assertContentDev
+        ),
+        buildProdNormal(
+          assertWebpackOk,
+          assertNoErrors,
+          assertDeprecationWarning('"engine" option has been removed'),
+          assertContentProd
+        )
+      ),
+      testSilent(
+        buildDevNormal(
+          assertWebpackOk,
+          assertNoErrors,
+          assertSilence,
+          assertContentDev
+        ),
+        buildProdNormal(
+          assertWebpackOk,
+          assertNoErrors,
+          assertSilence,
+          assertContentProd
+        )
+      )
+    ),
     testWithOption({ attempts: 1 })(
       testDefault(
         buildDevNormal(