Skip to content

Commit 741cbbe

Browse files
committed
fix(test): remove fake MS config fixture and logic in pipeline test
The actual fetch of the github repo would get no MS config, and the fake would be adding one. There appears to inconsistency in this test passing locally but failing on the server, so let us remove the MS config part.
1 parent ec84896 commit 741cbbe

1 file changed

Lines changed: 1 addition & 8 deletions

File tree

test/e2e/dags/pipelines/test_notice_processor_pipelines.py

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -12,20 +12,13 @@
1212
NOTICE_ID = "057215-2021"
1313

1414

15-
def test_notice_processor_pipelines(fake_mongodb_client, load_mapping_suite_and_package_fake, mapping_suite):
15+
def test_notice_processor_pipelines(fake_mongodb_client):
1616
load_mapping_suite_and_packages_from_github_to_mongo_db(
1717
mapping_package_name=MAPPING_PACKAGE_NAME,
1818
mongodb_client=fake_mongodb_client,
1919
load_test_data=True
2020
)
2121

22-
# Update GitHub-loaded packages to use the local mapping suite
23-
mapping_package_repository = MappingPackageRepositoryMongoDB(mongodb_client=fake_mongodb_client)
24-
for pkg in mapping_package_repository.list():
25-
if not pkg.mapping_suite_identifier:
26-
pkg.mapping_suite_identifier = mapping_suite.id
27-
mapping_package_repository.update(pkg)
28-
2922
notice_id = NOTICE_ID
3023
notice_repository = NoticeRepository(mongodb_client=fake_mongodb_client)
3124
notice = notice_repository.get(reference=notice_id)

0 commit comments

Comments
 (0)