Skip to content

Commit c03f6da

Browse files
committed
Add a note about not using the global mock variables directly in tests.
Signed off by: Priyanka Jiandani <[email protected]>
1 parent 837ae4b commit c03f6da

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

CONTRIBUTING.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,7 @@ make unit-test
119119

120120
*It is of special note that many of the mock objects in the [StorageCluster
121121
tests](./pkg/controller/storagecluster/storagecluster_controller_test.go) are
122-
reused in many other test cases. They may be useful in developing your own.*
122+
reused in many other test cases. Please avoid using the global mock variables directly, instead use the mock variables by creating DeepCopy() to prevent modification of global mock variables. They may be useful in developing your own.*
123123

124124
### Certificate of Origin
125125

0 commit comments

Comments
 (0)