Skip to content
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

chore(executor): Test Coverage over Executor Utilities #650

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

refcell
Copy link
Collaborator

@refcell refcell commented Oct 7, 2024

Description

Adds test coverage over utilities.

@refcell refcell requested a review from clabby as a code owner October 7, 2024 21:47
@refcell refcell self-assigned this Oct 7, 2024
@refcell refcell marked this pull request as draft October 7, 2024 21:48
@refcell refcell added K-chore Kind: chore M-tests Meta: Testing related A-executor Area: kona-executor crate labels Oct 7, 2024
@refcell refcell marked this pull request as ready for review October 8, 2024 17:26
@refcell
Copy link
Collaborator Author

refcell commented Oct 8, 2024

These utils will be upstreamed into op-alloy, but for now let's have coverage.

Copy link

codecov bot commented Oct 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.3%. Comparing base (4bf4264) to head (f573a0b).
Report is 14 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@clabby clabby added this pull request to the merge queue Oct 8, 2024
Merged via the queue into main with commit 25af581 Oct 8, 2024
17 checks passed
This was referenced Oct 8, 2024
@github-actions github-actions bot mentioned this pull request Oct 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-executor Area: kona-executor crate K-chore Kind: chore M-tests Meta: Testing related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants