-
Notifications
You must be signed in to change notification settings - Fork 32
🎨 e2e Metamodeling: Improve project creation #8477
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #8477 +/- ##
==========================================
- Coverage 87.63% 87.56% -0.07%
==========================================
Files 1995 1992 -3
Lines 77564 77345 -219
Branches 1333 1333
==========================================
- Hits 67973 67730 -243
- Misses 9192 9216 +24
Partials 399 399
*This pull request uses carry forward flags. Click here to find out more.
Continue to review full report in Codecov by Sentry.
🚀 New features to boost your workflow:
|
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.
Pull Request Overview
This PR enhances the end-to-end testing for response surface modeling by improving HTTP response handling and adding structured logging. The changes focus on making the test more robust with proper response validation and clearer debugging output.
- Refactored HTTP response expectations with proper status code assertions
- Added structured logging with context blocks for better test debugging
- Improved code organization by extracting the project UUID into a variable
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
tests/e2e-playwright/tests/metamodeling/test_response_surface_modeling.py
Outdated
Show resolved
Hide resolved
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.
Thanks @odeimaiz
🧪 CI InsightsHere's what we observed from your CI run for dc8a273. 🟢 All jobs passed!But CI Insights is watching 👀 |
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.
👍
|



What do these changes do?
This PR enhances the e2e MMUX testing by improving HTTP response handling and adding structured logging. The changes focus on making the pipeline creation more robust with proper response validation and clearer debugging output.
Related issue/s
How to test
Dev-ops