search

entropyxyz / sdk

Official JavaScript SDK for Entropy blockchain.
GNU Affero General Public License v3.0
7 stars 0 forks source link

Use `local-devnet` as infrastructure for the test harness. #228

Closed vitropy closed 10 months ago

vitropy commented 10 months ago

This PR makes use of the new Docker Compose configuration and the new local-devnet chain specification in CI builds. With this change, we're able to remove and thus simplify a lot of testing utilities and start dogfooding more generic base infrastructure in more parts of our CI pipelines.

frankiebee commented 10 months ago

this makes local environment times long and fail tests. Review of this PR requires an on call conversation please schedule some time with me tomorrow if available @vitropy

vitropy commented 10 months ago

this makes local environment times long and fail tests. Review of this PR requires an on call conversation please schedule some time with me tomorrow if available @vitropy

There are two issues with "local" testing, one addressed in this PR and the other is actually entropyxyz/entropy-core#473.

I pushed a commit to entropy-core that seems to build much more reliably. This should be available for local testing, too.

Note that the tests still fail, and this is the expected outcome because they aren't failing because of container crashes but rather because of legitimate code differences. This recent build log shows such a test failure.

As for the second issue, if you've got an Apple Silicon-based macOS machine for your local workstation, the fix for the moment is to make sure you've enabled the Rosetta translation in your Docker Desktop installation, then try running the tests again.