search

makerdao / spells-mainnet

Staging repo for MakerDAO weekly executive spells
GNU Affero General Public License v3.0
122 stars 50 forks source link

`make test-forge match="test"` is broken #350

Closed amusingaxl closed 1 year ago

amusingaxl commented 1 year ago

Running:

make test-forge match="testBytecodeMatches"

Results in the following error:

./scripts/test-dssspell-forge.sh match="testBytecodeMatches" block=""
Using DssExecLib at: 0x8De6DDbCd5053d32292AAA0D2105A32d108484a6
error: unexpected argument '--match' found

  tip: a similar argument exists: '--watch'

Usage: forge test --fork-url <URL> <--watch [<PATH>...]|--no-restart|--run-all|--watch-delay <DELAY>>

For more information, try '--help'.
make: *** [Makefile:7: test-forge] Error 2

The reason is that forge removed the deprecated --match flag in favor of --match-contract/--match-path/--match-test.

I created #349 to fix the issue.