Skip to content

[pull] master from bitcoinj:master#105

Merged
pull[bot] merged 11 commits intoAll-Blockchains:masterfrom
bitcoinj:master
Mar 25, 2026
Merged

[pull] master from bitcoinj:master#105
pull[bot] merged 11 commits intoAll-Blockchains:masterfrom
bitcoinj:master

Conversation

@pull
Copy link
Copy Markdown

@pull pull bot commented Mar 25, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

…b.test.integration.peer`, source path `main`

These integration test support classes are not tests and do not use anything
from JUnit. We created the `o.b.test.integration` package for integration tests
with the intention of moving integration tests into this package hierarchy.

Moving these support class to the `main` source path for `o.b.test.integration.peer`

* helps eliminate a split-package issue (`o.b.testing` is owned by the core module)
* allows the package-info to be defined in the `main` source path
* allows further development of these classes to be more general purpose
* allows these support classes to have their own tests

Also adds package-info to the new package `o.b.test.integration.peer`.
`TestWithPeerGroup` is a base class for implementing other tests and
does not contain any tests itself. Mark it as `abstract` for clarity.
This randomization does not prevent the "address already in use" errors
we have been seeing. It only randomizes the value upon class loading,
so it is not randomized between tests. Since it is not necessary or
helpful we remove it.
Add accessor `lastReceivedFilter()` and use it.
Use JUnit 6's support for constructor injection of `@TempDir` to
initialize `walletFile` in the constructor.
@pull pull bot locked and limited conversation to collaborators Mar 25, 2026
@pull pull bot added the ⤵️ pull label Mar 25, 2026
@pull pull bot merged commit dd54cce into All-Blockchains:master Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant