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

Fix/e2e test #2

Closed
wants to merge 3 commits into from
Closed

Fix/e2e test #2

wants to merge 3 commits into from

Conversation

MSghais
Copy link
Contributor

@MSghais MSghais commented Jul 25, 2024

Start fix #1 WIP

  • Change STWO with the main repo (Serde merged in main)
  • Add workspace tests
  • Test workspace with Cargo.toml

Running script CI/CD:
cargo test --package tests --lib -- e2e_test --show-output -- --no-capture

TODO:

  • Fix e2e_test
  • Fix or change CI/CD command maybe
  • STWO main breaking changes

Issues:

  1. With the fork of STWO with serde:

I encounter this error with the new tests workspace on the script:

notifications.next() is not defined now

no method named next found for struct tokio::sync::broadcast::Receiver in the current scope
private field, not a method

  1. With the STWO main branch:

Layer::GrandProduct(Mle::new(SecureFieldVec {
| ^^^^^^^^^^^^^^ help: a type alias with a similar name exists: SecureField

@MSghais MSghais closed this Jul 27, 2024
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.

fix: e2e tests github actions
1 participant