-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
chore: improved proptest logging #32884
Conversation
For the Aspect property tests, make some changes to make it easier to see what's going on: - Instead of just a visit log, keep a log of all actions: adding constructs, adding aspects, and visiting are now all part of the same time sequence with time steps so it's easier to reason about. - Clearly mark timed steps with `t=0`, `t=1`, etc.
ba94b40
to
530a018
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #32884 +/- ##
=======================================
Coverage 81.38% 81.38%
=======================================
Files 222 222
Lines 13698 13698
Branches 2413 2413
=======================================
Hits 11148 11148
Misses 2271 2271
Partials 279 279
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
AWS CodeBuild CI Report
Powered by github-codebuild-logs, available on the AWS Serverless Application Repository |
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
Comments on closed issues and PRs are hard for our team to see. |
For the Aspect property tests, make some changes to make it easier to see what's going on:
t=0
,t=1
, etc.FAST_CHECK_NUM_RUNS
, so we can run a burn-in test to be confident that the tests won't generate any problems during CI builds.readonly
.By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license