Skip to content

Commit 8472f96

Browse files
weswighamweswigham
committed
Remove extraneous arguments from runBaseline
1 parent f57b0fb commit 8472f96

File tree

8 files changed

+47
-70
lines changed

8 files changed

+47
-70
lines changed

‎src/harness/compilerRunner.ts

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ class CompilerBaselineRunner extends RunnerBase {
147147
// check errors
148148
it("Correct errors for " + fileName, () => {
149149
if (this.errors) {
150-
Harness.Baseline.runBaseline("Correct errors for " + fileName, justName.replace(/\.tsx?$/, ".errors.txt"), (): string => {
150+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?$/, ".errors.txt"), (): string => {
151151
if (result.errors.length === 0) return null;
152152
return getErrorBaseline(toBeCompiled, otherFiles, result);
153153
});
@@ -156,7 +156,7 @@ class CompilerBaselineRunner extends RunnerBase {
156156

157157
it (`Correct module resolution tracing for ${fileName}`, () => {
158158
if (options.traceResolution) {
159-
Harness.Baseline.runBaseline("Correct module resolution tracing for " + fileName, justName.replace(/\.tsx?$/, ".trace.json"), () => {
159+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?$/, ".trace.json"), () => {
160160
return JSON.stringify(result.traceResults || [], undefined, 4);
161161
});
162162
}
@@ -165,7 +165,7 @@ class CompilerBaselineRunner extends RunnerBase {
165165
// Source maps?
166166
it("Correct sourcemap content for " + fileName, () => {
167167
if (options.sourceMap || options.inlineSourceMap) {
168-
Harness.Baseline.runBaseline("Correct sourcemap content for " + fileName, justName.replace(/\.tsx?$/, ".sourcemap.txt"), () => {
168+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?$/, ".sourcemap.txt"), () => {
169169
const record = result.getSourceMapRecord();
170170
if (options.noEmitOnError && result.errors.length !== 0 && record === undefined) {
171171
// Because of the noEmitOnError option no files are created. We need to return null because baselining isn"t required.
@@ -183,7 +183,7 @@ class CompilerBaselineRunner extends RunnerBase {
183183
}
184184

185185
// check js output
186-
Harness.Baseline.runBaseline("Correct JS output for " + fileName, justName.replace(/\.tsx?/, ".js"), () => {
186+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?/, ".js"), () => {
187187
let tsCode = "";
188188
const tsSources = otherFiles.concat(toBeCompiled);
189189
if (tsSources.length > 1) {
@@ -242,7 +242,7 @@ class CompilerBaselineRunner extends RunnerBase {
242242
throw new Error("Number of sourcemap files should be same as js files.");
243243
}
244244

245-
Harness.Baseline.runBaseline("Correct Sourcemap output for " + fileName, justName.replace(/\.tsx?/, ".js.map"), () => {
245+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?/, ".js.map"), () => {
246246
if (options.noEmitOnError && result.errors.length !== 0 && result.sourceMaps.length === 0) {
247247
// We need to return null here or the runBaseLine will actually create a empty file.
248248
// Baselining isn't required here because there is no output.
@@ -330,11 +330,11 @@ class CompilerBaselineRunner extends RunnerBase {
330330
const pullExtension = isSymbolBaseLine ? ".symbols.pull" : ".types.pull";
331331

332332
if (fullBaseLine !== pullBaseLine) {
333-
Harness.Baseline.runBaseline("Correct full information for " + fileName, justName.replace(/\.tsx?/, fullExtension), () => fullBaseLine);
334-
Harness.Baseline.runBaseline("Correct pull information for " + fileName, justName.replace(/\.tsx?/, pullExtension), () => pullBaseLine);
333+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?/, fullExtension), () => fullBaseLine);
334+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?/, pullExtension), () => pullBaseLine);
335335
}
336336
else {
337-
Harness.Baseline.runBaseline("Correct information for " + fileName, justName.replace(/\.tsx?/, fullExtension), () => fullBaseLine);
337+
Harness.Baseline.runBaseline(justName.replace(/\.tsx?/, fullExtension), () => fullBaseLine);
338338
}
339339
}
340340

‎src/harness/fourslash.ts

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1132,12 +1132,10 @@ namespace FourSlash {
11321132

11331133
}
11341134
Harness.Baseline.runBaseline(
1135-
"Breakpoint Locations for " + this.activeFile.fileName,
11361135
baselineFile,
11371136
() => {
11381137
return this.baselineCurrentFileLocations(pos => this.getBreakpointStatementLocation(pos));
1139-
},
1140-
true /* run immediately */);
1138+
});
11411139
}
11421140

11431141
public baselineGetEmitOutput() {
@@ -1159,7 +1157,6 @@ namespace FourSlash {
11591157
}
11601158

11611159
Harness.Baseline.runBaseline(
1162-
"Generate getEmitOutput baseline : " + emitFiles.join(" "),
11631160
this.testData.globalOptions[metadataOptionNames.baselineFile],
11641161
() => {
11651162
let resultString = "";
@@ -1185,8 +1182,7 @@ namespace FourSlash {
11851182
});
11861183

11871184
return resultString;
1188-
},
1189-
true /* run immediately */);
1185+
});
11901186
}
11911187

11921188
public printBreakpointLocation(pos: number) {
@@ -1730,13 +1726,11 @@ namespace FourSlash {
17301726

17311727
public baselineCurrentFileNameOrDottedNameSpans() {
17321728
Harness.Baseline.runBaseline(
1733-
"Name OrDottedNameSpans for " + this.activeFile.fileName,
17341729
this.testData.globalOptions[metadataOptionNames.baselineFile],
17351730
() => {
17361731
return this.baselineCurrentFileLocations(pos =>
17371732
this.getNameOrDottedNameSpan(pos));
1738-
},
1739-
true /* run immediately */);
1733+
});
17401734
}
17411735

17421736
public printNameOrDottedNameSpans(pos: number) {

‎src/harness/harness.ts

Lines changed: 8 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1663,43 +1663,26 @@ namespace Harness {
16631663
return { expected, actual };
16641664
}
16651665

1666-
function writeComparison(expected: string, actual: string, relativeFileName: string, actualFileName: string, descriptionForDescribe: string) {
1666+
function writeComparison(expected: string, actual: string, relativeFileName: string, actualFileName: string) {
16671667
const encoded_actual = Utils.encodeString(actual);
16681668
if (expected !== encoded_actual) {
16691669
if (actual === NoContent) {
1670-
IO.writeFile(localPath(relativeFileName + ".delete"), "");
1670+
IO.writeFile(actualFileName + ".delete", "");
16711671
}
16721672
else {
1673-
IO.writeFile(localPath(relativeFileName), actual);
1673+
IO.writeFile(actualFileName, actual);
16741674
}
1675-
// Overwrite & issue error
1676-
const errMsg = "The baseline file " + relativeFileName + " has changed.";
1677-
throw new Error(errMsg);
1675+
throw new Error(`The baseline file ${relativeFileName} has changed.`);
16781676
}
16791677
}
16801678

1679+
export function runBaseline(relativeFileName: string, generateContent: () => string, opts?: BaselineOptions): void {
16811680

1682-
export function runBaseline(
1683-
descriptionForDescribe: string,
1684-
relativeFileName: string,
1685-
generateContent: () => string,
1686-
runImmediately = false,
1687-
opts?: BaselineOptions): void {
1688-
1689-
let actual = <string>undefined;
16901681
const actualFileName = localPath(relativeFileName, opts && opts.Baselinefolder, opts && opts.Subfolder);
16911682

1692-
if (runImmediately) {
1693-
actual = generateActual(actualFileName, generateContent);
1694-
const comparison = compareToBaseline(actual, relativeFileName, opts);
1695-
writeComparison(comparison.expected, comparison.actual, relativeFileName, actualFileName, descriptionForDescribe);
1696-
}
1697-
else {
1698-
actual = generateActual(actualFileName, generateContent);
1699-
1700-
const comparison = compareToBaseline(actual, relativeFileName, opts);
1701-
writeComparison(comparison.expected, comparison.actual, relativeFileName, actualFileName, descriptionForDescribe);
1702-
}
1683+
const actual = generateActual(actualFileName, generateContent);
1684+
const comparison = compareToBaseline(actual, relativeFileName, opts);
1685+
writeComparison(comparison.expected, comparison.actual, relativeFileName, actualFileName);
17031686
}
17041687
}
17051688

‎src/harness/projectsRunner.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -459,15 +459,15 @@ class ProjectRunner extends RunnerBase {
459459
});
460460

461461
it("Resolution information of (" + moduleNameToString(moduleKind) + "): " + testCaseFileName, () => {
462-
Harness.Baseline.runBaseline("Resolution information of (" + moduleNameToString(compilerResult.moduleKind) + "): " + testCaseFileName, getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".json", () => {
462+
Harness.Baseline.runBaseline(getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".json", () => {
463463
return JSON.stringify(getCompilerResolutionInfo(), undefined, " ");
464464
});
465465
});
466466

467467

468468
it("Errors for (" + moduleNameToString(moduleKind) + "): " + testCaseFileName, () => {
469469
if (compilerResult.errors.length) {
470-
Harness.Baseline.runBaseline("Errors for (" + moduleNameToString(compilerResult.moduleKind) + "): " + testCaseFileName, getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".errors.txt", () => {
470+
Harness.Baseline.runBaseline(getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".errors.txt", () => {
471471
return getErrorsBaseline(compilerResult);
472472
});
473473
}
@@ -481,7 +481,7 @@ class ProjectRunner extends RunnerBase {
481481
// There may be multiple files with different baselines. Run all and report at the end, else
482482
// it stops copying the remaining emitted files from 'local/projectOutput' to 'local/project'.
483483
try {
484-
Harness.Baseline.runBaseline("Baseline of emitted result (" + moduleNameToString(compilerResult.moduleKind) + "): " + testCaseFileName, getBaselineFolder(compilerResult.moduleKind) + outputFile.fileName, () => {
484+
Harness.Baseline.runBaseline(getBaselineFolder(compilerResult.moduleKind) + outputFile.fileName, () => {
485485
try {
486486
return Harness.IO.readFile(getProjectOutputFolder(outputFile.fileName, compilerResult.moduleKind));
487487
}
@@ -503,7 +503,7 @@ class ProjectRunner extends RunnerBase {
503503

504504
it("SourceMapRecord for (" + moduleNameToString(moduleKind) + "): " + testCaseFileName, () => {
505505
if (compilerResult.sourceMapData) {
506-
Harness.Baseline.runBaseline("SourceMapRecord for (" + moduleNameToString(compilerResult.moduleKind) + "): " + testCaseFileName, getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".sourcemap.txt", () => {
506+
Harness.Baseline.runBaseline(getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".sourcemap.txt", () => {
507507
return Harness.SourceMapRecorder.getSourceMapRecord(compilerResult.sourceMapData, compilerResult.program,
508508
ts.filter(compilerResult.outputFiles, outputFile => Harness.Compiler.isJS(outputFile.emittedFileName)));
509509
});
@@ -516,7 +516,7 @@ class ProjectRunner extends RunnerBase {
516516
if (!compilerResult.errors.length && testCase.declaration) {
517517
const dTsCompileResult = compileCompileDTsFiles(compilerResult);
518518
if (dTsCompileResult && dTsCompileResult.errors.length) {
519-
Harness.Baseline.runBaseline("Errors in generated Dts files for (" + moduleNameToString(compilerResult.moduleKind) + "): " + testCaseFileName, getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".dts.errors.txt", () => {
519+
Harness.Baseline.runBaseline(getBaselineFolder(compilerResult.moduleKind) + testCaseJustName + ".dts.errors.txt", () => {
520520
return getErrorsBaseline(dTsCompileResult);
521521
});
522522
}

‎src/harness/rwcRunner.ts

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -158,55 +158,55 @@ namespace RWC {
158158

159159

160160
it("has the expected emitted code", () => {
161-
Harness.Baseline.runBaseline("has the expected emitted code", baseName + ".output.js", () => {
161+
Harness.Baseline.runBaseline(baseName + ".output.js", () => {
162162
return Harness.Compiler.collateOutputs(compilerResult.files);
163-
}, false, baselineOpts);
163+
}, baselineOpts);
164164
});
165165

166166
it("has the expected declaration file content", () => {
167-
Harness.Baseline.runBaseline("has the expected declaration file content", baseName + ".d.ts", () => {
167+
Harness.Baseline.runBaseline(baseName + ".d.ts", () => {
168168
if (!compilerResult.declFilesCode.length) {
169169
return null;
170170
}
171171

172172
return Harness.Compiler.collateOutputs(compilerResult.declFilesCode);
173-
}, false, baselineOpts);
173+
}, baselineOpts);
174174
});
175175

176176
it("has the expected source maps", () => {
177-
Harness.Baseline.runBaseline("has the expected source maps", baseName + ".map", () => {
177+
Harness.Baseline.runBaseline(baseName + ".map", () => {
178178
if (!compilerResult.sourceMaps.length) {
179179
return null;
180180
}
181181

182182
return Harness.Compiler.collateOutputs(compilerResult.sourceMaps);
183-
}, false, baselineOpts);
183+
}, baselineOpts);
184184
});
185185

186186
/*it("has correct source map record", () => {
187187
if (compilerOptions.sourceMap) {
188-
Harness.Baseline.runBaseline("has correct source map record", baseName + ".sourcemap.txt", () => {
188+
Harness.Baseline.runBaseline(baseName + ".sourcemap.txt", () => {
189189
return compilerResult.getSourceMapRecord();
190-
}, false, baselineOpts);
190+
}, baselineOpts);
191191
}
192192
});*/
193193

194194
it("has the expected errors", () => {
195-
Harness.Baseline.runBaseline("has the expected errors", baseName + ".errors.txt", () => {
195+
Harness.Baseline.runBaseline(baseName + ".errors.txt", () => {
196196
if (compilerResult.errors.length === 0) {
197197
return null;
198198
}
199199
// Do not include the library in the baselines to avoid noise
200200
const baselineFiles = inputFiles.concat(otherFiles).filter(f => !Harness.isDefaultLibraryFile(f.unitName));
201201
return Harness.Compiler.getErrorBaseline(baselineFiles, compilerResult.errors);
202-
}, false, baselineOpts);
202+
}, baselineOpts);
203203
});
204204

205205
// Ideally, a generated declaration file will have no errors. But we allow generated
206206
// declaration file errors as part of the baseline.
207207
it("has the expected errors in generated declaration files", () => {
208208
if (compilerOptions.declaration && !compilerResult.errors.length) {
209-
Harness.Baseline.runBaseline("has the expected errors in generated declaration files", baseName + ".dts.errors.txt", () => {
209+
Harness.Baseline.runBaseline(baseName + ".dts.errors.txt", () => {
210210
const declFileCompilationResult = Harness.Compiler.compileDeclarationFiles(
211211
inputFiles, otherFiles, compilerResult, /*harnessSettings*/ undefined, compilerOptions, currentDirectory);
212212

@@ -217,7 +217,7 @@ namespace RWC {
217217
return Harness.Compiler.minimalDiagnosticsToString(declFileCompilationResult.declResult.errors) +
218218
Harness.IO.newLine() + Harness.IO.newLine() +
219219
Harness.Compiler.getErrorBaseline(declFileCompilationResult.declInputFiles.concat(declFileCompilationResult.declOtherFiles), declFileCompilationResult.declResult.errors);
220-
}, false, baselineOpts);
220+
}, baselineOpts);
221221
}
222222
});
223223

‎src/harness/test262Runner.ts

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -67,21 +67,21 @@ class Test262BaselineRunner extends RunnerBase {
6767
});
6868

6969
it("has the expected emitted code", () => {
70-
Harness.Baseline.runBaseline("has the expected emitted code", testState.filename + ".output.js", () => {
70+
Harness.Baseline.runBaseline(testState.filename + ".output.js", () => {
7171
const files = testState.compilerResult.files.filter(f => f.fileName !== Test262BaselineRunner.helpersFilePath);
7272
return Harness.Compiler.collateOutputs(files);
73-
}, false, Test262BaselineRunner.baselineOptions);
73+
}, Test262BaselineRunner.baselineOptions);
7474
});
7575

7676
it("has the expected errors", () => {
77-
Harness.Baseline.runBaseline("has the expected errors", testState.filename + ".errors.txt", () => {
77+
Harness.Baseline.runBaseline(testState.filename + ".errors.txt", () => {
7878
const errors = testState.compilerResult.errors;
7979
if (errors.length === 0) {
8080
return null;
8181
}
8282

8383
return Harness.Compiler.getErrorBaseline(testState.inputFiles, errors);
84-
}, false, Test262BaselineRunner.baselineOptions);
84+
}, Test262BaselineRunner.baselineOptions);
8585
});
8686

8787
it("satisfies invariants", () => {
@@ -90,10 +90,10 @@ class Test262BaselineRunner extends RunnerBase {
9090
});
9191

9292
it("has the expected AST", () => {
93-
Harness.Baseline.runBaseline("has the expected AST", testState.filename + ".AST.txt", () => {
93+
Harness.Baseline.runBaseline(testState.filename + ".AST.txt", () => {
9494
const sourceFile = testState.compilerResult.program.getSourceFile(Test262BaselineRunner.getTestFilePath(testState.filename));
9595
return Utils.sourceFileToJSON(sourceFile);
96-
}, false, Test262BaselineRunner.baselineOptions);
96+
}, Test262BaselineRunner.baselineOptions);
9797
});
9898
});
9999
}

‎src/harness/unittests/initializeTSConfig.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ namespace ts {
1010
const outputFileName = `tsConfig/${name.replace(/[^a-z0-9\-. ]/ig, "")}/tsconfig.json`;
1111

1212
it(`Correct output for ${outputFileName}`, () => {
13-
Harness.Baseline.runBaseline("Correct output", outputFileName, () => {
13+
Harness.Baseline.runBaseline(outputFileName, () => {
1414
if (initResult) {
1515
return JSON.stringify(initResult, undefined, 4);
1616
}

0 commit comments

Comments
 (0)