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: better typegen contract factory integration with launchTestNode #3398

Merged
merged 13 commits into from
Nov 19, 2024

Conversation

nedsalk
Copy link
Contributor

@nedsalk nedsalk commented Nov 15, 2024

In this release, we:

  • Improved launchTestNode and typegen'd contract factories integration

Summary

With this PR we can now pass in the typegen'd contract factory directly instead of having to wrap it in a { factory: TypegendContractFactory } object.

Checklist

  • All changes are covered by tests (or not applicable)
  • All changes are documented (or not applicable)
  • I reviewed the entire PR myself (preferably, on GH UI)
  • I described all Breaking Changes (or there's none)

Copy link

vercel bot commented Nov 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fuels-template ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 18, 2024 2:44pm
ts-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 18, 2024 2:44pm
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
ts-docs-api ⬜️ Ignored (Inspect) Nov 18, 2024 2:44pm

@nedsalk nedsalk marked this pull request as draft November 15, 2024 09:22
@nedsalk nedsalk marked this pull request as ready for review November 15, 2024 09:27
Copy link

codspeed-hq bot commented Nov 15, 2024

CodSpeed Performance Report

Merging #3398 will not alter performance

Comparing ns/feat/launch-test-node-contract-factory-integration (868d9ad) with master (c904a98)

Summary

✅ 18 untouched benchmarks

Torres-ssf
Torres-ssf previously approved these changes Nov 15, 2024
packages/contract/src/test-utils/launch-test-node.ts Outdated Show resolved Hide resolved
Copy link
Member

@maschad maschad left a comment

Choose a reason for hiding this comment

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

Great work on this @nedsalk , it will be a huge refactor to propagate these changes across the repo so that can be in a follow up PR

Copy link
Contributor

Coverage Report:

Lines Branches Functions Statements
64.2%(+0.01%) 69.71%(+0.06%) 72.99%(+0%) 64.64%(+0.01%)
Changed Files:
Ok File (✨=New File) Lines Branches Functions Statements
🔴 packages/contract/src/test-utils/launch-test-node.ts 97.61%
(+0.11%)
97.05%
(+0.63%)
100%
(+0%)
97.67%
(+0.11%)

@maschad maschad merged commit 6ab3e6b into master Nov 19, 2024
25 checks passed
@maschad maschad deleted the ns/feat/launch-test-node-contract-factory-integration branch November 19, 2024 07:09
nedsalk added a commit that referenced this pull request Nov 20, 2024
…e` (#3398)

* feat: better typegen contract factory integration with `launchTestNode`

* give name to type

* remove type assertion to improve inference

* update docs to new approach

* changeset

* revert unnecessary change

* rerun workflows

* revert change

* give name to intermediary type

---------

Co-authored-by: Chad Nehemiah <chad.nehemiah94@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat Issue is a feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve launchTestNode integration with typegen contract factories
5 participants