Skip to content

eth/client: add test for tx_succeeded?#183

Merged
q9f merged 2 commits intomainfrom
q9f/client/success
Dec 21, 2022
Merged

eth/client: add test for tx_succeeded?#183
q9f merged 2 commits intomainfrom
q9f/client/success

Conversation

@q9f
Copy link
Owner

@q9f q9f commented Dec 21, 2022

after #173

@q9f q9f added the spec-tests label Dec 21, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #183 (55de840) into main (1633ed0) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #183      +/-   ##
==========================================
+ Coverage   99.70%   99.75%   +0.04%     
==========================================
  Files          70       70              
  Lines        4077     4086       +9     
==========================================
+ Hits         4065     4076      +11     
+ Misses         12       10       -2     
Impacted Files Coverage Δ
lib/eth/client.rb 100.00% <100.00%> (+1.25%) ⬆️
spec/eth/client_spec.rb 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@q9f q9f merged commit 1399e47 into main Dec 21, 2022
@q9f q9f deleted the q9f/client/success branch December 21, 2022 16:21
@kurotaky
Copy link
Contributor

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants