Feat: Persist Testing Corpus artifacts

manifoldfinance / mevETH2

mevETH LST Protocol - Repo has migrated see link

https://github.com/MEV-Protocol/meveth

27 stars 2 forks source link

Feat: Persist Testing Corpus artifacts #159

Closed sambacha closed 1 year ago

sambacha commented 1 year ago

Currently, we ignore

test/echidna/echidna-corpus/

Should we instead make this a vendored attribute -- meaning it wont show up on diffs at all, but will be persisted within the repo? Or should we persist it outside and just fetch it potentially during testing workflows?

sandybradley commented 1 year ago

Corpus can be persisted this in some fashion. I didn't think these files had any value if tests are all passing?

FYI: only 1 echidna test for ERC4626

sambacha commented 1 year ago

We are doing this on ci.farm and @sutbot