refactor: simplify and make errors more explicit for Utils#frame method

Author: gaeljw

CHANGELOG.md

@@ -13,6 +13,8 @@ See also the [CHANGELOG](https://github.com/cucumber/cucumber-jvm/blob/master/CH
 
 ### Changed
 
+- [Internal] Fix `getLocation` wherever it was wrongly defined. ([#404](https://github.com/cucumber/cucumber-jvm-scala/issues/404))
+
 ### Deprecated
 
 ### Removed
@@ -24,6 +26,7 @@ See also the [CHANGELOG](https://github.com/cucumber/cucumber-jvm/blob/master/CH
 ### Changed
 
 - [Internal] Implement `HookDefinition.getHookType` and unimplement `Location.getLocation`. No impact expected for users.
+([#401](https://github.com/cucumber/cucumber-jvm-scala/issues/401))
 
 ## [8.29.0] (2025-07-15)
 

cucumber-scala/src/main/scala/io/cucumber/scala/Utils.scala

@@ -9,9 +9,25 @@ private[scala] object Utils {
     val frames = Thread.currentThread().getStackTrace
     val currentClass = self.getClass.getName
     // Note: the -1 check is here for Scala < 2.13 and objects
-    frames.reverse
-      .find(f => f.getClassName == currentClass && f.getLineNumber != -1)
-      .get
+    findLast(frames)(f =>
+      f.getClassName == currentClass && f.getLineNumber != -1
+    ) match {
+      case Some(stackFrame) => stackFrame
+      case None             =>
+        throw new IllegalStateException(
+          s"Not able to find stack frame for $currentClass"
+        )
+    }
+  }
+
+  // Copied from Scala 2.13 library, not available in 2.12 nor in scala-collections-compat
+  private def findLast[A](seq: Array[A])(p: A => Boolean): Option[A] = {
+    val it = seq.reverseIterator
+    while (it.hasNext) {
+      val elem = it.next()
+      if (p(elem)) return Some(elem)
+    }
+    None
   }
 
 }