-
Notifications
You must be signed in to change notification settings - Fork 57
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
test: refactor mock setup in maven plugin tests #2911
Conversation
Refactoring required by changes in vaadin/flow#20465.
The Flow change will be backported to 24.5 and 24.4, so probably this PR should also be cherry picked. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2911 +/- ##
=======================================
Coverage 92.64% 92.64%
=======================================
Files 83 83
Lines 2842 2842
Branches 739 739
=======================================
Hits 2633 2633
Misses 158 158
Partials 51 51
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Quality Gate passedIssues Measures |
Refactoring required by changes in vaadin/flow#20465. Co-authored-by: Zhe Sun <[email protected]>
Refactoring required by changes in vaadin/flow#20465. Co-authored-by: Zhe Sun <[email protected]>
…2912) test: refactor mock setup in maven plugin tests (#2911) Refactoring required by changes in vaadin/flow#20465. Co-authored-by: Marco Collovati <[email protected]> Co-authored-by: Zhe Sun <[email protected]>
…2913) test: refactor mock setup in maven plugin tests (#2911) Refactoring required by changes in vaadin/flow#20465. Co-authored-by: Marco Collovati <[email protected]> Co-authored-by: Zhe Sun <[email protected]> Co-authored-by: Soroosh Taefi <[email protected]>
Refactoring required by changes in vaadin/flow#20465.