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

ctxc/gasprice/gasprice test #671

Merged
merged 4 commits into from
Jul 10, 2020
Merged

ctxc/gasprice/gasprice test #671

merged 4 commits into from
Jul 10, 2020

Conversation

ShadowErii
Copy link
Contributor

add test in ctxc

@ucwong ucwong requested review from ucwong and DhunterAO July 9, 2020 09:37
@ShadowErii
Copy link
Contributor Author

ShadowErii commented Jul 9, 2020

the interface ctxcapi.Backend has too much method,but it seems that most of them are not impact on getting the suggest gas price

@DhunterAO
Copy link
Contributor

Creating a new OracleBackend interface can solve this confluct

@codecov
Copy link

codecov bot commented Jul 9, 2020

Codecov Report

Merging #671 into master will decrease coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #671      +/-   ##
==========================================
- Coverage   50.88%   50.87%   -0.01%     
==========================================
  Files         407      408       +1     
  Lines       51815    51907      +92     
==========================================
+ Hits        26364    26409      +45     
- Misses      23427    23466      +39     
- Partials     2024     2032       +8     
Impacted Files Coverage Δ
ctxc/gasprice/gasprice.go 38.04% <100.00%> (ø)
p2p/discover/table.go 83.01% <0.00%> (-3.15%) ⬇️
trie/proof.go 74.62% <0.00%> (-1.12%) ⬇️
trie/trie.go 83.40% <0.00%> (-0.86%) ⬇️
p2p/discover/v5_udp.go 72.85% <0.00%> (-0.47%) ⬇️
p2p/server.go 67.36% <0.00%> (+0.38%) ⬆️
p2p/simulations/http.go 70.10% <0.00%> (+0.54%) ⬆️
ctxc/downloader/downloader.go 78.97% <0.00%> (+0.54%) ⬆️
rpc/client.go 81.69% <0.00%> (+0.70%) ⬆️
... and 3 more

@ucwong ucwong added the unit test about unit test label Jul 9, 2020
@ucwong ucwong force-pushed the ctxc-test branch 2 times, most recently from cd7f293 to 588f391 Compare July 9, 2020 11:35
Copy link
Contributor

@DhunterAO DhunterAO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ucwong ucwong merged commit 7a064de into master Jul 10, 2020
@ShadowErii ShadowErii deleted the ctxc-test branch July 10, 2020 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
unit test about unit test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants