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

feat: Spawn electrs and esplora in fedimint-bin-tests #2125

Merged
merged 7 commits into from
Apr 5, 2023

Conversation

gotlougit
Copy link
Contributor

This is done to partially address #2101 issue 5, and will help migrate the remaining portions of Fedimint's testing infra to Rust away from Bash scripts.

@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.17 ⚠️

Comparison is base (ca89441) 60.46% compared to head (a6afd20) 60.29%.

❗ Current head a6afd20 differs from pull request most recent head 247ad27. Consider uploading reports for the commit 247ad27 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2125       /-   ##
==========================================
- Coverage   60.46%   60.29%   -0.17%     
==========================================
  Files         152      152              
  Lines       30873    30913       40     
==========================================
- Hits        18666    18638      -28     
- Misses      12207    12275       68     
Impacted Files Coverage Δ
fedimint-bin-tests/src/main.rs 0.21% <0.00%> (-0.02%) ⬇️
fedimint-testing/src/bin/fixtures.rs 0.24% <0.00%> (ø)

... and 12 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@maan2003 maan2003 marked this pull request as ready for review April 5, 2023 11:15
maan2003
maan2003 previously approved these changes Apr 5, 2023
Copy link
Member

@maan2003 maan2003 left a comment

Choose a reason for hiding this comment

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

LGTM

@maan2003
Copy link
Member

maan2003 commented Apr 5, 2023

Feel free to do esplora as well. It should be very similar.

@gotlougit gotlougit changed the title feat: Spawn electrs in fedimint-bin-tests feat: Spawn electrs and esplora in fedimint-bin-tests Apr 5, 2023
@gotlougit
Copy link
Contributor Author

Now we spawn esplora in fedimint-bin-tests too, so this addresses subissue 4 in #2101

Copy link
Member

@maan2003 maan2003 left a comment

Choose a reason for hiding this comment

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

Thanks!

@maan2003 maan2003 enabled auto-merge (squash) April 5, 2023 14:05
@maan2003 maan2003 merged commit 6524668 into fedimint:master Apr 5, 2023
ayush-ranjan-official pushed a commit to ayush-ranjan-official/fedimint that referenced this pull request Apr 5, 2023
@justinmoon
Copy link
Contributor

Great PR @gotlougit!

GetPsyched pushed a commit to GetPsyched/fedimint that referenced this pull request Apr 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants