vacuumlabs / cardano-hw-cli

Cardano CLI tool for hardware wallets
71 stars 24 forks source link

add support for Speculos integration testing #116

Closed janmazak closed 2 years ago

davidmisiak commented 2 years ago

Hm, do you have an idea why CI is failing here? Alonzo PR seems to be ok.

janmazak commented 2 years ago

CI gives details: https://app.circleci.com/pipelines/github/vacuumlabs/cardano-hw-cli/710/workflows/357baa4a-7626-481d-8d3d-64be42803c7d/jobs/687 yarn test-bin fails, but the error seems totally unrelated to the changes in this PR, and only occurs at nexe for mac-x64-12.18.2.

davidmisiak commented 2 years ago

and only occurs at nexe for mac-x64-12.18.2.

Actually, the error was not thrown during building for mac, but during executing the following command - running ./build/linux/archive/cardano-hw-cli/cardano-hw-cli. The reason was that speculos.ts was a TS file outside the src directory, which confused tsc. I changed it to speculos.js (in line with all other files in the test directory) and everything works now :+1: