Merge pull request #3101 from guardian/ensure-new-mock-root-folder-is-always-used-for-test-consistency

Ensure `mockRoot` test folder is always cleared before tests run

Author: fthomas

build.sbt

@@ -212,8 +212,6 @@ lazy val core = myCrossProject("core")
     // Uncomment for remote debugging:
     // run / javaOptions += "-agentlib:jdwp=transport=dt_socket,server=y,suspend=y,address=5005",
     Test / fork := true,
-    Test / testOptions +=
-      Tests.Cleanup(() => Path(file(Properties.tmpDir) / "scala-steward").deleteRecursively()),
     Compile / resourceGenerators += Def.task {
       val outDir = (Compile / resourceManaged).value
       def downloadPlugin(v: String): File = {

modules/core/src/test/scala/org/scalasteward/core/mock/MockConfig.scala

@@ -6,6 +6,7 @@ import org.scalasteward.core.application.{Cli, Config}
 
 object MockConfig {
   val mockRoot: File = File.temp / "scala-steward"
+  mockRoot.delete(true) // Ensure folder is cleared of previous test files
   private val args: List[String] = List(
     s"--workspace=$mockRoot/workspace",
     s"--repos-file=$mockRoot/repos.md",