Clarify that we discard mockMvc.perform((RequestBuilder) null) test

UnitTestBot · Aug 25, 2023 · 79ef8e0 · 79ef8e0
1 parent 1a253c4
commit 79ef8e0
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/...tlin/org/utbot/instrumentation/instrumentation/spring/SpringUtExecutionInstrumentation.kt b/...tlin/org/utbot/instrumentation/instrumentation/spring/SpringUtExecutionInstrumentation.kt
@@ -87,6 +87,10 @@ class SpringUtExecutionInstrumentation(
         if (parameters !is UtConcreteExecutionData) {
             throw IllegalArgumentException("Argument parameters must be of type UtConcreteExecutionData, but was: ${parameters?.javaClass}")
         }
+
+        // `RemovingConstructFailsUtExecutionInstrumentation` may detect that we fail to
+        // construct `RequestBuilder` and use `requestBuilder = null`, leading to a nonsensical
+        // test `mockMvc.perform((RequestBuilder) null)`, which we should discard
         if (parameters.stateBefore.executableToCall == mockMvcPerformMethodId && parameters.stateBefore.parameters.single().isNull())
             return UtConcreteExecutionResult(
                 stateBefore = parameters.stateBefore,