From 4018221da6af6f2767729e7f9f9d96e517791d99 Mon Sep 17 00:00:00 2001 From: Pepe Fagoaga Date: Tue, 5 Dec 2023 10:59:09 +0100 Subject: [PATCH] test(audit_info): refactor drs (#3127) --- .../aws/services/drs/drs_service_test.py | 59 +++++-------------- 1 file changed, 14 insertions(+), 45 deletions(-) diff --git a/tests/providers/aws/services/drs/drs_service_test.py b/tests/providers/aws/services/drs/drs_service_test.py index 1c85ce71..4cf35e61 100644 --- a/tests/providers/aws/services/drs/drs_service_test.py +++ b/tests/providers/aws/services/drs/drs_service_test.py @@ -2,14 +2,12 @@ from datetime import datetime from unittest.mock import patch import botocore -from boto3 import session -from prowler.providers.aws.lib.audit_info.audit_info import AWS_Audit_Info from prowler.providers.aws.services.drs.drs_service import DRS -from prowler.providers.common.models import Audit_Metadata - -# Mock Test Region -AWS_REGION = "us-east-1" +from tests.providers.aws.audit_info_utils import ( + AWS_REGION_US_EAST_1, + set_mocked_aws_audit_info, +) # Mocking Calls make_api_call = botocore.client.BaseClient._make_api_call @@ -44,9 +42,11 @@ def mock_make_api_call(self, operation_name, kwargs): def mock_generate_regional_clients(service, audit_info, _): - regional_client = audit_info.audit_session.client(service, region_name=AWS_REGION) - regional_client.region = AWS_REGION - return {AWS_REGION: regional_client} + regional_client = audit_info.audit_session.client( + service, region_name=AWS_REGION_US_EAST_1 + ) + regional_client.region = AWS_REGION_US_EAST_1 + return {AWS_REGION_US_EAST_1: regional_client} # Patch every AWS call using Boto3 and generate_regional_clients to have 1 client @@ -56,51 +56,20 @@ def mock_generate_regional_clients(service, audit_info, _): new=mock_generate_regional_clients, ) class Test_DRS_Service: - # Mocked Audit Info - def set_mocked_audit_info(self): - audit_info = AWS_Audit_Info( - session_config=None, - original_session=None, - audit_session=session.Session( - profile_name=None, - botocore_session=None, - ), - audited_account=None, - audited_account_arn=None, - audited_user_id=None, - audited_partition="aws", - audited_identity_arn=None, - profile=None, - profile_region=None, - credentials=None, - assumed_role_info=None, - audited_regions=None, - organizations_metadata=None, - audit_resources=None, - mfa_enabled=False, - audit_metadata=Audit_Metadata( - services_scanned=0, - expected_checks=[], - completed_checks=0, - audit_progress=0, - ), - ) - return audit_info - def test__get_client__(self): - audit_info = self.set_mocked_audit_info() + audit_info = set_mocked_aws_audit_info() drs = DRS(audit_info) - assert drs.regional_clients[AWS_REGION].__class__.__name__ == "drs" + assert drs.regional_clients[AWS_REGION_US_EAST_1].__class__.__name__ == "drs" def test__get_service__(self): - audit_info = self.set_mocked_audit_info() + audit_info = set_mocked_aws_audit_info() drs = DRS(audit_info) assert drs.service == "drs" def test__describe_jobs__(self): - audit_info = self.set_mocked_audit_info() + audit_info = set_mocked_aws_audit_info() drs = DRS(audit_info) assert len(drs.drs_services) == 1 assert drs.drs_services[0].id == "DRS" - assert drs.drs_services[0].region == AWS_REGION + assert drs.drs_services[0].region == AWS_REGION_US_EAST_1 assert drs.drs_services[0].status == "ENABLED"