Commit 608a58d8 authored by Mark Tyneway's avatar Mark Tyneway Committed by GitHub

contracts-bedrock: fix compiler warning (#10778)

Fixes the following compiler warning:

```
Compiler run successful with warnings:
Warning (2018): Function state mutability can be restricted to view
  --> test/periphery/drippie/dripchecks/CheckBalanceLow.t.sol:20:5:
   |
20 |     function test_name_succeeds() external {
   |     ^ (Relevant source part starts here and spans across multiple lines).

Warning (2018): Function state mutability can be restricted to view
  --> test/periphery/drippie/dripchecks/CheckGelatoLow.t.sol:45:5:
   |
45 |     function test_name_succeeds() external {
   |     ^ (Relevant source part starts here and spans across multiple lines).

Warning (2018): Function state mutability can be restricted to view
  --> test/periphery/drippie/dripchecks/CheckSecrets.t.sol:30:5:
   |
30 |     function test_name_succeeds() external {
   |     ^ (Relevant source part starts here and spans across multiple lines).

Warning (2018): Function state mutability can be restricted to view
  --> test/periphery/drippie/dripchecks/CheckTrue.t.sol:19:5:
   |
19 |     function test_name_succeeds() external {
   |     ^ (Relevant source part starts here and spans across multiple lines).
```

This should be caught in CI, not sure why it wasn't
parent efc29d7b
...@@ -17,7 +17,7 @@ contract CheckBalanceLowTest is Test { ...@@ -17,7 +17,7 @@ contract CheckBalanceLowTest is Test {
} }
/// @notice Test that the `name` function returns the correct value. /// @notice Test that the `name` function returns the correct value.
function test_name_succeeds() external { function test_name_succeeds() external view {
assertEq(c.name(), "CheckBalanceLow"); assertEq(c.name(), "CheckBalanceLow");
} }
......
...@@ -42,7 +42,7 @@ contract CheckGelatoLowTest is Test { ...@@ -42,7 +42,7 @@ contract CheckGelatoLowTest is Test {
} }
/// @notice Test that the `name` function returns the correct value. /// @notice Test that the `name` function returns the correct value.
function test_name_succeeds() external { function test_name_succeeds() external view {
assertEq(c.name(), "CheckGelatoLow"); assertEq(c.name(), "CheckGelatoLow");
} }
......
...@@ -27,7 +27,7 @@ contract CheckSecretsTest is Test { ...@@ -27,7 +27,7 @@ contract CheckSecretsTest is Test {
} }
/// @notice Test that the `name` function returns the correct value. /// @notice Test that the `name` function returns the correct value.
function test_name_succeeds() external { function test_name_succeeds() external view {
assertEq(c.name(), "CheckSecrets"); assertEq(c.name(), "CheckSecrets");
} }
......
...@@ -16,7 +16,7 @@ contract CheckTrueTest is Test { ...@@ -16,7 +16,7 @@ contract CheckTrueTest is Test {
} }
/// @notice Test that the `name` function returns the correct value. /// @notice Test that the `name` function returns the correct value.
function test_name_succeeds() external { function test_name_succeeds() external view {
assertEq(c.name(), "CheckTrue"); assertEq(c.name(), "CheckTrue");
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment