chore: send warning regarding invalid template to stdout

webpack · May 3, 2021 · 601ee07 · 601ee07
1 parent 60abcd4
commit 601ee07
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/packages/generators/src/addon-generator.ts b/packages/generators/src/addon-generator.ts
@@ -1,3 +1,4 @@
+import { yellow } from 'colorette';
 import fs from 'fs';
 import path from 'path';
 import Generator from 'yeoman-generator';
@@ -57,7 +58,7 @@ const addonGenerator = (
 
         public async prompting(): Promise<void> {
             if (!this.supportedTemplates.includes(this.template)) {
-                this.utils.logger.warn(`⚠ ${this.template} is not a valid template, please select one from below`);
+                this.utils.logger.log(`${yellow(`⚠ ${this.template} is not a valid template, please select one from below`)}`);
 
                 const { selectedTemplate } = await List(
                     this,

diff --git a/test/loader/loader.test.js b/test/loader/loader.test.js
@@ -147,8 +147,8 @@ describe('loader command', () => {
 
     it('should prompt on supplying an invalid template', async () => {
         const assetsPath = await uniqueDirectoryForTest();
-        const { stderr } = await runPromptWithAnswers(assetsPath, ['loader', '--template=unknown']);
+        const { stdout } = await runPromptWithAnswers(assetsPath, ['loader', '--template=unknown']);
 
-        expect(stderr).toContain('unknown is not a valid template');
+        expect(stdout).toContain('unknown is not a valid template');
     });
 });
diff --git a/test/plugin/plugin.test.js b/test/plugin/plugin.test.js
@@ -136,8 +136,8 @@ describe('plugin command', () => {
 
     it('should prompt on supplying an invalid template', async () => {
         const assetsPath = await uniqueDirectoryForTest();
-        const { stderr } = await runPromptWithAnswers(assetsPath, ['plugin', '--template=unknown']);
+        const { stdout } = await runPromptWithAnswers(assetsPath, ['plugin', '--template=unknown']);
 
-        expect(stderr).toContain('unknown is not a valid template');
+        expect(stdout).toContain('unknown is not a valid template');
     });
 });