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

Run blockbeat PRs as a whole #16

Open
wants to merge 178 commits into
base: master
Choose a base branch
from
Open

Run blockbeat PRs as a whole #16

wants to merge 178 commits into from

Conversation

yyforyongyu
Copy link
Owner

No description provided.

Copy link

Pull reviewers stats

Stats of the last 30 days for lnd:

User Total reviews Time to review Total comments

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11506222681

Details

  • 1924 of 2304 (83.51%) changed or added relevant lines in 37 files are covered.
  • 26166 unchanged lines in 437 files lost coverage.
  • Overall coverage decreased (-9.4%) to 49.3%

Changes Missing Coverage Covered Lines Changed/Added Lines %
chainio/beat_consumer.go 32 33 96.97%
chainntnfs/best_block_view.go 0 1 0.0%
chainntnfs/btcdnotify/btcd.go 15 16 93.75%
contractcourt/breach_resolver.go 19 20 95.0%
chainio/log.go 6 9 66.67%
contractcourt/htlc_outgoing_contest_resolver.go 30 34 88.24%
sweep/fee_bumper.go 130 134 97.01%
sweep/sweeper.go 117 123 95.12%
contractcourt/anchor_resolver.go 50 57 87.72%
contractcourt/chain_arbitrator.go 72 84 85.71%
Files with Coverage Reduction New Missed Lines %
graph/errors.go 1 94.74%
contractcourt/anchor_resolver.go 1 88.89%
lnwire/typed_fee.go 2 66.67%
htlcswitch/linkfailure.go 2 58.62%
channeldb/forwarding_policy.go 2 85.14%
record/hop.go 2 93.33%
lnrpc/invoicesrpc/invoices_server.go 2 74.55%
lnrpc/invoicesrpc/invoices_grpc.pb.go 2 41.04%
lnwire/short_channel_id.go 2 89.74%
lnwallet/sigpool.go 2 65.63%
Totals Coverage Status
Change from base Build 11484896457: -9.4%
Covered Lines: 98355
Relevant Lines: 199502

💛 - Coveralls

@yyforyongyu yyforyongyu force-pushed the shuffle-tests branch 27 times, most recently from 28c2fe6 to bf47085 Compare November 3, 2024 11:59
So we won't forget to assert the topology after opening a chain of
channels.
This is no longer needed since we don't have standby nodes, plus it's
causing panic in windows build due to `edge.Policy` being nil.
There's no need to mine 80ish blocks here.
Also removed the duplicate test cases.
To make the CI indicative, we now starting tracking the flaky tests
found when running on Windows. As a starting point, rather than ignore
the windows CI entirely, we now identify there are cases where lnd can
be buggy when running in windows.

We should fix the tests in the future, otherwise the windows build
should be deleted.
x - delete - print blocks

x

x

pri
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.

8 participants