bitcoindevkit / bdk / 18184616528 / 1

79%

master: 79%

push

github

Merge bitcoindevkit/bdk#2054: fix(ci,test): update `chained_mempool_txs` to use 6 secs timeout

<a class=hub.com/bitcoindevkit/bdk/commit/353d18bd69b9bfe0d70cf8e0658a98dabf44bb1f">353d18bd6 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: 62edc397c98d8dad487e675a03bc9bfc5aa

378 of 512 branches covered (73.83%)

Branch coverage included in aggregate %.

4203 of 5312 relevant lines covered (79.12%)

3296.18 hits per line