Python tests in justfile with nix

[jaoleal]

This piece of changes tries to automate our python tests inside our justfile.

test-int now just executes "poetry run poe tests" command defined in pyproject.toml and in run_tests.py.

the old test-int is renamed to test-wkspc to match better the internal command and to give space for the integration tests with python.

new command test-int-nix uses a nix-shell to setup every python crap, downloading and compiling utreexod and florestad, run the tests and exits

[jaoleal]

Still isnt ready to merge because im getting a strange behavior with the tests being executed by just and nix

[jaoleal]

@Davidson-Souza This error from actions is happening because i had to change the way that add_node_settings calls floresta. I removed the cargo dependency and thats solved the issue with the connection being refused by floresta.
Now the florestad package thats being called by the nix script its building the package using our nix derivation(build.nix).

Seeing your interest in increasing our use of nix in this repo(supposed by #335 ), the problem can be solved by using only nix to setup and run the tests inside functional.yml.

[Davidson-Souza]

Seeing your interest in increasing our use of nix in this repo(supposed by #335 ), the problem can be solved by using only nix to setup and run the tests inside functional.yml.

I don't want to tie our development to nix (I don't use nix). It's cool for specific things like #335, but I wouldn't make it a requirement for development.

[jaoleal]

@Davidson-Souza what did you think of the solution of f025e0e and 0f79c64 ?

prepare.sh verifies and setup things while checking dependencies

run.sh uses only things that prepare.sh builded and touched.

This avoids a user that has florestad installed in their $PATH not conflict with the tests that hes doing and also removes the cargo dependencie while calling floresta in our framework without changing it too much.

This makes it able to run inside nix without too much trouble... the deterministic builds of #335 will thank this change

[Davidson-Souza]

@Davidson-Souza what did you think of the solution of f025e0e and 0f79c64 ?

Seems ok, I concept ACK this (haven't tested yet)

[jaoleal]

dont merge, just triggering actions.

[jaoleal]

It works! Ready to merge.

[Davidson-Souza]

Concept ACK. I just have a few small requests before we can merge

[jaoleal]

Applied suggestions by @Davidson-Souza

[Davidson-Souza]

@jaoleal on 75e2a93 test-int-setup is still duplicate

[jaoleal]

oh, sorry