refactor(tests): remove unnecessary view modifier from interface tests

Le-Caignec · Le-Caignec · commit 3ee774faede8 · 2025-03-11T14:12:15.000+01:00
diff --git a/packages/sharing-smart-contract/test/SupportsInterface.t.sol b/packages/sharing-smart-contract/test/SupportsInterface.t.sol
@@ -20,11 +20,11 @@ contract SupportsInterface is Test {
         );
     }
 
-    function testSupports721Interface() external view {
+    function testSupports721Interface() external {
         assertTrue(_dataProtectorSharing.supportsInterface(type(IAccessControl).interfaceId));
     }
 
-    function testSupportsAccessControlInterface() external view {
+    function testSupportsAccessControlInterface() external {
         assertTrue(_dataProtectorSharing.supportsInterface(type(IERC721).interfaceId));
         assertTrue(_dataProtectorSharing.supportsInterface(type(IERC721Metadata).interfaceId));
     }

Original file line number	Diff line number	Diff line change
`@@ -20,11 +20,11 @@ contract SupportsInterface is Test {`
`20`	`20`	`);`
`21`	`21`	`}`
`22`	`22`
`23`		`- function testSupports721Interface() external view {`
	`23`	`+ function testSupports721Interface() external {`
`24`	`24`	`assertTrue(_dataProtectorSharing.supportsInterface(type(IAccessControl).interfaceId));`
`25`	`25`	`}`
`26`	`26`
`27`		`- function testSupportsAccessControlInterface() external view {`
	`27`	`+ function testSupportsAccessControlInterface() external {`
`28`	`28`	`assertTrue(_dataProtectorSharing.supportsInterface(type(IERC721).interfaceId));`
`29`	`29`	`assertTrue(_dataProtectorSharing.supportsInterface(type(IERC721Metadata).interfaceId));`
`30`	`30`	`}`