adding a function and editing the error message for the issue #4151

[Saty248]

• [x ] Because this PR includes a bug fix, relevant tests have been included.

---

i have changed the error message for the issue #4151 and i've also added a function deleteOnReset() which resets the snapshots array.

let me know what other changes I need to make on this issue.

[changeset-bot]

⚠️ No Changeset found

Latest commit: e27a7d9

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
hardhat	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 17, 2024 7:54pm
hardhat-storybook	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	May 17, 2024 7:54pm

[fvictorio]

Hey @Saty248, thanks for this. This is a really interesting idea. It won't work for raw hardhat_reset calls, so we should still detect those kind of problems, but that could be a separate efforts.

This needs tests though, and I dont' think the code is formatted correctly.

[fvictorio]

This function name is overspecified. I think clearSnapshots would be better.

[Saty248]

hey @fvictorio,
i've changed the function name from deleteOnRequest to clearSnapshots and formated the code using lint.Also I've added a test case for the clearsnapshot on reset.ts under test. Do I need to added additional test cases for hardhat reset with a url and block number?