From 07fab41be7d24e790cd37f625f3d7a10363c45ff Mon Sep 17 00:00:00 2001
From: Felix Mulder <felix.mulder@gmail.com>
Date: Thu, 6 Apr 2017 13:46:29 +0200
Subject: Add `Properties` object for dotty testing props and env

---
 compiler/test/dotty/Jars.scala                     | 24 ++++++++----
 compiler/test/dotty/Properties.scala               | 44 ++++++++++++++++++++++
 .../test/dotty/tools/dotc/CompilationTests.scala   |  4 +-
 3 files changed, 62 insertions(+), 10 deletions(-)
 create mode 100644 compiler/test/dotty/Properties.scala

(limited to 'compiler/test/dotty')

diff --git a/compiler/test/dotty/Jars.scala b/compiler/test/dotty/Jars.scala
index 06df9c891..bc000fced 100644
--- a/compiler/test/dotty/Jars.scala
+++ b/compiler/test/dotty/Jars.scala
@@ -2,25 +2,34 @@ package dotty
 
 /** Jars used when compiling test, normally set from the sbt build */
 object Jars {
+  /** Dotty library Jar */
   val dottyLib: String = sys.env.get("DOTTY_LIB")
-    .getOrElse(sys.props("dotty.tests.classes.library"))
+    .getOrElse(Properties.dottyLib)
 
+  /** Dotty Compiler Jar */
   val dottyCompiler: String = sys.env.get("DOTTY_COMPILER")
-    .getOrElse(sys.props("dotty.tests.classes.compiler"))
+    .getOrElse(Properties.dottyCompiler)
 
+  /** Dotty Interfaces Jar */
   val dottyInterfaces: String = sys.env.get("DOTTY_INTERFACE")
-    .getOrElse(sys.props("dotty.tests.classes.interfaces"))
+    .getOrElse(Properties.dottyInterfaces)
 
-  val dottyExtras: List[String] = Option(sys.env.get("DOTTY_EXTRAS")
-    .getOrElse(sys.props("dotty.tests.extraclasspath")))
-    .map(_.split(":").toList).getOrElse(Nil)
+  /** Dotty extras classpath from env or properties */
+  val dottyExtras: List[String] = sys.env.get("DOTTY_EXTRAS")
+    .map(_.split(":").toList).getOrElse(Properties.dottyExtras)
 
+  /** Dotty REPL dependencies */
   val dottyReplDeps: List[String] = dottyLib :: dottyExtras
 
+  /** Dotty test dependencies */
   val dottyTestDeps: List[String] =
     dottyLib :: dottyCompiler :: dottyInterfaces :: dottyExtras
 
-
+  /** Gets the scala 2.* library at runtime, note that doing this is unsafe
+   *  unless you know that the library will be on the classpath of the running
+   *  application. It is currently safe to call this function if the tests are
+   *  run by sbt.
+   */
   def scalaLibraryFromRuntime: String = findJarFromRuntime("scala-library-2.")
 
   private def findJarFromRuntime(partialName: String) = {
@@ -31,5 +40,4 @@ object Jars {
       )
     }
   }
-
 }
diff --git a/compiler/test/dotty/Properties.scala b/compiler/test/dotty/Properties.scala
new file mode 100644
index 000000000..6106c75b9
--- /dev/null
+++ b/compiler/test/dotty/Properties.scala
@@ -0,0 +1,44 @@
+package dotty
+
+/** Runtime properties from defines or environmnent */
+object Properties {
+
+  /** If property is unset or "TRUE" we consider it `true` */
+  private[this] def propIsNullOrTrue(prop: String): Boolean = {
+    val prop = System.getProperty("dotty.tests.interactive")
+    prop == null || prop == "TRUE"
+  }
+
+  /** Are we running on the Drone CI? */
+  val isRunByDrone: Boolean = sys.env.isDefinedAt("DRONE")
+
+  /** Tests should run interactive? */
+  val testsInteractive: Boolean = propIsNullOrTrue("dotty.tests.interactive")
+
+  /** Filter out tests not matching the regex supplied by "dotty.tests.filter"
+   *  define
+   */
+  val testsFilter: Option[String] = sys.props.get("dotty.tests.filter")
+
+  /** Should Unit tests run in safe mode?
+   *
+   *  For run tests this means that we respawn child JVM processes after each
+   *  test, so that they are never reused.
+   */
+  val testsSafeMode: Boolean = sys.props.isDefinedAt("dotty.tests.safemode")
+
+  /** Dotty compiler path provided through define */
+  def dottyCompiler: String = sys.props("dotty.tests.classes.compiler")
+
+  /** Dotty classpath extras provided through define */
+  def dottyExtras: List[String] =
+    Option(sys.props("dotty.tests.extraclasspath"))
+      .map(_.split(":").toList)
+      .getOrElse(Nil)
+
+  /** Dotty interfaces path provided through define */
+  def dottyInterfaces: String = sys.props("dotty.tests.classes.interfaces")
+
+  /** Dotty library path provided through define */
+  def dottyLib: String = sys.props("dotty.tests.classes.library")
+}
diff --git a/compiler/test/dotty/tools/dotc/CompilationTests.scala b/compiler/test/dotty/tools/dotc/CompilationTests.scala
index a3f44c74f..ab7dda850 100644
--- a/compiler/test/dotty/tools/dotc/CompilationTests.scala
+++ b/compiler/test/dotty/tools/dotc/CompilationTests.scala
@@ -16,9 +16,9 @@ class CompilationTests extends SummaryReport with ParallelTesting {
 
   def maxDuration = 180.seconds
   def numberOfSlaves = 5
-  def safeMode = sys.env.get("dotty.tests.safemode").isDefined
+  def safeMode = Properties.testsSafeMode
   def isInteractive = SummaryReport.isInteractive
-  def testFilter = sys.props.get("dotty.tests.filter").map(r => new Regex(r))
+  def testFilter = Properties.testsFilter.map(r => new Regex(r))
 
   // Positive tests ------------------------------------------------------------
 
-- 
cgit v1.2.3