Skip to content
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

Add prod extern sync infrastructure #1672

Merged
merged 24 commits into from
Nov 21, 2024
Merged

Conversation

flenny
Copy link
Contributor

@flenny flenny commented Nov 19, 2024

- Adds a task manager which executes a given list of tasks in sequence.
- Adds a sync base class to execute and validate tasks.
- Adds a sync context containing a source and a target database context.
to verify and test the CI/CD pipelines
- Adds some unit tests for the sync task manager.
- Adds a test sync context with in-memory or real database contexts.
in order to be able to inspect source and target database.
to improve performance.

Ref.: Sonar csharpsquid:S6603
into more suitable files.
using Directory.Build.props
@flenny flenny requested a review from danjov November 19, 2024 14:27
src/extern-sync/Program.cs Outdated Show resolved Hide resolved
src/extern-sync/SyncContextExtensions.cs Outdated Show resolved Hide resolved
src/extern-sync/SyncContextExtensions.cs Outdated Show resolved Hide resolved
tests/extern-sync/BDMS.ExternSync.Test.csproj Outdated Show resolved Hide resolved
tests/extern-sync/SyncContext.cs Outdated Show resolved Hide resolved
tests/extern-sync/BDMS.ExternSync.Test.csproj Outdated Show resolved Hide resolved
@flenny flenny requested a review from danjov November 21, 2024 10:44
tests/Directory.Build.props Outdated Show resolved Hide resolved
Copy link

sonarcloud bot commented Nov 21, 2024

@flenny flenny merged commit e9c83aa into main Nov 21, 2024
15 checks passed
@flenny flenny deleted the add-prod-extern-sync-infrastructure branch November 21, 2024 14:38
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