Changed to use parser combinator for parsing JSON in test, fixing link problem when running scalatest tests in scala-js. Please note that for scala-js 2.13.0-M2 to work we'll need to wait until parser combinator 1.0.7 is released, as 1.0.6 jar is empty and expected to be fixed in 1.0.7 (scala/scala-parser-combinators#130).

Author: cheeseng

common-test/src/main/scala/org/scalatest/JSON.scala

@@ -0,0 +1,70 @@
+/*
+ * Copyright 2001-2017 Artima, Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.scalatest
+
+import scala.util.parsing.combinator._
+
+object JSON extends JavaTokenParsers {
+
+  def toJsVal(v: Any): JsVal =
+    v match {
+      case j: JsVal => j
+      case m: Map[String, JsVal] => JsObj(m)
+      case l: List[JsVal] => JsArr(l)
+      case s: String => JsStr(s.take(s.length - 1).drop(1))
+      case n: Double => JsNum(n)
+      case false => JsFalse
+      case true => JsTrue
+      case null => JsNULL
+    }
+
+  sealed trait JsVal extends Any {
+    def value: Any
+    def \(key: String): Option[JsVal] = this.asInstanceOf[JsObj].value.get(key)
+  }
+  case class JsStr(value: String) extends AnyVal with JsVal
+  case class JsObj(value: Map[String, JsVal]) extends AnyVal with JsVal
+  case class JsArr(value: List[JsVal]) extends AnyVal with JsVal
+  case class JsNum(value: Double) extends AnyVal with JsVal
+  case object JsFalse extends JsVal {
+    def value = false
+  }
+  case object JsTrue extends JsVal {
+    def value = true
+  }
+  case object JsNULL extends JsVal {
+    def value = null
+  }
+
+  def obj: Parser[JsObj] =
+    "{" ~> repsep(member, ",") <~ "}" ^^ (v => JsObj(Map.empty[String, JsVal] ++ v))
+  def arr: Parser[List[JsVal]] =
+    "[" ~> repsep(value, ",") <~ "]" ^^ (v => v.map(toJsVal))
+  def member: Parser[(String, JsVal)] =
+    stringLiteral ~ ":" ~ value ^^
+      { case name ~ ":" ~ value => (name.take(name.length - 1).drop(1), toJsVal(value)) }
+  def value: Parser[Any] = (
+    obj
+      | arr
+      | stringLiteral
+      | floatingPointNumber ^^ (_.toDouble)
+      | "null" ^^ (x => null)
+      | "true" ^^ (x => true)
+      | "false" ^^ (x => false)
+    )
+
+  def parseJson(jsStr: String) = parseAll(value, jsStr).get.asInstanceOf[JsVal]
+}

project/GenCommonTestJS.scala

@@ -80,7 +80,8 @@ object GenCommonTestJS {
         "BookPropertyMatchers.scala",
         "EmptyMocks.scala",
         "FileMocks.scala",
-        "StringFixture.scala"
+        "StringFixture.scala",
+        "JSON.scala"
       ), targetDir) ++
     copyDir("common-test/src/main/scala/org/scalatest/path", "org/scalatest/path",
       List("ExampleLikeSpecs.scala"), targetDir)

project/scalatest.scala

@@ -179,7 +179,6 @@ object ScalatestBuild extends Build {
       case Some((2, scalaMajor)) if scalaMajor >= 11 =>
         Seq(
           "org.scala-lang.modules" %% "scala-xml" % "1.0.6",
-          //"org.scala-lang.modules" %% "scala-parser-combinators" % "1.0.6",   This is needed only by SbtCommandParser, but we are not support it currently.
           scalacheckDependency("optional")
         )
       case _ =>
@@ -222,8 +221,7 @@ object ScalatestBuild extends Build {
     Seq(
       "commons-io" % "commons-io" % "1.3.2" % "test",
       "org.eclipse.jetty" % "jetty-server" % "8.1.18.v20150929" % "test",
-      "org.eclipse.jetty" % "jetty-webapp" % "8.1.18.v20150929" % "test",
-      "io.spray" %%  "spray-json" % "1.3.4" % "test"
+      "org.eclipse.jetty" % "jetty-webapp" % "8.1.18.v20150929" % "test"
     )
 
   def scalatestJSLibraryDependencies =
@@ -293,6 +291,7 @@ object ScalatestBuild extends Build {
     .settings(
       projectTitle := "Common test classes used by scalactic and scalatest",
       libraryDependencies += scalacheckDependency("optional"),
+      libraryDependencies += "org.scala-lang.modules" %% "scala-parser-combinators" % "1.0.6",
       libraryDependencies ++= crossBuildTestLibraryDependencies(scalaVersion.value)
     ).dependsOn(scalacticMacro, LocalProject("scalatest"))
 
@@ -301,6 +300,9 @@ object ScalatestBuild extends Build {
     .settings(
       projectTitle := "Common test classes used by scalactic.js and scalatest.js",
       libraryDependencies += scalacheckDependency("optional"),
+      // Currently 1.0.6 for scala-js does not contain anything due to a reported bug: https://github.com/scala/scala-parser-combinators/issues/119
+      // 1.0.5 works for scala 2.12 but not 2.13, we should use 1.0.7 once it is published which suppose to support scala 2.13.
+      libraryDependencies += "org.scala-lang.modules" %%% "scala-parser-combinators" % "1.0.5",
       libraryDependencies ++= crossBuildTestLibraryDependencies(scalaVersion.value),
       sourceGenerators in Compile += {
         Def.task{
@@ -686,7 +688,6 @@ object ScalatestBuild extends Build {
       organization := "org.scalatest",
       libraryDependencies ++= crossBuildLibraryDependencies(scalaVersion.value),
       libraryDependencies += "org.scalacheck" %%% "scalacheck" % scalacheckVersion % "test",
-      libraryDependencies += "io.spray" %%  "spray-json" % "1.3.4" % "optional",
       //jsDependencies += RuntimeDOM % "test",
       scalaJSOptimizerOptions ~= { _.withDisableOptimizer(true) },
       //jsEnv := NodeJSEnv(executable = "node").value,
@@ -855,8 +856,7 @@ object ScalatestBuild extends Build {
       "junit" % "junit" % junitVersion % "optional",
       "org.testng" % "testng" % testngVersion % "optional",
       "org.jmock" % "jmock-legacy" % jmockVersion % "optional",
-      "org.pegdown" % "pegdown" % pegdownVersion % "optional",
-      "io.spray" %%  "spray-json" % "1.3.4" % "optional"
+      "org.pegdown" % "pegdown" % pegdownVersion % "optional"
 
     )