Skip to content

Commit

Permalink
Remove absolute pathnames from benchmark comparison
Browse files Browse the repository at this point in the history
  • Loading branch information
Burt Harris authored and Burt Harris committed Oct 12, 2016
1 parent 80afd31 commit a6812f0
Show file tree
Hide file tree
Showing 2 changed files with 16 additions and 10 deletions.
20 changes: 10 additions & 10 deletions baselines/reference/test.grammar.diagnostics
@@ -1,11 +1,11 @@
/// test.grammar:
C:/dev/grammarkdown/spec/test.grammar(7,1): error GM2006: Production 'D' is missing parameter 'B'. All definitions of production 'D' must specify the same formal parameters.
C:/dev/grammarkdown/spec/test.grammar(8,1): error GM2006: Production 'D' is missing parameter 'A'. All definitions of production 'D' must specify the same formal parameters.
C:/dev/grammarkdown/spec/test.grammar(12,5): error GM2007: There is no argument given for parameter 'A'.
C:/dev/grammarkdown/spec/test.grammar(13,7): error GM1001: '?', '+', or '~' expected.
C:/dev/grammarkdown/spec/test.grammar(14,7): error GM1001: '?', '+', or '~' expected.
C:/dev/grammarkdown/spec/test.grammar(14,10): error GM1001: '?', '+', or '~' expected.
C:/dev/grammarkdown/spec/test.grammar(14,10): error GM2004: Production 'E' does not have a parameter named 'B'.
C:/dev/grammarkdown/spec/test.grammar(15,5): error GM2007: There is no argument given for parameter 'A'.
C:/dev/grammarkdown/spec/test.grammar(15,8): error GM2004: Production 'E' does not have a parameter named 'B'.
C:/dev/grammarkdown/spec/test.grammar(15,8): error GM2004: Production 'F' does not have a parameter named 'B'.
./spec/test.grammar(7,1): error GM2006: Production 'D' is missing parameter 'B'. All definitions of production 'D' must specify the same formal parameters.
./spec/test.grammar(8,1): error GM2006: Production 'D' is missing parameter 'A'. All definitions of production 'D' must specify the same formal parameters.
./spec/test.grammar(12,5): error GM2007: There is no argument given for parameter 'A'.
./spec/test.grammar(13,7): error GM1001: '?', '+', or '~' expected.
./spec/test.grammar(14,7): error GM1001: '?', '+', or '~' expected.
./spec/test.grammar(14,10): error GM1001: '?', '+', or '~' expected.
./spec/test.grammar(14,10): error GM2004: Production 'E' does not have a parameter named 'B'.
./spec/test.grammar(15,5): error GM2007: There is no argument given for parameter 'A'.
./spec/test.grammar(15,8): error GM2004: Production 'E' does not have a parameter named 'B'.
./spec/test.grammar(15,8): error GM2004: Production 'F' does not have a parameter named 'B'.
6 changes: 6 additions & 0 deletions src/tests/diff.ts
Expand Up @@ -73,6 +73,12 @@ export function writeDiagnostics(test: string, diagnostics: DiagnosticMessages,
text += message + EOL;
});

if (!!text) {
// Remove absolute pathnames from text
const root = resolve(".").replace(/\\/g,"/");
const re = new RegExp(root.replace(/[.?*+^$[\]\\(){}|-]/g, "\\$&"), "gi");
text = text.replace(re, ".");
}
return writeBaseline(test + ".diagnostics", text, baselines);
}

Expand Down

0 comments on commit a6812f0

Please sign in to comment.