-
Notifications
You must be signed in to change notification settings - Fork 104
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: publishing mirtest-internal to provide access to all mock/stub files for testing #3333
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3333 +/- ##
========================================
Coverage 77.52% 77.53%
========================================
Files 1065 1075 +10
Lines 67869 68268 +399
========================================
+ Hits 52618 52931 +313
- Misses 15251 15337 +86 ☔ View full report in Codecov by Sentry. |
ae3dc53
to
948f24a
Compare
475c9a0
to
6e9cea3
Compare
9feda44
to
3c3e77d
Compare
@mattkae it is now unclear what happened to fail the test builds: Should we revive this PR or park it? |
3c3e77d
to
2462ccd
Compare
@AlanGriffiths Sorry for getting back so late on this one! Ideally I'd like to revive it. This provided access to testing utilities, which was very useful |
OK, I fixed the syntax error. Now need to check the content is actually useful |
It would be nice to include |
…cessary include in the test harness
Done! It is exposed as an internal header for now |
fixes #3283