test: fix

Author: snitin315

packages/webpack-cli/src/webpack-cli.ts

@@ -13,10 +13,8 @@ import {
   WebpackCLICommandOptions,
   WebpackCLIMainOption,
   WebpackCLILogger,
-  WebpackV4LegacyStats,
   WebpackDevServerOptions,
   WebpackRunOptions,
-  WebpackV4Compiler,
   WebpackCompiler,
   WebpackConfiguration,
   Argv,
@@ -34,7 +32,6 @@ import {
   Instantiable,
   JsonExt,
   ModuleName,
-  MultipleCompilerStatsOptions,
   PackageInstallOptions,
   PackageManager,
   Path,
@@ -2092,6 +2089,23 @@ class WebpackCLI implements IWebpackCLI {
         item.stats = { preset: item.stats };
       }
 
+      let colors;
+
+      // From arguments
+      if (typeof this.isColorSupportChanged !== "undefined") {
+        colors = Boolean(this.isColorSupportChanged);
+      }
+      // From stats
+      else if (typeof (item.stats as StatsOptions).colors !== "undefined") {
+        colors = (item.stats as StatsOptions).colors;
+      }
+      // Default
+      else {
+        colors = Boolean(this.colors.isColorSupported);
+      }
+
+      item.stats.colors = colors;
+
       // Apply CLI plugin
       if (!item.plugins) {
         item.plugins = [];

test/build/core-flags/core-flags.test.js

@@ -239,15 +239,17 @@ describe("core flags", () => {
       expect(stdout).toContain(`devtool: 'source-map'`);
     });
 
-    it("should allow string value devtool option using alias", async () => {
+    // TODO: Enable alias with webpack 5
+    it.skip("should allow string value devtool option using alias", async () => {
       const { exitCode, stderr, stdout } = await run(__dirname, ["-d", "source-map"]);
 
       expect(exitCode).toBe(0);
       expect(stderr).toBeFalsy();
       expect(stdout).toContain(`devtool: 'source-map'`);
     });
 
-    it("should allow string value devtool option using alias #1", async () => {
+    // TODO: Enable alias with webpack 5
+    it.skip("should allow string value devtool option using alias #1", async () => {
       // cSpell:ignore dsource
       const { exitCode, stderr, stdout } = await run(__dirname, ["-dsource-map"]);
 

test/build/defaults/output-defaults.test.js

@@ -38,7 +38,7 @@ describe("output flag defaults", () => {
     ]);
 
     expect(exitCode).toBe(2);
-    expect(stderr).toContain("Error: Option '-o, --output-path <value>' argument missing");
+    expect(stderr).toContain("Error: Option '--output-path <value>' argument missing");
     expect(stdout).toBeFalsy();
   });
 });

test/build/devtool/object/source-map-object.test.js

@@ -34,7 +34,7 @@ describe("source-map object", () => {
   it("should override config with source-map", async () => {
     const { exitCode, stderr, stdout } = await run(
       __dirname,
-      ["-c", "./webpack.eval.config.js", "--devtool", "source-map", "-o", "./binary"],
+      ["-c", "./webpack.eval.config.js", "--devtool", "source-map", "--output-path", "./binary"],
       false,
     );
 
@@ -47,7 +47,7 @@ describe("source-map object", () => {
   it("should override config with devtool false", async () => {
     const { exitCode, stderr, stdout } = await run(
       __dirname,
-      ["-c", "./webpack.eval.config.js", "--no-devtool", "-o", "./binary"],
+      ["-c", "./webpack.eval.config.js", "--no-devtool", "--output-path", "./binary"],
       false,
     );
 

test/build/entry/flag-entry/entry-with-flag.test.js

@@ -6,12 +6,7 @@ const { resolve } = require("path");
 
 describe("entry flag", () => {
   it("should resolve the path to src/index.cjs", async () => {
-    const { exitCode, stderr, stdout } = await run(__dirname, [
-      "--entry",
-      "./src/index.cjs",
-      "-o",
-      "./dist/",
-    ]);
+    const { exitCode, stderr, stdout } = await run(__dirname, ["--entry", "./src/index.cjs"]);
 
     expect(exitCode).toBe(0);
     expect(stderr).toBeFalsy();

test/build/output/__snapshots__/output-named-bundles.test.js.snap.webpack5

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`output flag named bundles should output file in bin directory using default webpack config with warning for empty output value: stderr 1`] = `
-"[webpack-cli] Error: Option '-o, --output-path <value>' argument missing
+"[webpack-cli] Error: Option '--output-path <value>' argument missing
 [webpack-cli] Run 'webpack --help' to see available commands and options"
 `;
 

test/build/target/flag-test/target-flag.test.js

@@ -22,7 +22,8 @@ describe("--target flag", () => {
       expect(stdout).toContain(`target: [ '${val}' ]`);
     });
 
-    it(`should accept ${val} with -t alias`, async () => {
+    // TODO: Enable aliases with webpack 5
+    it.skip(`should accept ${val} with -t alias`, async () => {
       const { exitCode, stderr, stdout } = await run(__dirname, ["-t", `${val}`]);
 
       expect(exitCode).toBe(0);

test/build/target/flag-test/webpack.config.js

@@ -3,6 +3,5 @@ const WebpackCLITestPlugin = require("../../../utils/webpack-cli-test-plugin");
 module.exports = {
   entry: "./index.js",
   mode: "development",
-  target: "node",
   plugins: [new WebpackCLITestPlugin()],
 };