refactor: emoji deprecate

cap-Bernardito · web-flow · commit 63b41beed479 · 2020-09-24T23:59:18.000+03:00
BREAKING CHANGE: `emoji` was deprecated
diff --git a/src/utils.js b/src/utils.js
@@ -210,6 +210,12 @@ function getModulesOptions(rawOptions, loaderContext) {
     }
   }
 
+  if (/\[emoji(?::(\d+))?\]/i.test(modulesOptions.localIdentName)) {
+    loaderContext.emitWarning(
+      'Emoji is deprecated and will be removed in next major release.'
+    );
+  }
+
   return modulesOptions;
 }
 
diff --git a/test/__snapshots__/modules-option.test.js.snap b/test/__snapshots__/modules-option.test.js.snap
@@ -699,6 +699,15 @@ Array [
 
 exports[`"modules" option should dedupe same modules in one module (issue #1037): warnings 1`] = `Array []`;
 
+exports[`"modules" option should emit warning when localIdentName is emoji: errors 1`] = `Array []`;
+
+exports[`"modules" option should emit warning when localIdentName is emoji: warnings 1`] = `
+Array [
+  "ModuleWarning: Module Warning (from \`replaced original path\`):
+(Emitted value instead of an instance of Error) Emoji is deprecated and will be removed in next major release.",
+]
+`;
+
 exports[`"modules" option should keep order: errors 1`] = `Array []`;
 
 exports[`"modules" option should keep order: module 1`] = `
diff --git a/test/modules-option.test.js b/test/modules-option.test.js
@@ -1389,4 +1389,16 @@ describe('"modules" option', () => {
     expect(getWarnings(stats)).toMatchSnapshot('warnings');
     expect(getErrors(stats)).toMatchSnapshot('errors');
   });
+
+  it('should emit warning when localIdentName is emoji', async () => {
+    const compiler = getCompiler('./modules/pure/pure.js', {
+      modules: {
+        localIdentName: '[emoji:0]',
+      },
+    });
+    const stats = await compile(compiler);
+
+    expect(getWarnings(stats)).toMatchSnapshot('warnings');
+    expect(getErrors(stats)).toMatchSnapshot('errors');
+  });
 });

Original file line number	Diff line number	Diff line change
`@@ -210,6 +210,12 @@ function getModulesOptions(rawOptions, loaderContext) {`
`210`	`210`	`}`
`211`	`211`	`}`
`212`	`212`
	`213`	`+ if (/\[emoji(?::(\d+))?\]/i.test(modulesOptions.localIdentName)) {`
	`214`	`+ loaderContext.emitWarning(`
	`215`	`+ 'Emoji is deprecated and will be removed in next major release.'`
	`216`	`+ );`
	`217`	`+ }`
	`218`	`+`
`213`	`219`	`return modulesOptions;`
`214`	`220`	`}`
`215`	`221`