Skip to content

fix forge tests#194

Merged
ivanzhelyazkov merged 2 commits intodevfrom
fix-forge-tests
Oct 23, 2025
Merged

fix forge tests#194
ivanzhelyazkov merged 2 commits intodevfrom
fix-forge-tests

Conversation

@ivanzhelyazkov
Copy link
Collaborator

@ivanzhelyazkov ivanzhelyazkov commented Oct 22, 2025 •
edited
Loading

  • Fix forge tests so they run with more recent stable foundry version:
  • Replace testFail with tests which check event emittal (getRecordedLogs)
  • Add view and pure modifiers where needed to remove warnings
  • Fix test case parser contracts json length counting - for Trading, CarbonPOL and CarbonVortex tests

@ivanzhelyazkov ivanzhelyazkov merged commit 5039212 into dev Oct 23, 2025
6 checks passed
@ivanzhelyazkov ivanzhelyazkov deleted the fix-forge-tests branch October 23, 2025 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yudilevi yudilevi yudilevi approved these changes

@lbeder lbeder lbeder approved these changes

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

@ivanzhelyazkov @yudilevi @lbeder