- 
                Notifications
    
You must be signed in to change notification settings  - Fork 32
 
🎨 [e2e] Improve MMUX test #8494
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    #8494      +/-   ##
==========================================
- Coverage   87.62%   87.62%   -0.01%     
==========================================
  Files        2001     2001              
  Lines       77905    77905              
  Branches     1338     1338              
==========================================
- Hits        68262    68261       -1     
- Misses       9243     9244       +1     
  Partials      400      400              
 *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 improves the MMUX (metamodeling) end-to-end test by adding better test stability and synchronization mechanisms. The changes focus on making the response surface modeling test more robust by properly waiting for UI state changes.
Key changes:
- Added explicit Enter key press after filling study title to ensure proper form submission
 - Added synchronization logic to wait for project saving completion before proceeding
 - Added test ID to the saving icon UI element to enable reliable test targeting
 
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description | 
|---|---|
| test_response_surface_modeling.py | Enhanced test flow with explicit form submission and project save synchronization | 
| NavigationBar.js | Added test ID to saving icon for improved test element targeting | 
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
          
🧪 CI InsightsHere's what we observed from your CI run for e283733. 🟢 All jobs passed!But CI Insights is watching 👀  | 
    
          
 | 
    



What do these changes do?
This PR improves the metamodeling e2e test by adding better test stability and synchronization mechanisms:
Related issue/s
How to test
Dev-ops