No need to fix hash of lib files since the content is now fixed

sheetalkamat · sheetalkamat · commit acbedade4249 · 2019-04-17T15:37:56.000-07:00
diff --git a/src/harness/fakes.ts b/src/harness/fakes.ts
@@ -397,19 +397,6 @@ namespace fakes {
             const value = super.readFile(path);
             if (!value || !ts.isBuildInfoFile(path)) return value;
             const buildInfo = ts.getBuildInfo(value);
-            if (buildInfo.program) {
-                // Fix lib signatures
-                for (const path of ts.getOwnKeys(buildInfo.program.fileInfos)) {
-                    if (ts.startsWith(path, "/lib/")) {
-                        const currentValue = buildInfo.program.fileInfos[path];
-                        ts.Debug.assert(currentValue.signature === path);
-                        ts.Debug.assert(currentValue.signature === currentValue.version);
-                        const text = super.readFile(path)!;
-                        const signature = ts.generateDjb2Hash(text);
-                        buildInfo.program.fileInfos[path] = { version: signature, signature };
-                    }
-                }
-            }
             ts.Debug.assert(buildInfo.version === version);
             buildInfo.version = ts.version;
             return ts.getBuildInfoText(buildInfo);
@@ -419,15 +406,6 @@ namespace fakes {
             if (!ts.isBuildInfoFile(fileName)) return super.writeFile(fileName, content, writeByteOrderMark);
             const buildInfo = ts.getBuildInfo(content);
             if (buildInfo.program) {
-                // Fix lib signatures
-                for (const path of ts.getOwnKeys(buildInfo.program.fileInfos)) {
-                    if (ts.startsWith(path, "/lib/")) {
-                        const currentValue = buildInfo.program.fileInfos[path];
-                        ts.Debug.assert(currentValue.signature === currentValue.version);
-                        buildInfo.program.fileInfos[path] = { version: path, signature: path };
-                    }
-                }
-
                 // reference Map
                 if (buildInfo.program.referencedMap) {
                     const referencedMap: ts.MapLike<string[]> = {};
diff --git a/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/sample.js b/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/sample.js
@@ -172,8 +172,8 @@ export class someClass { }
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
@@ -212,8 +212,8 @@ export class someClass { }
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-2069755619",
@@ -262,8 +262,8 @@ export class someClass { }
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-2069755619",
diff --git a/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/when-declaration-option-changes.js b/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/when-declaration-option-changes.js
@@ -21,8 +21,8 @@ export declare function multiply(a: number, b: number): number;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
diff --git a/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/when-logic-config-changes-declaration-dir.js b/tests/baselines/reference/tsbuild/sample1/incremental-declaration-changes/when-logic-config-changes-declaration-dir.js
@@ -25,8 +25,8 @@ export declare const m: typeof mod;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
@@ -76,8 +76,8 @@ export declare const m: typeof mod;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
diff --git a/tests/baselines/reference/tsbuild/sample1/incremental-declaration-doesnt-change/sample.js b/tests/baselines/reference/tsbuild/sample1/incremental-declaration-doesnt-change/sample.js
@@ -25,8 +25,8 @@ class someClass { }
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
diff --git a/tests/baselines/reference/tsbuild/sample1/initial-Build/sample.js b/tests/baselines/reference/tsbuild/sample1/initial-Build/sample.js
@@ -185,8 +185,8 @@ exports.multiply = multiply;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
@@ -370,8 +370,8 @@ sourceFile:index.ts
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
@@ -436,8 +436,8 @@ exports.m = mod;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
diff --git a/tests/baselines/reference/tsbuild/sample1/initial-Build/when-declaration-option-changes.js b/tests/baselines/reference/tsbuild/sample1/initial-Build/when-declaration-option-changes.js
@@ -27,8 +27,8 @@ exports.multiply = multiply;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
diff --git a/tests/baselines/reference/tsbuild/sample1/initial-Build/when-logic-config-changes-declaration-dir.js b/tests/baselines/reference/tsbuild/sample1/initial-Build/when-logic-config-changes-declaration-dir.js
@@ -185,8 +185,8 @@ exports.multiply = multiply;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
@@ -370,8 +370,8 @@ sourceFile:index.ts
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
@@ -436,8 +436,8 @@ exports.m = mod;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
diff --git a/tests/baselines/reference/tsbuild/sample1/initial-Build/when-logic-specifies-tsBuildInfoFile.js b/tests/baselines/reference/tsbuild/sample1/initial-Build/when-logic-specifies-tsBuildInfoFile.js
@@ -185,8 +185,8 @@ exports.multiply = multiply;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/anothermodule.ts": {
         "version": "-2676574883",
@@ -370,8 +370,8 @@ sourceFile:index.ts
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
@@ -453,8 +453,8 @@ exports.m = mod;
   "program": {
     "fileInfos": {
       "/lib/lib.d.ts": {
-        "version": "/lib/lib.d.ts",
-        "signature": "/lib/lib.d.ts"
+        "version": "-15964756381",
+        "signature": "-15964756381"
       },
       "/src/core/index.ts": {
         "version": "-13851440507",
