Skip to content

Tests updated to be more independent and test multiple files #341

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
michael-richey merged 16 commits into from
Apr 17, 2025
Merged

Tests updated to be more independent and test multiple files #341

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
26bb6bd
Split resource files into multiple files
michael-richey Apr 7, 2025
aea66ff
Added multiple files to aws path
michael-richey Apr 8, 2025
feb7c01
Some general fixes for the filenames and passing in the boolean
michael-richey Apr 9, 2025
0ada04e
First pass at tests
michael-richey Apr 9, 2025
68f19ee
New tests and reworking some old tests
michael-richey Apr 15, 2025
613c292
Remove unused const
michael-richey Apr 15, 2025
cce2ffd
pylint cleanup and black
michael-richey Apr 15, 2025
7647a71
Tests are independent and test multiple files
michael-richey Apr 16, 2025
8066927
add cassette files
michael-richey Apr 16, 2025
1d61428
remove metrics from tests because of timestamps
michael-richey Apr 17, 2025
15847d8
Add dependencies to synth priv locations
michael-richey Apr 17, 2025
eedf884
Fix for windows filenames
michael-richey Apr 17, 2025
14424e4
Merge branch 'michael.richey/multiple-files' into michael.richey/mult…
michael-richey Apr 17, 2025
175d540
Fix for windows file names
michael-richey Apr 17, 2025
73af1f1
ruff
michael-richey Apr 17, 2025
1354ed1
Merge branch 'main' into michael.richey/multiple-files-tests
michael-richey Apr 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
2 changes: 1 addition & 1 deletion datadog_sync/model/logs_pipelines_order.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ async def import_resource(self, _id: Optional[str] = None, resource: Optional[Di
return self.default_id, resource

async def pre_resource_action_hook(self, _id, resource: Dict) -> None:
pass
self.destination_pipeline_order = await self.get_destination_pipeline_order()

async def pre_apply_hook(self) -> None:
self.destination_pipeline_order = await self.get_destination_pipeline_order()
Expand Down
1 change: 1 addition & 0 deletions datadog_sync/model/roles.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,7 @@ async def pre_apply_hook(self) -> None:
self.destination_roles_mapping = await self.get_destination_roles_mapping()

async def pre_resource_action_hook(self, _id, resource: Dict) -> None:
self.destination_roles_mapping = await self.get_destination_roles_mapping()
await self.remap_permissions(resource)

async def create_resource(self, _id, resource) -> Tuple[str, Dict]:
Expand Down
2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_cleanup.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:35:08.970133-04:00
2025-04-17T08:55:29.573654-04:00
6,389 changes: 5,647 additions & 742 deletions tests/integration/cassettes/test_cli/TestCli.test_cleanup.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_import.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:28.161923-04:00
2025-04-17T08:54:46.354171-04:00
9,737 changes: 4,518 additions & 5,219 deletions tests/integration/cassettes/test_cli/TestCli.test_import.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_import_without_verify_ddr_status.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:43.531283-04:00
2025-04-17T08:55:03.355404-04:00
7,573 changes: 5,040 additions & 2,533 deletions tests/integration/cassettes/test_cli/TestCli.test_import_without_verify_ddr_status.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_migrate.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:59.801765-04:00
2025-04-17T08:55:20.364845-04:00
32,824 changes: 21,179 additions & 11,645 deletions tests/integration/cassettes/test_cli/TestCli.test_migrate.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_migrate_without_verify_ddr_status.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:51.622783-04:00
2025-04-17T08:55:11.788313-04:00
17,392 changes: 13,495 additions & 3,897 deletions tests/integration/cassettes/test_cli/TestCli.test_migrate_without_verify_ddr_status.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_sync.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:33.016583-04:00
2025-04-17T08:54:52.077281-04:00
14,918 changes: 12,171 additions & 2,747 deletions tests/integration/cassettes/test_cli/TestCli.test_sync.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_sync_without_verify_ddr_status.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-26T14:34:45.071643-04:00
2025-04-17T08:55:05.609728-04:00
13,318 changes: 11,409 additions & 1,909 deletions tests/integration/cassettes/test_cli/TestCli.test_sync_without_verify_ddr_status.yaml
View file
Open in desktop

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion tests/integration/cassettes/test_cli/TestCli.test_verify_ddr_status_failure.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1 +1 @@
2025-03-20T10:50:46.693087-04:00
2025-04-17T08:55:01.212612-04:00
1 change: 1 addition & 0 deletions ...st_resource_per_file/TestResourcePerFile.test_combined_files_not_created_with_flag.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-09T14:15:50.455469-04:00
1,442 changes: 1,442 additions & 0 deletions ...test_resource_per_file/TestResourcePerFile.test_combined_files_not_created_with_flag.yaml
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions ...ttes/test_resource_per_file/TestResourcePerFile.test_import_with_resource_per_file.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-15T13:20:48.757766-04:00
722 changes: 722 additions & 0 deletions ...settes/test_resource_per_file/TestResourcePerFile.test_import_with_resource_per_file.yaml
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions ...tes/test_resource_per_file/TestResourcePerFile.test_migrate_with_resource_per_file.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-15T13:20:53.968358-04:00
1 change: 1 addition & 0 deletions .../cassettes/test_resource_per_file/TestResourcePerFile.test_reading_combined_format.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-15T13:16:17.617666-04:00
23,242 changes: 23,242 additions & 0 deletions ...on/cassettes/test_resource_per_file/TestResourcePerFile.test_reading_combined_format.yaml
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions ...gration/cassettes/test_resource_per_file/TestResourcePerFile.test_resource_cleanup.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-15T13:17:22.137208-04:00
1 change: 1 addition & 0 deletions ..._per_file/TestResourcePerFile.test_resource_per_file_writes_after_reading_combined.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-09T14:15:53.469787-04:00
1,442 changes: 1,442 additions & 0 deletions ...ce_per_file/TestResourcePerFile.test_resource_per_file_writes_after_reading_combined.yaml
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions ...settes/test_resource_per_file/TestResourcePerFile.test_sync_with_resource_per_file.frozen
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
2025-04-15T13:20:51.012276-04:00
807 changes: 807 additions & 0 deletions ...assettes/test_resource_per_file/TestResourcePerFile.test_sync_with_resource_per_file.yaml
View file
Open in desktop

Large diffs are not rendered by default.

Loading