Remove redundant CodeSize verify skip

Is already part of the default transformer
localstack · Mar 1, 2023 · 25e9a0f · 25e9a0f
1 parent 6d60f90
commit 25e9a0f
Showing 1 changed file with 18 additions and 23 deletions.
diff --git a/tests/integration/awslambda/test_lambda.py b/tests/integration/awslambda/test_lambda.py
@@ -156,28 +156,24 @@ def fixture_snapshot(snapshot):
 
 
 # some more common ones that usually don't work in the old provider
-if is_old_provider():
-    pytestmark = pytest.mark.skip_snapshot_verify(
-        paths=[
-            "$..Architectures",
-            "$..EphemeralStorage",
-            "$..LastUpdateStatus",
-            "$..MemorySize",
-            "$..State",
-            "$..StateReason",
-            "$..StateReasonCode",
-            "$..VpcConfig",
-            "$..CodeSigningConfig",
-            "$..Environment",  # missing
-            "$..HTTPStatusCode",  # 201 vs 200
-            "$..Layers",
-            "$..SnapStart",
-        ],
-    )
-else:
-    pytestmark = pytest.mark.skip_snapshot_verify(
-        paths=["$..CodeSize"],  # FIXME
-    )
+pytestmark = pytest.mark.skip_snapshot_verify(
+    condition=is_old_provider,
+    paths=[
+        "$..Architectures",
+        "$..EphemeralStorage",
+        "$..LastUpdateStatus",
+        "$..MemorySize",
+        "$..State",
+        "$..StateReason",
+        "$..StateReasonCode",
+        "$..VpcConfig",
+        "$..CodeSigningConfig",
+        "$..Environment",  # missing
+        "$..HTTPStatusCode",  # 201 vs 200
+        "$..Layers",
+        "$..SnapStart",
+    ],
+)
 
 
 class TestLambdaBaseFeatures:
@@ -208,7 +204,6 @@ def test_large_payloads(self, caplog, lambda_client, create_lambda_function, sna
             "$..Tags",
             "$..Configuration.RevisionId",
             "$..Code.RepositoryType",
-            "$..CodeSize",  # CI reports different code size here,
             "$..Layers",  # PRO
             "$..RuntimeVersionConfig",
         ],