Remove cleanup now handled by the rig

Author: pjbull

tests/test_s3_specific.py

@@ -56,37 +56,31 @@ def _download_with_threads(s3_rig, tmp_path, use_threads):
     """Job used by tests to ensure Transfer config changes are
     actually passed through to boto3 and respected.
     """
-    try:
-        sleep(1)  # give test monitoring process time to start watching
-
-        transfer_config = TransferConfig(
-            max_concurrency=100,
-            use_threads=use_threads,
-            multipart_chunksize=1 * 1024,
-            multipart_threshold=10 * 1024,
-        )
-        client = s3_rig.client_class(boto3_transfer_config=transfer_config)
-        p = client.CloudPath(f"s3://{s3_rig.drive}/{s3_rig.test_dir}/dir_0/file0_to_download.txt")
+    sleep(1)  # give test monitoring process time to start watching
+
+    transfer_config = TransferConfig(
+        max_concurrency=100,
+        use_threads=use_threads,
+        multipart_chunksize=1 * 1024,
+        multipart_threshold=10 * 1024,
+    )
+    client = s3_rig.client_class(boto3_transfer_config=transfer_config)
+    p = client.CloudPath(f"s3://{s3_rig.drive}/{s3_rig.test_dir}/dir_0/file0_to_download.txt")
 
-        assert not p.exists()
+    assert not p.exists()
 
-        # file should be about 60KB
-        text = "lalala" * 10_000
-        p.write_text(text)
+    # file should be about 60KB
+    text = "lalala" * 10_000
+    p.write_text(text)
 
-        assert p.exists()
-
-        # assert not (dl_dir / p.name).exists()
-        p.download_to(tmp_path)
+    assert p.exists()
 
-        p.unlink()
+    # assert not (dl_dir / p.name).exists()
+    p.download_to(tmp_path)
 
-        assert not p.exists()
+    p.unlink()
 
-    finally:
-        p = s3_rig.create_cloud_path("dir_0/file0_0.txt")
-        if p.exists():
-            p.unlink()
+    assert not p.exists()
 
 
 def test_transfer_config_live(s3_rig, tmp_path):