Review fixes

Author: CaelmBleidd

usvm-ts/src/main/kotlin/org/usvm/machine/expr/TsExprResolver.kt

@@ -878,7 +878,7 @@ class TsExprResolver(
                 val fakeType = instanceRef.getFakeType(scope)
 
                 // If we want to get length from a fake object, we assume that it is an array.
-                scope.assert(fakeType.refTypeExpr)
+                scope.doWithState { pathConstraints += fakeType.refTypeExpr }
 
                 val refLValue = getIntermediateRefLValue(instanceRef.address)
                 val obj = scope.calcOnState { memory.read(refLValue) }
@@ -973,6 +973,12 @@ class TsExprResolver(
     }
 
     override fun visit(expr: EtsNewArrayExpr): UExpr<out USort>? = with(ctx) {
+        val arrayType = expr.type
+
+        require(arrayType is EtsArrayType) {
+            "Expected EtsArrayType in newArrayExpr, but got ${arrayType::class.simpleName}"
+        }
+
         scope.calcOnState {
             val size = resolve(expr.size) ?: return@calcOnState null
 
@@ -1003,8 +1009,6 @@ class TsExprResolver(
                 blockOnFalseState = allocateException(EtsStringType) // TODO incorrect exception type
             )
 
-            val arrayType = expr.type as EtsArrayType
-
             if (arrayType.elementType is EtsArrayType) {
                 TODO("Multidimensional arrays are not supported yet, https://github.com/UnitTestBot/usvm/issues/287")
             }

usvm-ts/src/test/kotlin/org/usvm/util/TsTestResolver.kt

@@ -109,7 +109,9 @@ class TsTestResolver {
                     resolvedLValuesToFakeObjects += arrayIndexLValue to fakeObject
                 }
 
-                else -> error("Unexpected lValue type: ${lValue::class.java.name}")
+                else -> {
+                    error("Unexpected lValue type: ${lValue::class.java.name}")
+                }
             }
         }
     }
@@ -265,7 +267,9 @@ open class TsTestStateResolver(
                         resolveExpr(fakeObject.extractRef(finalStateMemory))
                     }
 
-                    else -> error("Unsupported fake object type: $fakeType")
+                    else -> {
+                        error("Unsupported fake object type: $fakeType")
+                    }
                 }
             }
 

usvm-util/src/main/kotlin/org/usvm/test/util/checkers/Matchers.kt

@@ -22,7 +22,7 @@ val ignoreNumberOfAnalysisResults = AnalysisResultsNumberMatcher(
 
 val noResultsExpected = AnalysisResultsNumberMatcher(
     description = "No executions expected",
-    matcherFailedMessage = { it -> "Expected no analysis results, but $it executions were found" }
+    matcherFailedMessage = { "Expected no analysis results, but $it executions were found" }
 ) { it == 0 }
 
 class AnalysisResultsNumberMatcher(