Get rid of BotorchTestProblemRunner (#2945)

Summary:

It's redundant as per previous diff.

Reviewed By: saitcakmak

Differential Revision: D63722334

ax/benchmark/runners/botorch_test.py

@@ -207,6 +207,3 @@ def get_Y_true(self, params: Mapping[str, TParamValue]) -> Tensor:
                 dim=-1,
             )
         return Y_true
-
-
-BotorchTestProblemRunner = ParamBasedTestProblemRunner

ax/storage/tests/test_registry_bundle.py

@@ -6,7 +6,7 @@
 # pyre-strict
 
 from ax.benchmark.benchmark_metric import BenchmarkMetric
-from ax.benchmark.runners.botorch_test import BotorchTestProblemRunner
+from ax.benchmark.runners.botorch_test import ParamBasedTestProblemRunner
 from ax.metrics.branin import BraninMetric
 from ax.runners.synthetic import SyntheticRunner
 from ax.storage.registry_bundle import RegistryBundle
@@ -26,7 +26,7 @@ def test_from_registry_bundles(self) -> None:
 
         right = RegistryBundle(
             metric_clss={BenchmarkMetric: None},
-            runner_clss={BotorchTestProblemRunner: None},
+            runner_clss={ParamBasedTestProblemRunner: None},
             json_encoder_registry={},
             json_class_encoder_registry={},
             json_decoder_registry={},
@@ -41,4 +41,4 @@ def test_from_registry_bundles(self) -> None:
         self.assertIn(BraninMetric, combined.encoder_registry)
         self.assertIn(SyntheticRunner, combined.encoder_registry)
         self.assertIn(BenchmarkMetric, combined.encoder_registry)
-        self.assertIn(BotorchTestProblemRunner, combined.encoder_registry)
+        self.assertIn(ParamBasedTestProblemRunner, combined.encoder_registry)