Remove copy_dataset API (#218)

It is now no longer needed for any platform

Author: borrrden

environment/aws/topology_setup/test_server.py

@@ -43,7 +43,6 @@
 from __future__ import annotations
 
 import importlib
-import shutil
 from abc import ABC, abstractmethod
 from collections.abc import Callable
 from pathlib import Path
@@ -272,28 +271,4 @@ def create_bridge(self, **kwargs) -> PlatformBridge:
         pass
 
 
-def copy_dataset(dest_dir: Path):
-    header("Copying dataset resources")
-    click.secho(
-        "WARNING: This call is deprecated, and the server should not need this anymore.",
-        fg="yellow",
-    )
-    db_dir = TEST_SERVER_DIR.parent / "dataset" / "server" / "dbs" / "3.2"
-    blob_dir = TEST_SERVER_DIR.parent / "dataset" / "server" / "blobs"
-
-    dest_db_dir = dest_dir / "dbs"
-    shutil.rmtree(dest_db_dir, ignore_errors=True)
-    dest_db_dir.mkdir(0o755)
-    for db in db_dir.glob("*.zip"):
-        click.echo(f"Copying {db} -> {dest_db_dir / db.name}")
-        shutil.copy2(db, dest_db_dir)
-
-    dest_blob_dir = dest_dir / "blobs"
-    shutil.rmtree(dest_blob_dir, ignore_errors=True)
-    dest_blob_dir.mkdir(0o755)
-    for blob in blob_dir.iterdir():
-        click.echo(f"Copying {blob} -> {dest_blob_dir / blob.name}")
-        shutil.copy2(blob, dest_blob_dir)
-
-
 assert __name__ != "__main__", "This module is not meant to be run directly"

environment/aws/topology_setup/test_server_platforms/java_register.py

@@ -38,11 +38,7 @@
 
 from environment.aws.common.io import download_progress_bar, unzip_directory
 from environment.aws.common.output import header
-from environment.aws.topology_setup.test_server import (
-    TEST_SERVER_DIR,
-    TestServer,
-    copy_dataset,
-)
+from environment.aws.topology_setup.test_server import TEST_SERVER_DIR, TestServer
 
 from .android_bridge import AndroidBridge
 from .platform_bridge import PlatformBridge
@@ -251,21 +247,10 @@ def __init__(self, version: str, gradle_target: str = "jar"):
         super().__init__(version)
         self.__gradle_target = gradle_target
 
-    def _copy_dataset(self) -> None:
-        # The original script this was ported from copied all versions
-        # regardless of what was being built.  I opted to not do this
-        # and instead only copy the one that is going to be built
-        # so that way there is no need to keep a manual list of versions
-        # here.
-        dest_dir = JAK_TEST_SERVER_DIR / "assets" / "3.2"
-        dest_dir.mkdir(0o755, parents=True, exist_ok=True)
-        copy_dataset(dest_dir)
-
     def build(self) -> None:
         """
         Build the JAK test server.
         """
-        self._copy_dataset()
         gradle_path = JAK_TEST_SERVER_DIR / self.test_server_path / "gradlew"
         if platform.system() == "Windows":
             gradle_path = gradle_path.with_suffix(".bat")

environment/aws/topology_setup/test_server_platforms/swift_register.py

@@ -36,7 +36,6 @@
     DOWNLOADED_TEST_SERVER_DIR,
     TEST_SERVER_DIR,
     TestServer,
-    copy_dataset,
 )
 
 from .ios_bridge import iOSBridge
@@ -85,10 +84,6 @@ def _download_cbl(self) -> None:
         unzip_directory(download_file, FRAMEWORKS_DIR)
         download_file.unlink()
 
-    def _copy_dataset(self) -> None:
-        dest_dir = SWIFT_TEST_SERVER_DIR / "Assets"
-        copy_dataset(dest_dir)
-
 
 @TestServer.register("swift_ios")
 class SwiftTestServer_iOS(SwiftTestServer):
@@ -124,7 +119,6 @@ def latestbuilds_path(self) -> str:
         return f"couchbase-lite-ios/{version_parts[0]}/{version_parts[1]}/testserver_ios.zip"
 
     def build(self) -> None:
-        self._copy_dataset()
         self._download_cbl()
         header("Building")
         env = os.environ.copy()