chore(cli): Update cartesi-machine-stored-hash test case. - #508
chore(cli): Update cartesi-machine-stored-hash test case.#508brunomenezes wants to merge 1 commit into
Conversation
…ad of fix hash value. * Depending on environment it was build the hash will change making the test fail even though the dapp code template did not change.
|
Coverage Report
📁 File Coverage (19 files)
|
We should aim a reproducible build. Isn't there any thing we can do to ensure this. |
I am not sure, as from time to time we are changing the application-template Dockerfile, as it will eventually fail to build and updates are necessary. If I understood it right, the machine hash-generated factor, not just the code. |
Summary
This PR changes the test assertion for
cartesi-machine-stored-hashcases to verify that a valid hash string is present, rather than asserting against a fixed hash value.Because build environments can produce slight variations in the resulting output hash, strict hash comparison was causing test failures in CI/local environments—even when the dApp template code itself remained unchanged.