[release test] use AnyscaleJobRunner in testing (#59858)

AnyscaleJobRunner is the only implementation/child class of
CommandRunner right now. There is no need to use inheritance.

Signed-off-by: Lonnie Liu <lonnie@anyscale.com>

Author: aslonnie

release/ray_release/tests/test_glue.py

@@ -11,7 +11,7 @@
 from ray_release.alerts.handle import result_to_handle_map
 from ray_release.cluster_manager.cluster_manager import ClusterManager
 from ray_release.cluster_manager.minimal import MinimalClusterManager
-from ray_release.command_runner.command_runner import CommandRunner
+from ray_release.command_runner.anyscale_job_runner import AnyscaleJobRunner
 from ray_release.exception import (
     CommandError,
     CommandTimeout,
@@ -25,6 +25,7 @@
     TestCommandError,
     TestCommandTimeout,
 )
+from ray_release.file_manager.job_file_manager import JobFileManager
 from ray_release.glue import (
     command_runner_to_cluster_manager,
     run_release_test,
@@ -123,17 +124,37 @@ def __init__(
                 self.return_dict = this_cluster_manager_return
                 this_instances["cluster_manager"] = self
 
-        class MockCommandRunner(MockReturn, CommandRunner):
+        class FakeFileManager(JobFileManager):
+            def __init__(self, cluster_manager: ClusterManager):
+                super(FakeFileManager, self).__init__(cluster_manager)
+
+            def download_from_cloud(
+                self, key: str, target: str, delete_after_download: bool = False
+            ):
+                with open(target, "wt") as f:
+                    f.write("fake download content")
+
+            def delete(self, key: str, recursive: bool = False):
+                pass
+
+        class MockCommandRunner(MockReturn, AnyscaleJobRunner):
             return_dict = self.cluster_manager_return
 
             def __init__(
                 self,
                 cluster_manager: ClusterManager,
+                file_manager: JobFileManager,
                 working_dir,
                 sdk=None,
                 artifact_path: Optional[str] = None,
             ):
-                super(MockCommandRunner, self).__init__(cluster_manager, this_tempdir)
+                super(MockCommandRunner, self).__init__(
+                    cluster_manager,
+                    FakeFileManager(cluster_manager),
+                    this_tempdir,
+                    sdk=this_sdk,
+                    artifact_path=artifact_path,
+                )
                 self.return_dict = this_command_runner_return
 
         self.mock_alert_return = None