Skip to content

Commit 059bf57

Browse files
NSsirenalukeseawalker
authored andcommitted
Change fixture scope in test_ad_integration
With AZ Override changes, vpc_stack is created with class scope. To avoid scope mismatch issues we are also changing the scope of the fixture in ad_integration. Signed-off-by: Nicola Sirena <[email protected]>
1 parent 1d0f49d commit 059bf57

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/integration-tests/tests/ad_integration/test_ad_integration.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ def zip_dir(path):
142142
return file_out
143143

144144

145-
@pytest.fixture(scope="module")
145+
@pytest.fixture(scope="class")
146146
def store_secret_in_secret_manager(request, cfn_stacks_factory):
147147
secret_arns = {}
148148

@@ -356,7 +356,7 @@ def _delete_certificate(certificate_arn, region):
356356
boto3.client("acm", region_name=region).delete_certificate(CertificateArn=certificate_arn)
357357

358358

359-
@pytest.fixture(scope="module")
359+
@pytest.fixture(scope="class")
360360
def directory_factory(request, cfn_stacks_factory, vpc_stack, store_secret_in_secret_manager): # noqa: C901
361361
# TODO: use external data file and file locking in order to share directories across processes
362362
created_directory_stacks = defaultdict(dict)

0 commit comments

Comments
 (0)