Add tests on Windows CI

Author: tgodzik

.gitattributes

@@ -0,0 +1,2 @@
+* eol=lf
+*.png eol=autocrlf

.github/workflows/ci.yml

@@ -19,6 +19,16 @@ jobs:
       - uses: actions/checkout@v2
       - uses: olafurpg/setup-scala@v10
       - run: sbt '++2.12.13 docs/mdoc'
+  test-windows:
+    name: "Windows 'test'"
+    runs-on: windows-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: olafurpg/setup-scala@v10
+        with:
+          java-version: adopt@1.11
+      - run: sbt test
+        shell: bash
   test:
     name: ${{ matrix.command }} ${{ matrix.java }}
     runs-on: ubuntu-latest

.jvmopts

@@ -1,6 +1,6 @@
 -Xss4m
 -Xms1G
--Xmx4G
+-Xmx2G
 -XX:ReservedCodeCacheSize=1024m
 -XX:+TieredCompilation
 -XX:+CMSClassUnloadingEnabled

cli/src/main/scala/mdoc/internal/cli/CliEnrichments.scala

@@ -14,12 +14,16 @@ import coursierapi.Dependency
 import scala.meta.internal.io.FileIO
 import scala.meta.internal.inputs._
 import java.io.PrintStream
+import scala.util.Try
 
 trait CliEnrichments {
   implicit class XtensionInputMdoc(input: Input) {
     def filename: String =
       input match {
         case s: Input.Slice => s.input.filename
+        case f: Input.File => f.path.filename
+        case v: Input.VirtualFile =>
+          Try(Paths.get(v.path).getFileName().toString()).getOrElse(v.path)
         case _ => input.syntax
       }
     def relativeFilename(sourceroot: AbsolutePath): RelativePath =

mdoc/src/main/scala-2/mdoc/internal/markdown/MarkdownCompiler.scala

@@ -70,8 +70,7 @@ class MarkdownCompiler(
     }
 
   private def toSource(input: Input): BatchSourceFile = {
-    val filename = Paths.get(input.syntax).getFileName.toString
-    new BatchSourceFile(filename, new String(input.chars))
+    new BatchSourceFile(input.filename, new String(input.chars))
   }
 
   def shutdown(): Unit = {

mdoc/src/main/scala-3/mdoc/internal/markdown/MarkdownCompiler.scala

@@ -98,8 +98,7 @@ class MarkdownCompiler(
   }
 
   private def toSource(input: Input): SourceFile = {
-    val filename = Paths.get(input.syntax).getFileName.toString
-    SourceFile.virtual(filename, new String(input.chars))
+    SourceFile.virtual(input.filename, new String(input.chars))
   }
 
   private def toInput(sourceFile: ISourceFile): Input = {

tests/unit/src/main/scala/tests/cli/BaseCliSuite.scala

@@ -26,6 +26,8 @@ abstract class BaseCliSuite extends BaseSuite {
   override def postProcessObtained: Map[Compat.ScalaVersion, String => String] =
     Map(
       Compat.All -> { old =>
+        pprint.log(out().toString())
+        pprint.log(in().toString())
         old
           .replace(out().toString(), "<output>")
           .replace(in().toString(), "<input>")

tests/unit/src/main/scala/tests/markdown/BaseMarkdownSuite.scala

@@ -65,7 +65,7 @@ abstract class BaseMarkdownSuite extends tests.BaseSuite {
     test(name) {
       val reporter = newReporter()
       val context = newContext(settings, reporter)
-      val input = Input.VirtualFile(name.name + ".md", original)
+      val input = Input.VirtualFile(name.name.replace(':', '-') + ".md", original)
       val file = InputFile.fromRelativeFilename(input.path, settings)
       Markdown.toMarkdown(input, context, file, baseSettings.site, reporter, settings)
       assert(reporter.hasErrors, "Expected errors but reporter.hasErrors=false")
@@ -90,7 +90,7 @@ abstract class BaseMarkdownSuite extends tests.BaseSuite {
     test(name) {
       val reporter = newReporter()
       val context = newContext(settings, reporter)
-      val input = Input.VirtualFile(name.name + ".md", original)
+      val input = Input.VirtualFile(name.name.replace(":", "-") + ".md", original)
       val file = InputFile.fromRelativeFilename(input.path, settings)
       val obtained =
         Markdown.toMarkdown(input, context, file, baseSettings.site, reporter, settings)

tests/unit/src/test/scala/tests/imports/FileSuite.scala

@@ -3,6 +3,7 @@ package tests.imports
 import tests.markdown.BaseMarkdownSuite
 import tests.cli.BaseCliSuite
 import scala.meta.io.RelativePath
+import java.io.File
 
 class FileSuite extends BaseCliSuite {
 
@@ -124,7 +125,7 @@ class FileSuite extends BaseCliSuite {
     includeOutputPath = includeOutputPath,
     onStdout = { stdout =>
       assertNoDiff(
-        stdout,
+        stdout.replace('\\', '/'),
         """|info: Compiling 3 files to <output>
            |error: <input>/readme.md:2:14: illegal cyclic dependency. To fix this problem, refactor the code so that no transitive $file imports end up depending on the original file.
            | -- root       --> <input>/readme.md:1
@@ -158,7 +159,7 @@ class FileSuite extends BaseCliSuite {
     includeOutputPath = includeOutputPath,
     onStdout = { stdout =>
       assertNoDiff(
-        stdout,
+        stdout.replace('\\', '/'),
         """|info: Compiling 3 files to <output>
            |error: <input>/hello1.sc:1:23: type mismatch;
            | found   : Int(42)
@@ -260,7 +261,7 @@ class FileSuite extends BaseCliSuite {
     includeOutputPath = includeOutputPath,
     onStdout = { stdout =>
       assertNoDiff(
-        stdout,
+        stdout.replace('\\', '/'),
         """|info: Compiling 2 files to <output>
            |error: <input>/readme.md:2:16: unimports are not supported for $file imports. To fix this problem, remove the unimported symbol.
            |import $file.{ hello0 => _ }
@@ -286,7 +287,7 @@ class FileSuite extends BaseCliSuite {
     includeOutputPath = includeOutputPath,
     onStdout = { stdout =>
       assertNoDiff(
-        stdout,
+        stdout.replace('\\', '/'),
         """|info: Compiling 2 files to <output>
            |error: <input>/readme.md:2:14: wildcards are not supported for $file imports. To fix this problem, explicitly import files using the `import $file.FILENAME` syntax.
            |import $file._
@@ -320,7 +321,7 @@ class FileSuite extends BaseCliSuite {
       // import qualifier. This works because we employ token edit distance to
       // map positions from the instrumented source to the original source.
       assertNoDiff(
-        stdout,
+        stdout.replace('\\', '/'),
         """|info: Compiling 3 files to <output>
            |error: <input>/inner/hello1.sc:2:11: value number is not a member of object $file.hello0
            |val one = hello0.number + 1