Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore/fix cardano wasm integration test #204

Conversation

FelipeRosa
Copy link

@FelipeRosa FelipeRosa commented Apr 8, 2024

No description provided.

@FelipeRosa FelipeRosa requested a review from stevenj as a code owner April 8, 2024 20:26
@FelipeRosa FelipeRosa self-assigned this Apr 8, 2024
@FelipeRosa FelipeRosa changed the base branch from main to feat/verbose-rust-unit-tests April 8, 2024 20:43
@@ -4,7 +4,7 @@ VERSION 0.7

# Set up our target toolchains, and copy source files.
builder:
DO github.com/input-output-hk/catalyst-ci/earthly/rust:v2.11.0+SETUP
DO github.com/input-output-hk/catalyst-ci/earthly/rust:refs/heads/chore/update-wit-binding-cli+SETUP
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This must be changed to the release version of catalyst-ci in this PR: #204 before merging.

Copy link
Collaborator

@stevenj stevenj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@stevenj stevenj merged commit 608c2c7 into feat/verbose-rust-unit-tests Apr 9, 2024
23 of 28 checks passed
@stevenj stevenj deleted the chore/fix-cardano-wasm-integration-test branch April 9, 2024 01:52
stevenj added a commit that referenced this pull request Apr 9, 2024
* fix(wasm): Fix running wasm integration tests

* fix(wasm): Make logging work inside the wasm integration tests

* feat(wasm): working hashing integration test

* ci(wasm): align rust builds with cat-ci latest version

* feat(wasm): Add crypto wasm integration tests

* fix(hermes): Re-enable hermes full build after getting wasm integration tests working

* fix(wasm): silence the initial print inside wasm integration tests to prevent issues when running

* fix(wasm): remove unused import after silencing the wasm integration tester

* style(hermes): Fix code format

* fix(spelling): spelling issue

* fix(cbork): Align deny,toml with CI

* fix(wasm): Align deny.toml with CI

* Update hermes/bin/src/runtime_extensions/hermes/crypto/host.rs

Co-authored-by: bkioshn <35752733+bkioshn@users.noreply.github.com>

* Update hermes/bin/src/runtime_extensions/hermes/crypto/host.rs

Co-authored-by: bkioshn <35752733+bkioshn@users.noreply.github.com>

* feat: hermes-cron integration test cases (#199)

* feat(wip): stub hermes-cron integration test cases

* feat: add tests for cron api functions

* fix: compare returned string with expected

* feat(warm): Add Cardano Runtime Extension integration test skeleton (#198)

* feat(wasm): Add Cardano Runtime Extension integration tests skeleton

* feat(wasm): Use pallas to parse blocks in Cardano RTE integration test component

---------

Co-authored-by: Steven Johnson <stevenj@users.noreply.github.com>

* fix(wasm): remove commented obsolete code

* Update wasm/integration-test/crypto/crypto.c

Co-authored-by: bkioshn <35752733+bkioshn@users.noreply.github.com>

* test: time module (#200)

* feat: initial commit

* feat: simple function

* ci: add test

* test: restructure and add test to localtime

* fix: minor format

* refactor: remove localtime name

* fix: localtime issue

* feat: clocks test

* fix: test warms

* fix: use existing tests

* chore: earthfile tmp

* fix: builder

* fix: remove wall test

* chore: fmtfix

* fix(spelling): add to project words

* fix(wasm): remove generated bindings, try and use autogenerated ones

* chore(wasm): Fix Cardano RTE wasm integration test module (#204)

* fix(hermes): Update rust builders to the latest

---------

Co-authored-by: bkioshn <35752733+bkioshn@users.noreply.github.com>
Co-authored-by: Joaquín Rosales <joaquin.rosales@iohk.io>
Co-authored-by: Felipe Rosa <felipe.rosa@iohk.io>
Co-authored-by: Apisit Ritreungroj <38898766+apskhem@users.noreply.github.com>
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.

2 participants