Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: add systemtest capabilities#304

Merged
technicallyty merged 12 commits intomainfrom
technicallyty/systemtests
Jul 18, 2025
Merged

feat: add systemtest capabilities#304
technicallyty merged 12 commits intomainfrom
technicallyty/systemtests

Conversation

@technicallyty
Copy link
Contributor

Description

adds the systemtest framework

depends on #301


Author Checklist

All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.

I have...

  • tackled an existing issue or discussed with a team member
  • left instructions on how to review the changes
  • targeted the main branch

Reviewers Checklist

All items are required.
Please add a note if the item is not applicable
and please add your handle next to the items reviewed
if you only reviewed selected items.

I have...

  • added a relevant changelog entry to the Unreleased section in CHANGELOG.md
  • confirmed all author checklist items have been addressed
  • confirmed that this PR does not change production code
  • reviewed content
  • tested instructions (if applicable)
  • confirmed all CI checks have passed

@technicallyty technicallyty mentioned this pull request Jul 18, 2025
9 tasks
@technicallyty technicallyty marked this pull request as ready for review July 18, 2025 18:58
@technicallyty technicallyty requested review from a team as code owners July 18, 2025 18:58
@technicallyty technicallyty marked this pull request as draft July 18, 2025 19:06
Copy link
Contributor

@aljo242 aljo242 left a comment

Choose a reason for hiding this comment

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

lgtm - can we add a CI target for this too ? (even if there are no tests now)

return maps.Clone(maccPerms)
}

type EVMAppConfig struct {
Copy link
Contributor

Choose a reason for hiding this comment

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

in the future should we define this in the EVM library and not in the evmd package? can make an issue if you think that makes sense

Copy link
Contributor Author

Choose a reason for hiding this comment

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

maybe, i imagine most people are just going to copy paste the stuff in root

@technicallyty technicallyty marked this pull request as ready for review July 18, 2025 20:56
@technicallyty technicallyty added this pull request to the merge queue Jul 18, 2025
Merged via the queue into main with commit 03ede6c Jul 18, 2025
16 checks passed
@technicallyty technicallyty deleted the technicallyty/systemtests branch July 18, 2025 20:57
zsystm pushed a commit to zsystm/evm that referenced this pull request Nov 2, 2025
* localnet

* remove that'

* some fixes

* expose json rpc from container

* json rpc metrics

* add systemtest stuff

* make testnet command compatible with systemtests

* ci

* merge_group on systemtest
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