Partial fix for lampepfl#7113: Add Scala.js handling of @test annotations with arguments

To be able to use the `expected` and `timeout` annotation arguments in
JUnit @test annotations, add handling for these arguments by supplying
them as arguments to the resolved constructor for the JUnit test class.

Enable the JUnitAnnotationsParamTest, which depends on this
functionality.

Author: Lacaranian

compiler/src/dotty/tools/backend/sjs/JUnitBootstrappers.scala

@@ -224,12 +224,16 @@ class JUnitBootstrappers extends MiniPhase {
       val metadata = for (test <- tests) yield {
         val name = Literal(Constant(test.name.toString))
         val ignored = Literal(Constant(test.hasAnnotation(junitdefn.IgnoreAnnotClass)))
-        // TODO Handle @Test annotations with arguments
-        // val reifiedAnnot = New(mydefn.TestAnnotType, test.getAnnotation(mydefn.TestAnnotClass).get.arguments)
         val testAnnot = test.getAnnotation(junitdefn.TestAnnotClass).get
-        if (testAnnot.arguments.nonEmpty)
-          ctx.error("@Test annotations with arguments are not yet supported in Scala.js for dotty", testAnnot.tree.sourcePos)
-        val reifiedAnnot = resolveConstructor(junitdefn.TestAnnotType, Nil)
+
+        val mappedArguments = testAnnot.arguments.collect{
+          // Since classOf[...] in annotations would not be transformed, grab the resulting class constant here
+          case NamedArg(expectedName, typeApply @ TypeApply(classOfIdent, fstArg :: _)) => clsOf(fstArg.tpe)
+          // The only other valid argument to @Test annotations is timeout
+          case NamedArg(timeoutName, timeoutLiteral) => timeoutLiteral
+        }
+
+        val reifiedAnnot = resolveConstructor(junitdefn.TestAnnotType, mappedArguments)
         New(junitdefn.TestMetadataType, List(name, ignored, reifiedAnnot))
       }
       JavaSeqLiteral(metadata, TypeTree(junitdefn.TestMetadataType))

project/Build.scala

@@ -957,6 +957,7 @@ object Build {
         val dir = fetchScalaJSSource.value / "test-suite"
         (
           (dir / "shared/src/test/scala/org/scalajs/testsuite/compiler" ** (("*.scala":FileFilter) -- "RegressionTest.scala" -- "ReflectiveCallTest.scala")).get
+
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/javalib/lang" ** (("*.scala": FileFilter) -- "StringTest.scala")).get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/javalib/io" ** (("*.scala": FileFilter) -- "ByteArrayInputStreamTest.scala" -- "ByteArrayOutputStreamTest.scala" -- "DataInputStreamTest.scala" -- "DataOutputStreamTest.scala" -- "InputStreamTest.scala" -- "OutputStreamWriterTest.scala" -- "PrintStreamTest.scala" -- "CommonStreamsTests.scala")).get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/javalib/math" ** "*.scala").get
@@ -975,7 +976,7 @@ object Build {
             -- "NavigableSetTest.scala" -- "SetTest.scala" -- "SortedMapTest.scala" -- "SortedSetTest.scala" -- "TreeSetTest.scala")).get
 
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/utils" ** "*.scala").get
-          ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/junit" ** (("*.scala": FileFilter) -- "JUnitAnnotationsParamTest.scala")).get
+          ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/junit" ** (("*.scala": FileFilter))).get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/niobuffer" ** (("*.scala": FileFilter)  -- "ByteBufferTest.scala")).get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/niocharset" ** (("*.scala": FileFilter)  -- "BaseCharsetTest.scala" -- "Latin1Test.scala" -- "USASCIITest.scala" -- "UTF16Test.scala" -- "UTF8Test.scala")).get
           ++ (dir / "shared/src/test/scala/org/scalajs/testsuite/scalalib" ** (("*.scala": FileFilter)  -- "ArrayBuilderTest.scala" -- "ClassTagTest.scala" -- "EnumerationTest.scala" -- "RangesTest.scala" -- "SymbolTest.scala")).get