Fix-test: simplify panic message syntax in should_panic

[CaptainLEVI-XXX]

Fixes [#1139 ]

Overview

Simplifying panic message syntax across test files by removing unnecessary tuple wrappers for single-value messages.

Changes

// Before
#[should_panic(expected: ('EthAccount: invalid tx version',))]
#[should_panic(expected: ('EthAccount: invalid signature',))]
#[should_panic(expected: ('EthAccount: invalid caller',))]

// After
#[should_panic(expected: 'EthAccount: invalid tx version')]
#[should_panic(expected: 'EthAccount: invalid signature')]
#[should_panic(expected: 'EthAccount: invalid caller')]

• Ran full test suite and formatting checks:
  • ✅ snforge test - All tests passing
  • ✅ scarb fmt -w --check - Code formatting verified

[ericnordelo]

Hey @CaptainLEVI-XXX, it looks like you are using a different scarb version locally than 2.9.2, and then the format is giving you a different output. Can you please use 2.9.2 locally and format the files again?

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.41%. Comparing base (821a77a) to head (8049c88).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1332   +/-   ##
=======================================
  Coverage   89.41%   89.41%           
=======================================
  Files          81       81           
  Lines        3496     3496           
=======================================
  Hits         3126     3126           
  Misses        370      370           

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 821a77a...8049c88. Read the comment docs.

[ericnordelo]

LGTM!