fix(protocol): fix vault name and symbol validation bug with more unit tests

[adaki2004]

• 1 bug (do not validate inputs for symbol and name)
• some additional tests
• fix commenting

[openzeppelin-code]

chore(protocol): more vault tests

Generated at commit: 0d741cf9175a90d285e5ac0fbf28584f40075975

🚨 Report Summary

	Severity Level	Results
Contracts	Critical High Medium Low Note Total	2 2 0 5 41 50
Dependencies	Critical High Medium Low Note Total	0 0 0 0 0 0

---

For more details view the full report in OpenZeppelin Code Inspector

[adaki2004]

@dantaik @davidtaikocha Could one of you please check out this branch locally, and forge fmt ?
Yesterday i was trying out something so i did a foundryup locally.. Since then the forge fmt command seemingly works differently than in our CICD.. so it seems it is making the forge format check fails here..
Or can we upgrade foundry with foundryup ? My current local version is:
forge 0.2.0 (c486fca 2024-05-06T00:20:07.911403000Z)

[dantaik]

@dantaik @davidtaikocha Could one of you please check out this branch locally, and forge fmt ? Yesterday i was trying out something so i did a foundryup locally.. Since then the forge fmt command seemingly works differently than in our CICD.. so it seems it is making the forge format check fails here.. Or can we upgrade foundry with foundryup ? My current local version is: forge 0.2.0 (c486fca 2024-05-06T00:20:07.911403000Z)

No worries about local forge fmt, now modified solidity files will be formatted automatically by a github workflow and all changes done by the formatter will be commited.

Local formats are now temporary, you can safely ignore them.