Search for name in range of lines and add flexible sourcelinks

Author: romanowski

project/Build.scala

@@ -1244,7 +1244,16 @@ object Build {
       // TODO add versions etc.
       def srcManaged(v: String, s: String) = s"out/bootstrap/stdlib-bootstrapped/scala-$v/src_managed/main/$s-library-src"
       def scalaSrcLink(v: String, s: String) = s"-source-links:$s=github://scala/scala/v$v#src/library"
-      def dottySrcLink(v: String, s: String) = s"-source-links:$s=github://lampepfl/dotty/$v#library/src"
+      def dottySrcLink(v: String, s: String) = {
+        val srcLinks = sys.env.get("GITHUB_SHA") match {
+          case Some(sha) =>
+            s"-source-links:$s=github://${sys.env("GITHUB_REPOSITORY")}/$sha#library/src"
+          case None => s"-source-links:$s=github://lampepfl/dotty/$v#library/src"
+        }
+        println("############ sourcelinks: " + srcLinks)
+        srcLinks
+      }
+
 
       val revision = Seq("-revision", ref, "-project-version", projectVersion)
       val cmd = Seq(

scaladoc/test-source-links/dotty/tools/scaladoc/source-links/RemoteLinksTest.scala

@@ -36,16 +36,23 @@ class RemoteLinksTest:
   @Test
   def runTest =
     assertTrue(mtslAll.nonEmpty)
-    val mtsl = randomGenerator.shuffle(mtslAll).take(20) // take 20 random entries
+    val mtsl = randomGenerator.shuffle(mtslAll.filter(_._1 != "Predef")).take(20) // take 20 random entries
     val pageToMtsl: Map[String, List[(String, String)]] = mtsl.groupMap(_._2.split("#L").head)(v => (v._1, v._2.split("#L").last))
     pageToMtsl.foreach { case (link, members) =>
       try
         val doc = getDocumentFromUrl(link)
-        members.foreach { (member, line) =>
+        println(s"Checking $link")
+        members.foreach { case (member, expectedLine) =>
           if !member.startsWith("given_") then // TODO: handle synthetic givens, for now we disable them from testing
-            val loc = doc.select(s"#LC$line").text
+            val toLine = expectedLine + 3
             val memberToMatch = member.replace("`", "")
-            assertTrue(s"Expected to find $memberToMatch at $link at line $line", loc.contains(memberToMatch))
+            val lineCorrectlyDefined = (expectedLine.toInt until toLine.toInt).exists{ line =>
+              val loc = doc.select(s"#LC$line").text
+
+              loc.contains(memberToMatch)
+            }
+
+            assertTrue(s"Expected to find $memberToMatch at $link at lines $expectedLine-$toLine", lineCorrectlyDefined)
         }
       catch
         case e: java.lang.IllegalArgumentException =>