Skip to content

Conversation

@sethdusek
Copy link
Collaborator

@sethdusek sethdusek commented Nov 24, 2025

CI tests were failing because of a bunch of reasons:

Additionally this PR moves to nextest, which should help reduce CI failures because of a few flaky tests (mostly ones that do network calls), and as an added bonus makes tests & coverage run significantly faster

@coveralls
Copy link

coveralls commented Nov 24, 2025

Pull Request Test Coverage Report for Build 19909456309

Details

  • 20 of 24 (83.33%) changed or added relevant lines in 12 files are covered.
  • 1614 unchanged lines in 221 files lost coverage.
  • Overall coverage increased (+8.5%) to 86.947%

Changes Missing Coverage Covered Lines Changed/Added Lines %
ergo-rest/src/api.rs 7 8 87.5%
ergotree-ir/src/source_span.rs 0 1 0.0%
ergo-rest/src/api/node.rs 2 4 50.0%
Files with Coverage Reduction New Missed Lines %
ergo-chain-types/src/autolykos_pow_scheme.rs 1 99.02%
ergo-chain-types/src/block_id.rs 1 57.14%
ergo-chain-types/src/json/autolykos_solution.rs 1 91.67%
ergo-lib/src/chain/json/transaction.rs 1 96.4%
ergo-p2p/src/codec.rs 1 0.0%
ergo-p2p/src/peer_database/in_memory.rs 1 95.0%
ergo-p2p/src/peer_feature.rs 1 96.49%
ergo-rest/src/api/peer_discovery_internals/non_chrome.rs 1 97.53%
ergo-rest/src/node_info.rs 1 0.0%
ergo-rest/src/peer_info.rs 1 86.11%
Totals Coverage Status
Change from base Build 16925157956: 8.5%
Covered Lines: 27478
Relevant Lines: 31603

💛 - Coveralls

@sethdusek sethdusek force-pushed the cifixes branch 12 times, most recently from 07d5de5 to 45f5906 Compare November 25, 2025 07:32
@sethdusek sethdusek force-pushed the cifixes branch 3 times, most recently from 7412b0e to ec73e60 Compare November 27, 2025 20:07
@sethdusek sethdusek force-pushed the cifixes branch 2 times, most recently from 3c77843 to a4e6cfe Compare November 27, 2025 21:03
@sethdusek sethdusek force-pushed the cifixes branch 6 times, most recently from 279e1d7 to 0928c8a Compare November 29, 2025 21:49
@sethdusek sethdusek changed the title CI fixes Fix CI, bump dependencies and rust toolchain Dec 2, 2025
The node we were using before seems to have some sort of aggressive rate-limiting, and starts rejecting connections after only a few requests. Try using a different node for now
@sethdusek sethdusek requested a review from kushti December 4, 2025 04:40
@sethdusek sethdusek marked this pull request as ready for review December 4, 2025 04:40
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.

2 participants