Skip to content

fix(GHA): polish test workflow #475

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.

Sign up for GitHub

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

Jump to bottom
Draft
lucas2brh wants to merge 28 commits into
base: main
Choose a base branch
from
Draft

fix(GHA): polish test workflow #475

lucas2brh wants to merge 28 commits into from

Conversation

@lucas2brh
Copy link
Collaborator

@lucas2brh lucas2brh commented Sep 24, 2025 •
edited
Loading

Description

  • Refactor workflow structure for improved readability and maintainability
  • Add test pattern selection to enable running specific test suites (e.g., dispute, licensing, royalty)
  • Update default currency token to WIP for consistency

Test Plan

Ran tests and all passed.
https://github.com/storyprotocol/protocol-core-v1/actions/runs/17963876029
https://github.com/storyprotocol/protocol-core-v1/actions/runs/17963319330

Related Issue

jia57b and others added 23 commits July 18, 2025 15:01
@jia57b
add aeneid config for hardhat e2e test
735b518
@jia57b
update constants address for testing
4e4a240
@jia57b
update mockerc20 address
115a566
@jia57b
use mintAmount for mockerc20
9baa217
@jia57b
fix address
ff7e5a8
@jia57b
test upgrade
9b69c6d
@jia57b
script to verify upgrade
2eac416
@jia57b
update verification script
dd9ed55
@jia57b
update v2 test contract
9e6fc5c
@jia57b
update v2 test contract with constant
06098c8
@jia57b
update contract address for public poc
9f92e1c
@jia57b
update to use WIP
39a8ef3
@lucas2brh
chore
bd57e93
@lucas2brh
tmp: QA for cantina 124
38cd269
@jia57b
update for aeneid test config
2b32fe6
@jia57b
update Admin whitelist invalid group reward pool test
33873e6
@lucas2brh
polish workflow
5878c92
@lucas2brh
use 0x1514 as the default token
4175cc2
@lucas2brh
whitelist the currencyToken first
185d73f
@lucas2brh
modify the erc20 related logic
0d0d8b2
@lucas2brh
polish
02cc344
@lucas2brh
fix: support new DisputeRaised event signature in dispute tests
30786ad 
- Add backward compatibility for event signature changes
- Handle both old and new DisputeRaised event formats
@lucas2brh
feat: add test pattern selection to e2e workflow
84767d5
@lucas2brh lucas2brh mentioned this pull request Sep 24, 2025
Closed
lucas2brh
lucas2brh commented Sep 24, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kingster-will kingster-will Awaiting requested review from kingster-will kingster-will will be requested when the pull request is marked ready for review kingster-will is a code owner

@romain-mg romain-mg Awaiting requested review from romain-mg romain-mg will be requested when the pull request is marked ready for review romain-mg is a code owner

@LeoHChen LeoHChen Awaiting requested review from LeoHChen LeoHChen will be requested when the pull request is marked ready for review LeoHChen is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@lucas2brh @jia57b