fix

uc-cdis · Jan 10, 2025 · 435bc05 · 435bc05
1 parent 9113999
commit 435bc05
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 11 deletions.
diff --git a/.secrets.baseline b/.secrets.baseline
@@ -314,14 +314,14 @@
         "filename": "tests/data/test_indexed_file.py",
         "hashed_secret": "a62f2225bf70bfaccbc7f1ef2a397836717377de",
         "is_verified": false,
-        "line_number": 467
+        "line_number": 462
       },
       {
         "type": "Secret Keyword",
         "filename": "tests/data/test_indexed_file.py",
         "hashed_secret": "c258a8d1264cc59de81f8b1975ac06732b1cf182",
         "is_verified": false,
-        "line_number": 488
+        "line_number": 483
       }
     ],
     "tests/keys/2018-05-01T21:29:02Z/jwt_private_key.pem": [
@@ -422,5 +422,5 @@
       }
     ]
   },
-  "generated_at": "2025-01-10T17:57:26Z"
+  "generated_at": "2025-01-10T18:59:11Z"
 }
diff --git a/tests/data/test_indexed_file.py b/tests/data/test_indexed_file.py
@@ -401,18 +401,13 @@ def test_get_signed_url_s3_bucket_name(mock_get_value, s3_indexed_file_location,
                     s3_indexed_file_location.parsed_url.netloc
                     == "validbucketname-netloc"
                 ):
-                    assert (
-                        "validbucketname-netloc" in result_url
-                        and "endpoint3" in result_url
-                    )
+                    # this should be in the signed URL since the bucket name with wildcard matches
+                    assert "validbucketname-netloc" in result_url
                 elif (
                     s3_indexed_file_location.parsed_url.netloc
                     == "validbucketname-alreadyvalid"
                 ):
-                    assert (
-                        "validbucketname-alreadyvalid" in result_url
-                        and "endpoint2" in result_url
-                    )
+                    assert "validbucketname-alreadyvalid" in result_url
 
 
 @pytest.mark.parametrize("supported_action", ["download"], indirect=True)