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

[PRDP-318] feat: Setup integration test for transaction APIs #31

Merged

Conversation

gioelemella
Copy link
Collaborator

@gioelemella gioelemella commented Jan 16, 2024

List of Changes

  • added integration test for API /transactions
  • added integration test for API /transactions/{id}
  • added COSMOS_DB_CONN_STRING secret to identity

Apply identity before merge

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@gioelemella gioelemella force-pushed the PRDP-318-integ-test-upg-integ-test-biz-events-service branch from ac77f6b to 872d8e3 Compare January 18, 2024 14:19
Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@gioelemella gioelemella marked this pull request as ready for review January 18, 2024 14:45
@gioelemella gioelemella requested a review from a team as a code owner January 18, 2024 14:45
@pasqualespica pasqualespica merged commit 931dceb into main Jan 21, 2024
12 of 16 checks passed
@pasqualespica pasqualespica deleted the PRDP-318-integ-test-upg-integ-test-biz-events-service branch January 21, 2024 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants