fix: tests (#1226)

* fix: parsing error test

* fix: SBT scala.js test

* fix: metals test

* fix: SBT scala.js timeout

* revert timeout changes

* cache test

* Revert "cache test"

This reverts commit 587482c.

* scala.js test

* Revert "scala.js test"

This reverts commit e2760f2.

* fix scala.js?

* Revert "fix scala.js?"

This reverts commit ef0e022.

* fix: metals test?

* debug

* fix: scala.js test

There was an error with artifact version suffix.
It was searching for sjs1_2.13 instead of sjs1_2.13.18.

Author: warcholjakub

api/src/main/scala/org/scastie/api/ScalaTarget.scala

@@ -164,9 +164,9 @@ case class Js(scalaVersion: String, scalaJsVersion: String) extends SbtScalaTarg
           case v if v.startsWith("3") =>
             renderLibWithSuffix(lib, s"${scalaJsVersionPrefix}_3.1.3")
           case v if v.startsWith("2.13") =>
-            renderLibWithSuffix(lib, s"${scalaJsVersionPrefix}_2.13")
+            renderLibWithSuffix(lib, s"${scalaJsVersionPrefix}_${BuildInfo.latest213}")
           case v if v.startsWith("2.12") =>
-            renderLibWithSuffix(lib, s"${scalaJsVersionPrefix}_2.12")
+            renderLibWithSuffix(lib, s"${scalaJsVersionPrefix}_${BuildInfo.latest212}")
           case _ => renderSbtCross(lib)
         }
       case _ => renderSbtCross(lib)

metals-runner/src/main/scala/org/scastie/metals/MetalsDispatcher.scala

@@ -93,7 +93,7 @@ class MetalsDispatcher(cache: Cache[IO, ScastieMetalsOptions, ScastiePresentatio
                 compiler <- EitherT.right(
                   cache.getOrUpdateReleasable(configuration) {
                     initializeCompiler(configuration, mtags).map: newPC =>
-                      Releasable(newPC, IO(newPC.shutdown()))
+                      Releasable(newPC, newPC.shutdown())
                   })
               yield compiler
             .value

metals-runner/src/main/scala/org/scastie/metals/ScastiePresentationCompiler.scala

@@ -113,11 +113,15 @@ case class OldScastiePresentationCompiler(underlyingPC: PresentationCompiler) ex
   private val logger = LoggerFactory.getLogger(getClass)
 
   def inifiniteCompilationDetection[Input, Output](task: IO[Output])(input: Input): IO[Output] =
-    task.timeout(30.seconds).onError { case _: TimeoutException =>
-      IO.pure {
-        logger.error("FATAL ERROR: Timeout while computing completions, possible inifinite compilation")
-        logger.error(input.toString)
-      }
+    task.timeout(30.seconds).onError { case e: Throwable =>
+      e match
+        case _: TimeoutException =>
+          IO.pure {
+            logger.error("FATAL ERROR: Timeout while computing completions, possible inifinite compilation")
+            logger.error(input.toString)
+          }
+        case _ =>
+          IO.unit
     }
 
   def complete(offsetParams: ScastieOffsetParams): IO[ScalaCompletionList] =

metals-runner/src/test/scala/org/scastie/metals/MetalsDispatcherTest.scala

@@ -25,7 +25,7 @@ class MetalsDispatcherTest extends CatsEffectSuite with Assertions with CatsEffe
     None
   )
 
-  override val munitTimeout = 60.seconds
+  override val munitTimeout = 120.seconds
 
   test("single thread metals access") {
     cache.use { cache =>

scala-cli-runner/src/test/scala/org/scastie/scalacli/ScalaCliRunnerTest.scala

@@ -110,7 +110,7 @@ class ScalaCliRunnerTest extends TestKit(ActorSystem("ScalaCliRunnerTest")) with
   test("report parsing error 2") {
     runCode("val x = 1a", allowFailure = true)(
       assertCompilationInfo(
-        assertProblemInfo("Invalid literal number, followed by identifier character", Some(1), Some(10), Some(10))(_)
+        assertProblemInfo("end of statement expected but identifier found", Some(1), Some(10), Some(11))(_)
       )
     )
   }