-
Notifications
You must be signed in to change notification settings - Fork 14
Description
Hi @pradeeban @karthiksathishjeemain,
Hope you’re doing well.
For a few recent contract related PRs, I validated the changes locally using Hardhat tests covering document creation, updates, deletion, purchases, and access control. I noticed that the repository currently does not include any test files.
I wanted to check if you would be open to adding Hardhat tests to the repo going forward. I believe having basic contract tests could be useful for future contributors to validate changes locally and reduce regressions, especially since deploying on a testnet takes more time just to catch compilation or basic logic issues.
If this sounds good, I can add the existing tests in a separate PR and follow any structure or conventions you’d prefer.
Thanks for your time.