From b5da80202f41e6f293153a2df945063d584b022a Mon Sep 17 00:00:00 2001 From: Bill Ticehurst Date: Tue, 1 Mar 2016 10:36:37 -0800 Subject: [PATCH] Changed how parserContextFlags are handled in tests --- src/harness/harness.ts | 4 ++++ tests/cases/unittests/jsDocParsing.ts | 5 ----- 2 files changed, 4 insertions(+), 5 deletions(-) diff --git a/src/harness/harness.ts b/src/harness/harness.ts index 5c191a3b9d..858607143d 100644 --- a/src/harness/harness.ts +++ b/src/harness/harness.ts @@ -285,6 +285,10 @@ namespace Utils { // data we don't care about in the dump. We only care what the parser set directly // on the ast. let value = n.parserContextFlags & ts.ParserContextFlags.ParserGeneratedFlags; + + // Remove the JavaScriptFile flag, as this is just noise for JSDocParser and Test262 tests + // (which are the only tests that use this function). + value = value & ~ts.ParserContextFlags.JavaScriptFile; if (value) { o[propertyName] = getParserContextFlagName(value); } diff --git a/tests/cases/unittests/jsDocParsing.ts b/tests/cases/unittests/jsDocParsing.ts index e2941f7fc1..9c68cea329 100644 --- a/tests/cases/unittests/jsDocParsing.ts +++ b/tests/cases/unittests/jsDocParsing.ts @@ -12,8 +12,6 @@ module ts { let result = Utils.sourceFileToJSON(typeAndDiagnostics.jsDocTypeExpression.type); - // Remove the parserContextFlags from the comparison - result = result.replace(/\,\n\s+\"parserContextFlags\": \"JavaScriptFile\"/g, ""); assert.equal(result, expected); } @@ -1002,9 +1000,6 @@ module ts { : v; }, 4); - // Remove the parserContextFlags from the comparison - result = result.replace(/\,\n\s+\"parserContextFlags\": \"JavaScriptFile\"/g, ""); - if (result !== expected) { // Turn on a human-readable diff if (typeof require !== 'undefined') {