Clean up and Modernize Unit Testing Infrastructure

[DeveloperAmrit]

Fixes #6

Summary
This PR cleans up the testing suite by migrating from legacy custom test utilities to the standard forge-std framework and resolving compiler limitations.

Changes

1. Modernize Testing Framework
   Migrated the unit tests to use standard Foundry Test contracts and the vm cheatcode, reducing reliance on custom-built legacy utilities.

• Replaced CTest inheritance with forge-std/Test.sol.
• Replaced custom CheatCodes instantiation with the built-in vm object.
• Updated imports and calls in:
  • Djed.t.sol
  • Coin.t.sol
  • Fee.t.sol
  • TransactionLimit.t.sol

2. Infrastructure Cleanup

• Removed Redundant Utils: Deleted the src/test/utils/ directory containing legacy Cheatcodes.sol, Console.sol, and Ctest.sol, as these are now provided by forge-std.
• Compiler Optimization: Enabled the Solidity optimizer in foundry.toml to resolve "Stack too deep" errors during compilation of the test suite.

3. Stability & Formatting

• Standardized pragma versions and imports across test files.
  Fixed minor syntax errors (redundant braces) introduced during refactoring.

Verification Results
All tests passed successfully after the changes.

Summary by CodeRabbit

Release Notes

• Chores
  • Enabled Solidity optimizer in build configuration with 200 optimization runs
  • Migrated test suite to Foundry's standard testing framework for improved compatibility and maintainability
  • Removed legacy testing utility files and replaced with standard Foundry testing primitives

[coderabbitai]

📝 Walkthrough

Walkthrough

The test suite has been migrated from custom test utilities (CTest, CheatCodes, Utilities) to Foundry's standard forge-std Test framework. All test files have been updated to use vm primitives. The foundry.toml configuration has been updated to enable the optimizer with 200 runs.

Changes

Cohort / File(s)	Summary
Configuration foundry.toml	Added optimizer configuration: optimizer = true and optimizer_runs = 200.
Test Framework Migration src/test/Coin.t.sol, src/test/Djed.t.sol, src/test/Fee.t.sol, src/test/TransactionLimit.t.sol	Updated test contracts to inherit from Test instead of CTest. Replaced all CheatCodes cheat calls with vm primitives (e.g., cheats.deal → vm.deal, cheats.prank → vm.prank, cheats.expectRevert → vm.expectRevert). Updated imports to use forge-std/Test.sol.
Utility Removal src/test/utils/Cheatcodes.sol, src/test/utils/Ctest.sol, src/test/utils/Utilities.sol	Deleted entire custom utility contracts: CheatCodes interface, CTest contract with assertion and logging helpers, and Utilities contract with test user and block mining helpers. These are replaced by forge-std equivalents.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Poem

A bunny hops from old to new,
From CheatCodes bright to vm true.
Test utilities fade away,
As forge-std leads the way! 🐰✨
With optimizer on, we're fast and lean,
Best test migration ever seen!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main objective of the PR: modernizing the test infrastructure by migrating to Foundry's forge-std and removing legacy testing utilities.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}