Merge pull request #1275 from bjaglin/revert-breaking-change

revert unnecessary breaking change in testkit

Author: github-brice-jaglin

build.sbt

@@ -219,12 +219,8 @@ lazy val unit = project
 
       put(
         "inputClasspath",
-        fullClasspath.in(testsInput, Compile).value.map(_.data)
-      )
-
-      put(
-        "semanticdbRootDirectory",
-        Seq(semanticdbTargetRoot.in(testsInput, Compile).value)
+        fullClasspath.in(testsInput, Compile).value.map(_.data) :+
+          semanticdbTargetRoot.in(testsInput, Compile).value
       )
       put(
         "inputSourceDirectories",

scalafix-testkit/src/main/scala/scalafix/testkit/TestkitProperties.scala

@@ -96,9 +96,7 @@ object TestkitProperties {
         case None => Nil
       }
       new TestkitProperties(
-        Classpath(sprops("semanticdbRootDirectory")) ++ Classpath(
-          sprops("inputClasspath")
-        ),
+        Classpath(sprops("inputClasspath")),
         Classpath(sprops("inputSourceDirectories")).entries,
         Classpath(sprops("outputSourceDirectories")).entries,
         sourceroot,

scalafix-tests/unit/src/test/scala/scalafix/tests/cli/CliSemanticSuite.scala

@@ -170,7 +170,7 @@ class CliSemanticSuite extends BaseCliSuite {
   checkSemantic(
     name = "-P:semanticdb:targetroot",
     args = {
-      val (targetroot :: classDir :: Nil, jars) =
+      val (_ :: targetroot :: Nil, jars) =
         props.inputClasspath.entries.partition(_.isDirectory)
       Array(
         s"--scalacOptions",