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

feat: Client tests using Example app #50

Merged
merged 13 commits into from
Dec 19, 2024

Conversation

dstenroejl
Copy link
Contributor

@dstenroejl dstenroejl commented Dec 18, 2024

Description

  • Classes that we will refactor/move in another PR has been renamed and prefixed with Obsolete.
  • Added DisableTestParallelization to disable test parallelization because we have to set of apps (Process Manager + Orchestrations; Process Manager + Example Orchestrations) and their tests cannot run in parallel.
  • Implemented tests in ProcessManager.Client.Tests that uses the Example app to test the framework through the Client.
  • Implemented tests in Example.Orchestrations.Tests that uses the Example app to test the framework through the Api.

References

Link to assignment: https://app.zenhub.com/workspaces/mosaic-60a6105157304f00119be86e/issues/gh/energinet-datahub/team-mosaic/424

Checklist

  • Should the change be behind a feature flag?
  • Can the feature be meaningfully disabled or circumvented if there are issues (e.g., database-breaking changes)?
  • Has it been considered whether data is being delivered to the wrong actor?
  • Subsystem test executed (dev_002/dev_003)
  • Is there time to monitor state of the release to Production?
  • Reference to the task

Copy link

github-actions bot commented Dec 18, 2024

Test results for \source\ProcessManager.Client.Tests\bin\Release\net8.0\Energinet.DataHub.ProcessManager.Client.Tests.dll

10 tests   10 ✅  3m 6s ⏱️
 1 suites   0 💤
 1 files     0 ❌

Results for commit c37ee92.

♻️ This comment has been updated with latest results.

@dstenroejl dstenroejl merged commit 85bb40d into main Dec 19, 2024
24 checks passed
@dstenroejl dstenroejl deleted the dstenroejl/client-tests-using-example-app branch December 19, 2024 09:11
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.

3 participants