Skip to content

Conversation

@qlrd
Copy link

@qlrd qlrd commented Jul 25, 2025

This PR aim to improve a development environment for integration tests while keep it reproductible and well formated.

Basically it will setup a bitcoind and elementsd for testing purposes without touching a pre-installed one.

Notable proposed changes:

  • formatting: update pre-commit to run black 25.1.0;

  • add a run.sh to tests folder: Prepare and run a reproductible environment to integration tests while avoiding conflicts with different versions of embit. This build bitcoind and elementsd to a temporary environment;

  • change integration tests to comply with the tests/run.sh: add a EMBIT_TEMP_DIR to scripts will allow us to compile bitcoind and elementsd with a proper configuration as well separated from user's system;

  • we can have diffent binaries for each different versions of embit -- based on a git commit and release number.

qlrd added 2 commits July 25, 2025 13:35
Prepare and run a reproductible environment to integration tests while
avoinding conflicts with different versions of embit. This build
bitcoind and elementsd to a temporary environment.
@qlrd qlrd changed the title Feat/improve dev environment test: improve dev environment for integration tests Jul 25, 2025
qlrd added 4 commits July 26, 2025 08:23
Adding a EMBIT_TEMP_DIR to scripts will allow us to have bitcoind and
elementsd installed on different versions of embit -- based on a git
commit. Also change some imports to finde properly some local libs.
This source code was formated automatically by the job defined in
pre-commit tool
Describe in the README that is recomended to use a virtual environment
package (virtualenv) while developing, so it will be possible to have a
reproductible code. Also explain how to install dependencies and which
are. Then how to run unit and integration tests.
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.

1 participant