Skip to content

Add db-sync config tests #2975

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.

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
merged 6 commits into from
May 15, 2025
Merged

Add db-sync config tests #2975

merged 6 commits into from
May 15, 2025

Conversation

ArturWieczorek
Copy link
Collaborator

No description provided.

@ArturWieczorek ArturWieczorek force-pushed the artur/db-sync-config-test branch 5 times, most recently from ab9e9e4 to 03963b3 Compare April 4, 2025 10:50
@ArturWieczorek ArturWieczorek force-pushed the artur/db-sync-config-test branch 5 times, most recently from c0ec45d to bb4cab7 Compare April 30, 2025 17:08
@ArturWieczorek ArturWieczorek force-pushed the artur/db-sync-config-test branch 7 times, most recently from 947c35e to a1f7d0d Compare May 8, 2025 10:51
@ArturWieczorek
Copy link
Collaborator Author

@ArturWieczorek ArturWieczorek force-pushed the artur/db-sync-config-test branch 2 times, most recently from 19dc36a to 76b3a5e Compare May 13, 2025 13:03
@ArturWieczorek ArturWieczorek force-pushed the artur/db-sync-config-test branch from 76b3a5e to cdf67fa Compare May 13, 2025 13:08
@ArturWieczorek ArturWieczorek marked this pull request as ready for review May 13, 2025 13:10
@mkoura mkoura force-pushed the artur/db-sync-config-test branch from f979098 to 7073baf Compare May 15, 2025 12:20
Refactor DBSyncConfigBuilder methods to use `tp.Self` for return type
annotations, improving type hinting and chaining.

Update the DBSyncManager to remove pytest dependency and simplify
initialization.

Revise test_dbsync_config.py to use subtests for config scenarios
instead of parametrized tests, and ensure db-sync config is restored
after testing is finished.
@mkoura mkoura force-pushed the artur/db-sync-config-test branch from 7073baf to 2c5f9d7 Compare May 15, 2025 13:44
@mkoura mkoura merged commit bed0a2b into master May 15, 2025
3 checks passed
@mkoura mkoura deleted the artur/db-sync-config-test branch May 15, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants