Remove OptionalBool usage from ApiTest

Change-Id: I9383045f148c6328790b18811592703d15d8daaf

src/test/java/com/android/tools/r8/compilerapi/protectapisurface/ProtectApiSurfaceTest.java

diff --git a/src/test/java/com/android/tools/r8/compilerapi/protectapisurface/ProtectApiSurfaceTest.java b/src/test/java/com/android/tools/r8/compilerapi/protectapisurface/ProtectApiSurfaceTest.java
index b261a41..961512d 100644
--- a/src/test/java/com/android/tools/r8/compilerapi/protectapisurface/ProtectApiSurfaceTest.java
+++ b/src/test/java/com/android/tools/r8/compilerapi/protectapisurface/ProtectApiSurfaceTest.java
@@ -14,8 +14,8 @@
 import com.android.tools.r8.TestParameters;
 import com.android.tools.r8.compilerapi.CompilerApiTest;
 import com.android.tools.r8.compilerapi.CompilerApiTestRunner;
+import com.android.tools.r8.compilerapi.protectapisurface.ProtectApiSurfaceTest.ApiTest.ProtectApiSurfaceConfiguration;
 import com.android.tools.r8.origin.Origin;
-import com.android.tools.r8.utils.OptionalBool;
 import com.android.tools.r8.utils.codeinspector.CodeInspector;
 import com.android.tools.r8.utils.codeinspector.MethodSubject;
 import java.nio.file.Path;
@@ -37,31 +37,33 @@
 
   @Test
   public void testDefault() throws Exception {
-    runTest(OptionalBool.UNKNOWN);
+    runTest(ProtectApiSurfaceConfiguration.DEFAULT);
   }
 
   @Test
   public void testEnabled() throws Exception {
-    runTest(OptionalBool.TRUE);
+    runTest(ProtectApiSurfaceConfiguration.ENABLED);
   }
 
   @Test
   public void testDisabled() throws Exception {
-    runTest(OptionalBool.FALSE);
+    runTest(ProtectApiSurfaceConfiguration.DISABLED);
   }
 
-  private void runTest(OptionalBool protectApiSurface) throws Exception {
+  private void runTest(ProtectApiSurfaceConfiguration protectApiSurface) throws Exception {
     Path out = temp.newFolder().toPath().resolve("out.jar");
     ApiTest test = new ApiTest(ApiTest.PARAMETERS);
     test.runR8(new DexIndexedConsumer.ArchiveConsumer(out), protectApiSurface);
     inspect(new CodeInspector(out), protectApiSurface);
   }
 
-  private void inspect(CodeInspector inspector, OptionalBool protectApiSurface) {
+  private void inspect(CodeInspector inspector, ProtectApiSurfaceConfiguration protectApiSurface) {
     MethodSubject method =
         inspector.allClasses().iterator().next().uniqueMethodWithFinalName("greet");
     assertThat(method, isPresent());
-    assertThat(method, protectApiSurface.isTrue() ? isPrivate() : isPublic());
+    assertThat(
+        method,
+        protectApiSurface == ProtectApiSurfaceConfiguration.ENABLED ? isPrivate() : isPublic());
   }
 
   public static class ApiTest extends CompilerApiTest {
@@ -70,7 +72,8 @@
       super(parameters);
     }
 
-    public void runR8(DexIndexedConsumer programConsumer, OptionalBool protectApiSurface)
+    public void runR8(
+        DexIndexedConsumer programConsumer, ProtectApiSurfaceConfiguration protectApiSurface)
         throws Exception {
       R8Command.Builder commandBuilder =
           R8Command.builder()
@@ -82,15 +85,22 @@
               .addLibraryFiles(getAndroidJar())
               .setMinApiLevel(SOME_API_LEVEL)
               .setProgramConsumer(programConsumer);
-      if (!protectApiSurface.isUnknown()) {
-        commandBuilder.setProtectApiSurface(protectApiSurface.isTrue());
+      if (protectApiSurface != ProtectApiSurfaceConfiguration.DEFAULT) {
+        commandBuilder.setProtectApiSurface(
+            protectApiSurface == ProtectApiSurfaceConfiguration.ENABLED);
       }
       R8.run(commandBuilder.build());
     }
 
     @Test
     public void testEnabled() throws Exception {
-      runR8(DexIndexedConsumer.emptyConsumer(), OptionalBool.TRUE);
+      runR8(DexIndexedConsumer.emptyConsumer(), ProtectApiSurfaceConfiguration.ENABLED);
+    }
+
+    public enum ProtectApiSurfaceConfiguration {
+      DEFAULT,
+      ENABLED,
+      DISABLED
     }
   }
 }

third_party/binary_compatibility_tests/compiler_api_tests.tar.gz.sha1

diff --git a/third_party/binary_compatibility_tests/compiler_api_tests.tar.gz.sha1 b/third_party/binary_compatibility_tests/compiler_api_tests.tar.gz.sha1
index 68e1371..0ecf6f8 100644
--- a/third_party/binary_compatibility_tests/compiler_api_tests.tar.gz.sha1
+++ b/third_party/binary_compatibility_tests/compiler_api_tests.tar.gz.sha1
@@ -1 +1 @@
-78a7774ece1679df4ccc301a976155d45660a73d
\ No newline at end of file
+bc8b8c206fce4217e71e17b3f23a94449892db6c
\ No newline at end of file