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

feat: local integration tests #70

Merged
merged 93 commits into from
May 23, 2024
Merged

feat: local integration tests #70

merged 93 commits into from
May 23, 2024

Conversation

qalisander
Copy link
Member

@qalisander qalisander commented May 16, 2024

Fixes #53

To make tests pass stylus-sdk was reverted to 0.4.3 and associated shim changes also.
All tests pass serially (in future there is a chance to run them concurrently).

PR Checklist

  • Tests
  • Documentation

Copy link

netlify bot commented May 16, 2024

👷 Deploy request for contracts-stylus pending review.

A Netlify team Owner will need to approve the deploy before you can run your build.

Are you a team Owner? Visit the deploys page to approve it

Need more help? Learn more in the Netlify docs

Name Link
🔨 Latest commit 403f904

@alexfertel
Copy link
Contributor

Let's goooo, excited to review this!

@qalisander qalisander force-pushed the local-integration-tests branch from 4d136a7 to d226068 Compare May 16, 2024 14:56
e2e-tests/src/context/mod.rs Outdated Show resolved Hide resolved
e2e-tests/src/context/mod.rs Outdated Show resolved Hide resolved
@qalisander qalisander requested review from alexfertel and bidzyyys May 23, 2024 14:09
alexfertel
alexfertel previously approved these changes May 23, 2024
Copy link
Contributor

@alexfertel alexfertel left a comment

Choose a reason for hiding this comment

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

Ship ship ship 🚀

I just realized that this PR is in conflict with the new constructors, so we have to figure out how to introduce koba in these files as well on that PR.

@qalisander qalisander merged commit d4b0554 into main May 23, 2024
15 checks passed
@alexfertel alexfertel deleted the local-integration-tests branch May 23, 2024 14:36
@alexfertel alexfertel removed this from the Release v0.1.0 milestone Jun 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
effort: high Large or difficult task. type: feature New feature request. type: test Changes to the testing suite.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Setup integration tests using nitro-testnode
3 participants