]> Untitled Git - bdk/commit
Merge bitcoindevkit/bdk#2054: fix(ci,test): update `chained_mempool_txs` to use 6...
authormerge-script <hello@evanlinjin.me>
Thu, 2 Oct 2025 05:34:52 +0000 (05:34 +0000)
committermerge-script <hello@evanlinjin.me>
Thu, 2 Oct 2025 05:34:52 +0000 (05:34 +0000)
commit24654a51bd0a82c74c4cb0dff74e5f94c34d01ee
treef1af983c551dfbfb9de3919b27843e4f479b9fb2
parent00efcdee5d4d2c5473b15da358aec474327d4d73
parent353d18bd69b9bfe0d70cf8e0658a98dabf44bb1f
Merge bitcoindevkit/bdk#2054: fix(ci,test): update `chained_mempool_txs` to use 6 secs timeout

353d18bd69b9bfe0d70cf8e0658a98dabf44bb1f fix(ci,test): update `chained_mempool_txs` to use 6 secs timeout (Leonardo Lima)

Pull request description:

  fixes #2045

  ### Description

  It's an initial attempt to solve the timeout failures we have been facing on CI when running electrum tests. It it's not solved at all, at least it now adds the total time elapsed so we can know how long it's taking in CI.

  The PR updates the `chained_mempool_txs` test to use 6 seconds as timeout, instead of 5, it was the only spot using 5 instead of 6. Also updates the error message to show the total time elapsed, can help debug the CI if the timeout error persists.

  ### Notes to the reviewers

  ### Changelog notice

  ```
  ### Changed
  - Update the `chained_mempool_txs` to use 6 seconds as timeout.
  - Update the error message to show the time elapsed.
  ```

  ### Checklists

  #### All Submissions:

  * [x] I followed the [contribution guidelines](https://github.com/bitcoindevkit/bdk/blob/master/CONTRIBUTING.md)

ACKs for top commit:
  evanlinjin:
    ACK 353d18bd69b9bfe0d70cf8e0658a98dabf44bb1f

Tree-SHA512: 62edc397ca74dd8b3a90eb300ea8d52772b3089698d8dad481bf2f9e4ef394ab2a826abdfa7f5d177e675a03bb83ac56445273dbc1d35885ff78651ec9bfc5aa