✅ Improve tests reduce runtime

- Improve tests reduce runtime

Signed-off-by: Shan E Ahmed Raza <[email protected]>

Author: shaneahmed

tests/conftest.py

@@ -144,6 +144,25 @@ def sample_all_wsis2(sample_ndpi2, sample_svs, sample_jp2, tmpdir_factory):
     return dir_path
 
 
+@pytest.fixture(scope="session")
+def sample_svs_ndpi_wsis(sample_ndpi2, sample_svs, tmpdir_factory):
+    """Sample SVS and NDPI wsi(s).
+
+    Uses sample fluorescence ndpi image.
+
+    """
+    dir_path = pathlib.Path(tmpdir_factory.mktemp("data"))
+
+    try:
+        dir_path.joinpath(sample_ndpi2.name).symlink_to(sample_ndpi2)
+        dir_path.joinpath(sample_svs.name).symlink_to(sample_svs)
+    except OSError:
+        shutil.copy(sample_ndpi2, dir_path.joinpath(sample_ndpi2.name))
+        shutil.copy(sample_svs, dir_path.joinpath(sample_svs.name))
+
+    return dir_path
+
+
 @pytest.fixture(scope="session")
 def source_image(remote_sample) -> pathlib.Path:
     """Sample pytest fixture for source image.

tests/test_save_tiles.py

@@ -12,17 +12,17 @@
 # -------------------------------------------------------------------------------------
 
 
-def test_command_line_save_tiles(sample_all_wsis2, tmp_path):
+def test_command_line_save_tiles(sample_svs_ndpi_wsis, tmp_path):
     """Test for save_tiles CLI."""
     runner = CliRunner()
     save_tiles_result = runner.invoke(
         cli.main,
         [
             "save-tiles",
             "--img-input",
-            str(pathlib.Path(sample_all_wsis2)),
+            str(pathlib.Path(sample_svs_ndpi_wsis)),
             "--file-types",
-            "*.ndpi, *.svs, *.jp2",
+            "*.ndpi, *.svs",
             "--tile-objective-value",
             "5",
             "--output-path",
@@ -33,7 +33,7 @@ def test_command_line_save_tiles(sample_all_wsis2, tmp_path):
     tmp_path = pathlib.Path(tmp_path)
     cmu_small_region = tmp_path / "all_tiles" / "CMU-1-Small-Region.svs"
     bioformatspull2759 = tmp_path / "all_tiles" / "bioformatspull2759.ndpi"
-    test1jp2 = tmp_path / "all_tiles" / "test1.jp2"
+
     assert save_tiles_result.exit_code == 0
     assert (cmu_small_region / "Output.csv").exists()
     assert (cmu_small_region / "slide_thumbnail.jpg").exists()
@@ -42,9 +42,6 @@ def test_command_line_save_tiles(sample_all_wsis2, tmp_path):
     assert (bioformatspull2759 / "Output.csv").exists()
     assert (bioformatspull2759 / "slide_thumbnail.jpg").exists()
     assert (bioformatspull2759 / "Tile_5_0_0.jpg").exists()
-    assert (test1jp2 / "Output.csv").exists()
-    assert (test1jp2 / "slide_thumbnail.jpg").exists()
-    assert (test1jp2 / "Tile_5_0_0.jpg").exists()
 
 
 def test_command_line_save_tiles_single_file(sample_svs, tmp_path):