From ea283283474b65e551059414f133b7b70e43697a Mon Sep 17 00:00:00 2001 From: Hongjing Chen Date: Thu, 14 Mar 2024 21:48:31 +0000 Subject: [PATCH] fix linting Signed-off-by: Hongjing Chen --- legal-api/tests/unit/models/test_alternate_name.py | 3 +-- legal-api/tests/unit/models/test_legal_entity.py | 10 +++------- legal-api/tests/unit/resources/v2/test_business.py | 1 - .../tests/unit/resources/v2/test_business_aliases.py | 3 ++- 4 files changed, 6 insertions(+), 11 deletions(-) diff --git a/legal-api/tests/unit/models/test_alternate_name.py b/legal-api/tests/unit/models/test_alternate_name.py index 57234bc657..d54ced0482 100644 --- a/legal-api/tests/unit/models/test_alternate_name.py +++ b/legal-api/tests/unit/models/test_alternate_name.py @@ -111,7 +111,7 @@ def test_find_by_name_type(session): with nested_session(session): identifier = "BC1234567" legal_entity = factory_legal_entity(identifier=identifier) - + alternate_name = factory_alternate_name( name="ABC Ltd.", name_type=AlternateName.NameType.DBA, @@ -139,4 +139,3 @@ def test_find_by_name_type(session): assert len(res) == 2 assert res[0].name == "NAME TRANSLATION 1" assert res[1].name == "NAME TRANSLATION 2" - diff --git a/legal-api/tests/unit/models/test_legal_entity.py b/legal-api/tests/unit/models/test_legal_entity.py index 52bb985157..1dd74a8de1 100644 --- a/legal-api/tests/unit/models/test_legal_entity.py +++ b/legal-api/tests/unit/models/test_legal_entity.py @@ -915,15 +915,11 @@ def test_alternate_names(session, test_name, legal_entities_info, alternate_name if alternate_names_info: for alternate_name_info in alternate_names_info: - business_start_date = datetime.strptime( - alternate_name_info["businessStartDate"], "%Y-%m-%dT%H:%M:%S%z" - ) - + business_start_date = datetime.strptime(alternate_name_info["businessStartDate"], "%Y-%m-%dT%H:%M:%S%z") + alternate_name_identifier = alternate_name_info["identifier"] - start_date = datetime.strptime( - alternate_name_info["startDate"], "%Y-%m-%dT%H:%M:%S%z" - ) + start_date = datetime.strptime(alternate_name_info["startDate"], "%Y-%m-%dT%H:%M:%S%z") if alternate_name_identifier != le.identifier: le_alternate_name = LegalEntity( _entity_type=alternate_name_info["entityType"], diff --git a/legal-api/tests/unit/resources/v2/test_business.py b/legal-api/tests/unit/resources/v2/test_business.py index 6244d93efb..01e22d8001 100644 --- a/legal-api/tests/unit/resources/v2/test_business.py +++ b/legal-api/tests/unit/resources/v2/test_business.py @@ -325,7 +325,6 @@ def test_get_business_with_alternate_names(session, client, jwt): assert len(rv.json["business"]["alternateNames"]) == 2 - @pytest.mark.parametrize( "test_name, legal_type, owner_legal_type, identifier, owner_identifier, has_missing_business_info," + "missing_business_info_warning_expected", diff --git a/legal-api/tests/unit/resources/v2/test_business_aliases.py b/legal-api/tests/unit/resources/v2/test_business_aliases.py index ebbf0ff092..0a778598b7 100644 --- a/legal-api/tests/unit/resources/v2/test_business_aliases.py +++ b/legal-api/tests/unit/resources/v2/test_business_aliases.py @@ -157,7 +157,8 @@ def test_get_business_alias_by_type(session, client, jwt): # test rv = client.get( - f"/api/v2/businesses/{identifier}/aliases?type=translation", headers=create_header(jwt, [STAFF_ROLE], identifier) + f"/api/v2/businesses/{identifier}/aliases?type=translation", + headers=create_header(jwt, [STAFF_ROLE], identifier), ) # check assert rv.status_code == HTTPStatus.OK