feat: ability to disable hard links on a CI server

develar · develar · commit 92af26228ea0 · 2016-11-30T10:57:43.000+01:00
diff --git a/src/util/fs.ts b/src/util/fs.ts
@@ -83,7 +83,7 @@ export async function walk(initialDirPath: string, filter?: Filter, consumer?: (
   return result
 }
 
-const _isUseHardLink = process.platform != "win32" && (isCi || process.env.USE_HARD_LINKS === "true")
+const _isUseHardLink = process.platform != "win32" && process.env.USE_HARD_LINKS !== "false" && (isCi || process.env.USE_HARD_LINKS === "true")
 
 /**
  * Hard links is used if supported and allowed.
diff --git a/test/jestSetup.js b/test/jestSetup.js
@@ -37,4 +37,6 @@ test.ifWinCi = isCi && isWindows ? test : skip
 
 delete process.env.CSC_NAME
 process.env.CSC_IDENTITY_AUTO_DISCOVERY = "false"
-process.env.USE_HARD_LINKS = "true"
+if (!process.env.USE_HARD_LINKS) {
+  process.env.USE_HARD_LINKS = "true"
+}
diff --git a/test/src/filesTest.ts b/test/src/filesTest.ts
@@ -1,5 +1,5 @@
 import { expectedWinContents } from "./helpers/expectedContents"
-import { outputFile, stat } from "fs-extra-p"
+import { outputFile, stat, readFile } from "fs-extra-p"
 import { assertPack, modifyPackageJson, getPossiblePlatforms, app } from "./helpers/packTester"
 import BluebirdPromise from "bluebird-lst-c"
 import * as path from "path"
@@ -170,6 +170,8 @@ test("extraResources - one-package", () => {
 
         allCan(path.join(resourcesDir, "bar", "hello.txt"), false),
       ])
+
+      expect(await readFile(path.join(resourcesDir, "bar", "hello.txt"), "utf-8")).toEqual("data")
     },
     expectedContents: platform === Platform.WINDOWS ? pathSorter(expectedWinContents.concat(
       winDirPrefix + "bar/hello.txt",
diff --git a/test/src/helpers/runTests.ts b/test/src/helpers/runTests.ts
@@ -98,7 +98,10 @@ async function runTests() {
     const circleNodeIndex = parseInt(process.env.CIRCLE_NODE_INDEX, 10)
     if (circleNodeIndex === 0 || circleNodeIndex === 2) {
       skipWin = true
-      args.push("linux.*", "BuildTest.js", "extraMetadataTest.js", "mainEntryTest.js", "globTest.js", "filesTest.js", "ignoreTest.js", "nsisUpdaterTest")
+      args.push("linux.*", "BuildTest.js", "extraMetadataTest.js", "mainEntryTest.js", "globTest.js", "filesTest.js", "ignoreTest.js")
+      if (circleNodeIndex === 0) {
+        args.push("nsisUpdaterTest")
+      }
     }
     else {
       args.push("windows.*", "mac.*")

Original file line number	Diff line number	Diff line change
`@@ -83,7 +83,7 @@ export async function walk(initialDirPath: string, filter?: Filter, consumer?: (`
`83`	`83`	`return result`
`84`	`84`	`}`
`85`	`85`
`86`		`-const _isUseHardLink = process.platform != "win32" && (isCi \|\| process.env.USE_HARD_LINKS === "true")`
	`86`	`+const _isUseHardLink = process.platform != "win32" && process.env.USE_HARD_LINKS !== "false" && (isCi \|\| process.env.USE_HARD_LINKS === "true")`
`87`	`87`
`88`	`88`	`/**`
`89`	`89`	`* Hard links is used if supported and allowed.`