tests(watch): use copyFile functionality

webpack · Jan 21, 2019 · ce7ab1c · ce7ab1c
1 parent 5078914
commit ce7ab1c
Show file tree

Hide file tree

Showing 6 changed files with 19 additions and 43 deletions.
diff --git a/test/binCases/watch/info-verbosity-off/info-verbosity-off.test.js b/test/binCases/watch/info-verbosity-off/info-verbosity-off.test.js
@@ -5,18 +5,15 @@ jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features/es-syntax  */
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {

diff --git a/test/binCases/watch/info-verbosity-verbose/info-verbosity-verbose.test.js b/test/binCases/watch/info-verbosity-verbose/info-verbosity-verbose.test.js
@@ -3,21 +3,17 @@
 jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features  */
 /* eslint-disable node/no-unsupported-features/es-syntax  */
-
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {

diff --git a/test/binCases/watch/multi-config-watch-opt/multi-config-watch-opt.test.js b/test/binCases/watch/multi-config-watch-opt/multi-config-watch-opt.test.js
@@ -3,21 +3,17 @@
 jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features  */
 /* eslint-disable node/no-unsupported-features/es-syntax  */
-
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {

diff --git a/test/binCases/watch/multi-config/multi-config.test.js b/test/binCases/watch/multi-config/multi-config.test.js
@@ -3,21 +3,17 @@
 jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features  */
 /* eslint-disable node/no-unsupported-features/es-syntax  */
-
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {

diff --git a/test/binCases/watch/single-config-watch-opt/single-config-watch-opt.test.js b/test/binCases/watch/single-config-watch-opt/single-config-watch-opt.test.js
@@ -3,21 +3,17 @@
 jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features  */
 /* eslint-disable node/no-unsupported-features/es-syntax  */
-
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {

diff --git a/test/binCases/watch/single-config/single-config.test.js b/test/binCases/watch/single-config/single-config.test.js
@@ -1,24 +1,19 @@
 "use strict";
 
+jest.setTimeout(10E6);
 /* eslint-disable node/no-unsupported-features  */
 /* eslint-disable node/no-unsupported-features/es-syntax  */
-
-jest.setTimeout(10E6);
-
 const fs = require("fs");
 const path = require("path");
-const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc } = require("../../../testUtils");
+const { extractSummary, extractHash, appendDataIfFileExists, runAndGetWatchProc, copyFile } = require("../../../testUtils");
 
 const fileToChange = "index.js";
-const copyFile = "index_copy.js";
 const fileToChangePath = path.resolve(__dirname, fileToChange);
-const copyFilePath = path.resolve(__dirname, copyFile);
+var copyFilePath;
 
 // create copy of "index.js" => "index_copy.js"
 beforeEach(() => {
-	// fs.copyFileSync was added in Added in: v8.5.0
-	// We should refactor the below code once our minimal supported version is v8.5.0
-	fs.createReadStream(fileToChangePath).pipe(fs.createWriteStream(copyFilePath));
+	copyFilePath = copyFile(__dirname, fileToChange);
 });
 
 afterEach(() => {