add test and lint

TingDaoK · TingDaoK · commit 3880d0364a63 · 2025-02-26T10:30:16.000-08:00
diff --git a/s3transfer/crt.py b/s3transfer/crt.py
@@ -901,7 +901,7 @@ def _default_get_make_request_args(
                 algorithm=AwsSigningAlgorithm.V4_ASYMMETRIC,
                 region="*",
                 use_double_uri_encode=False,
-                should_normalize_uri_path=False
+                should_normalize_uri_path=False,
             )
             call_args.bucket = accesspoint_arn_details['resource_name']
         elif is_s3express_bucket(call_args.bucket):
@@ -910,7 +910,7 @@ def _default_get_make_request_args(
             make_request_args['signing_config'] = AwsSigningConfig(
                 algorithm=AwsSigningAlgorithm.V4_S3EXPRESS,
                 use_double_uri_encode=False,
-                should_normalize_uri_path=False
+                should_normalize_uri_path=False,
             )
         return make_request_args
 
diff --git a/tests/functional/test_crt.py b/tests/functional/test_crt.py
@@ -163,6 +163,12 @@ def _assert_expected_s3express_request(
             make_request_kwargs['signing_config'].algorithm,
             awscrt.auth.AwsSigningAlgorithm.V4_S3EXPRESS,
         )
+        self.assertFalse(
+            make_request_kwargs['signing_config'].use_double_uri_encode,
+        )
+        self.assertFalse(
+            make_request_kwargs['signing_config'].should_normalize_uri_path,
+        )
 
     def _assert_expected_mrap_request(
         self, make_request_kwargs, expected_http_method='GET'
@@ -179,6 +185,12 @@ def _assert_expected_mrap_request(
             awscrt.auth.AwsSigningAlgorithm.V4_ASYMMETRIC,
         )
         self.assertEqual(make_request_kwargs['signing_config'].region, "*")
+        self.assertFalse(
+            make_request_kwargs['signing_config'].use_double_uri_encode,
+        )
+        self.assertFalse(
+            make_request_kwargs['signing_config'].should_normalize_uri_path,
+        )
 
     def _assert_subscribers_called(self, expected_future=None):
         self.assertTrue(self.record_subscriber.on_queued_called)
