-
Notifications
You must be signed in to change notification settings - Fork 22
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
Feature/226 test enhancements for integration testing #732
Feature/226 test enhancements for integration testing #732
Conversation
d9a64d9
to
4dec8f6
Compare
️✅ There are no secrets present in this pull request anymore.If these secrets were true positive and are still valid, we highly recommend you to revoke them. 🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request. |
sed "s/access_tok.*:s = '.*'//g" tavern-results.xml | sed "s/Authorization: Bearer .*/Authorization:/g" | uniq > result.xml | ||
|
||
- name: Publish Tavern Results | ||
uses: EnricoMi/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA Note
sed "s/access_tok.*:s = '.*'//g" tavern-ess-results.xml | sed "s/Authorization: Bearer .*/Authorization:/g" | uniq > ess-result.xml | ||
|
||
- name: Publish Tavern Results | ||
uses: EnricoMi/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA Note
…mbrella test environment
efae970
to
4a7e4ee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
sed "s/access_tok.*:s = '.*'//g" tavern-ess-results.xml | sed "s/Authorization: Bearer .*/Authorization:/g" | uniq > ess-result.xml | ||
|
||
- name: Publish Tavern Results | ||
uses: EnricoMi/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA Note
sed "s/access_tok.*:s = '.*'//g" tavern-ess-results.xml | sed "s/Authorization: Bearer .*/Authorization:/g" | uniq > ess-result.xml | ||
|
||
- name: Publish Tavern Results | ||
uses: EnricoMi/[email protected] |
Check notice
Code scanning / KICS
Unpinned Actions Full Length Commit SHA Note
Quality Gate passedIssues Measures |
Description
This PR introduces new GitHub workflows for E2E Tests using the Umbrella Helm Chart as well as a GitHub action to setup the umbrella environment which can be reused in other workflows.
Test data needed to be adjusted since the Umbrella setup only allows EDCs to process requests for their actual BPN. So all BPNs in the test data files now point to the same BPN
A cucumber and tavern test workflow was prepared to execute the tests against umbrella environment.
Most tests are passing as you can see in the summaries on my fork, but the ESS cases still needs further configuration to work properly
Tavern: https://github.com/ds-jhartmann/item-relationship-service/actions/runs/9808714509
Cucumber: https://github.com/ds-jhartmann/item-relationship-service/actions/runs/9807918899
The PR also includes a small change in the validation for EDC URLs received by EDC Discovery or Digital Twin Registry, so it is no longer failing on .test and internal URLs.
Pre-review checks
Please ensure to do as many of the following checks as possible, before asking for committer review: