Move @CheckReturnValue from methods to class scope (#2379)

mockito · Aug 2, 2021 · f32973d · f32973d
1 parent 5891de1
commit f32973d
Show file tree

Hide file tree

Showing 10 changed files with 25 additions and 65 deletions.
diff --git a/src/main/java/org/mockito/ArgumentCaptor.java b/src/main/java/org/mockito/ArgumentCaptor.java
@@ -59,6 +59,7 @@
  * @see Captor
  * @since 1.8.0
  */
+@CheckReturnValue
 public class ArgumentCaptor<T> {
 
     private final CapturingMatcher<T> capturingMatcher = new CapturingMatcher<T>();
@@ -79,7 +80,7 @@ private ArgumentCaptor(Class<? extends T> clazz) {
      * @return null or default values
      */
     public T capture() {
-        Mockito.argThat(capturingMatcher);
+        T ignored = Mockito.argThat(capturingMatcher);
         return defaultValue(clazz);
     }
 

diff --git a/src/main/java/org/mockito/ArgumentMatchers.java b/src/main/java/org/mockito/ArgumentMatchers.java
@@ -112,6 +112,7 @@
  *
  * @see AdditionalMatchers
  */
+@CheckReturnValue
 @SuppressWarnings("unchecked")
 public class ArgumentMatchers {
 
@@ -258,7 +259,7 @@ public static <T> T isA(Class<T> type) {
      */
     @Deprecated
     public static <T> T anyVararg() {
-        any();
+        Object ignored = any();
         return null;
     }