Skip to content

Commit

Permalink
Remove a deprecated method in CompilerTestCase.
Browse files Browse the repository at this point in the history
-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=126705356
  • Loading branch information
tbreisacher authored and blickly committed Jul 7, 2016
1 parent 5474f3f commit e74ca62
Showing 1 changed file with 4 additions and 21 deletions.
25 changes: 4 additions & 21 deletions test/com/google/javascript/jscomp/CompilerTestCase.java
Original file line number Diff line number Diff line change
Expand Up @@ -90,8 +90,6 @@ public abstract class CompilerTestCase extends TestCase {
*/
private boolean newTypeInferenceEnabled = false;

@Deprecated private CheckLevel reportMissingOverrideCheckLevel = CheckLevel.WARNING;

/** Whether to test the compiler pass before the type check. */
protected boolean runTypeCheckAfterProcessing = false;

Expand Down Expand Up @@ -381,21 +379,6 @@ protected void allowExternsChanges(boolean allowExternsChanges) {
this.allowExternsChanges = allowExternsChanges;
}

/**
* Perform type checking before running the test pass. This will check
* for type errors and annotate nodes with type information.
*
* @param level the level of severity to report for type errors
*
* @deprecated Use enableTypeCheck()
* @see TypeCheck
*/
@Deprecated
public void enableTypeCheck(CheckLevel level) {
enableTypeCheck();
reportMissingOverrideCheckLevel = level;
}

/**
* Perform type checking before running the test pass. This will check
* for type errors and annotate nodes with type information.
Expand Down Expand Up @@ -540,11 +523,11 @@ protected void setExpectParseWarningsThisTest() {
}

/** Returns a newly created TypeCheck. */
private static TypeCheck createTypeCheck(Compiler compiler, CheckLevel level) {
private static TypeCheck createTypeCheck(Compiler compiler) {
ReverseAbstractInterpreter rai =
new SemanticReverseAbstractInterpreter(compiler.getTypeRegistry());

return new TypeCheck(compiler, rai, compiler.getTypeRegistry(), level);
return new TypeCheck(compiler, rai, compiler.getTypeRegistry());
}

private static void runNewTypeInference(Compiler compiler, Node externs, Node js) {
Expand Down Expand Up @@ -1273,7 +1256,7 @@ private void test(
// objects for the same type are created, and the type system
// uses reference equality to compare many types.
if (!runTypeCheckAfterProcessing && typeCheckEnabled && i == 0) {
TypeCheck check = createTypeCheck(compiler, reportMissingOverrideCheckLevel);
TypeCheck check = createTypeCheck(compiler);
check.processForTesting(externsRoot, mainRoot);
} else if (!this.runNTIAfterProcessing
&& this.newTypeInferenceEnabled
Expand Down Expand Up @@ -1316,7 +1299,7 @@ private void test(
}

if (runTypeCheckAfterProcessing && typeCheckEnabled && i == 0) {
TypeCheck check = createTypeCheck(compiler, reportMissingOverrideCheckLevel);
TypeCheck check = createTypeCheck(compiler);
check.processForTesting(externsRoot, mainRoot);
} else if (this.runNTIAfterProcessing
&& this.newTypeInferenceEnabled
Expand Down

0 comments on commit e74ca62

Please sign in to comment.