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: Add Enqueue Messages Orchestration #1023

Merged
merged 37 commits into from
May 22, 2024
Merged

Conversation

dstenroejl
Copy link
Contributor

@dstenroejl dstenroejl commented May 17, 2024

Description

⚠️ REVIEWER please ensure the code follow the guidelines specified here: https://energinet.atlassian.net/wiki/spaces/D3/pages/824803345/Durable+Functions+Development+Guidelines

  • Cleanup Directory.Build.props

    • Remove unnecessary "PackageReference Update" listings in project files
    • Cleanup unnecessary usings now that implicit using is enabled
  • Add new test project B2BApi.AppTests

    • Implement fixtures to support app tests
    • Implement health checks tests
    • Implement orchestration tests
      • Feature flag is enabled
      • Feature flag is disabled

References

Part of:
https://app.zenhub.com/workspaces/mosaic-60a6105157304f00119be86e/issues/gh/energinet-datahub/team-mosaic/200

@dstenroejl dstenroejl changed the title feat: Add Enqueue MEssages Orchestration feat: Add Enqueue Messages Orchestration May 17, 2024
Copy link

github-actions bot commented May 17, 2024

Test results for \source\IntegrationTests\bin\Release\net8.0\Energinet.DataHub.EDI.IntegrationTests.dll

  1 files    1 suites   2m 58s ⏱️
828 tests 828 ✅ 0 💤 0 ❌
829 runs  829 ✅ 0 💤 0 ❌

Results for commit 3311c16.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented May 17, 2024

Test results for \source\Tests\bin\Release\net8.0\Energinet.DataHub.EDI.B2BApi.AppTests.dll

3 tests   3 ✅  39s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit ca37f69.

♻️ This comment has been updated with latest results.

@dstenroejl dstenroejl marked this pull request as ready for review May 21, 2024 15:25
@dstenroejl dstenroejl requested review from a team as code owners May 21, 2024 15:25
Renetnielsen
Renetnielsen previously approved these changes May 21, 2024
Renetnielsen
Renetnielsen previously approved these changes May 22, 2024
RasmusGraabaek
RasmusGraabaek previously approved these changes May 22, 2024
@dstenroejl dstenroejl enabled auto-merge (squash) May 22, 2024 08:38
Co-authored-by: Nicolaj Aalykke Hansen <nicolaj.it@icloud.com>
@dstenroejl dstenroejl merged commit 20a837e into main May 22, 2024
21 checks passed
@dstenroejl dstenroejl deleted the dstenroejl/emo-part-two branch May 22, 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
5 participants