tests: fix

webpack · Nov 10, 2020 · d1f9bbd · d1f9bbd
1 parent 19a4c8f
commit d1f9bbd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/test/build-warnings/warnings.test.js b/test/build-warnings/warnings.test.js
@@ -21,7 +21,7 @@ describe('warnings', () => {
         const json = JSON.parse(stdout);
 
         expect(json['hash']).toBeDefined();
-        expect(json['warnings']).toHaveLength(1);
+        expect(json['warnings']).toHaveLength(2);
         // `message` for `webpack@5`
         expect(json['warnings'][0].message ? json['warnings'][0].message : json['warnings'][0]).toMatch(/Can't resolve/);
     });
@@ -43,7 +43,7 @@ describe('warnings', () => {
                 const json = JSON.parse(data);
 
                 expect(json['hash']).toBeDefined();
-                expect(json['warnings']).toHaveLength(1);
+                expect(json['warnings']).toHaveLength(2);
                 // `message` for `webpack@5`
                 expect(json['warnings'][0].message ? json['warnings'][0].message : json['warnings'][0]).toMatch(/Can't resolve/);
 

diff --git a/test/defaults/output-defaults.test.js b/test/defaults/output-defaults.test.js
@@ -9,8 +9,8 @@ describe('output flag defaults', () => {
 
         expect(stderr).toBeFalsy();
         expect(exitCode).toBe(0);
-        // Should not print warning about config fallback, as we have production as default
-        expect(stdout).not.toContain('option has not been set, webpack will fallback to');
+        // Should print warning about config fallback
+        expect(stdout).toContain('option has not been set, webpack will fallback to');
         stat(resolve(__dirname, './binary/main.js'), (err, stats) => {
             expect(err).toBe(null);
             expect(stats.isFile()).toBe(true);

diff --git a/test/merge/config/merge-config.test.js b/test/merge/config/merge-config.test.js
@@ -8,7 +8,7 @@ const { run } = require('../../utils/test-utils');
 describe('merge flag configuration', () => {
     it('merges two configurations together', () => {
         const { stdout, stderr, exitCode } = run(__dirname, ['--config', './1.js', '-c', './2.js', '--merge'], false);
-        expect(stdout).not.toContain('option has not been set, webpack will fallback to');
+        expect(stdout).toContain('option has not been set, webpack will fallback to');
         expect(existsSync(resolve(__dirname, './dist/merged.js'))).toBeTruthy();
         expect(stderr).toBeFalsy();
         expect(exitCode).toBe(0);