Merge pull request #122 from gzm0/re-enable-large-message-test

Fix #105: Re-enable largeMessageTest

Author: sjrd

seleniumJSEnv/src/test/scala/org/scalajs/jsenv/selenium/SeleniumJSEnvSuite.scala

@@ -10,28 +10,7 @@ import org.junit.runners.Suite
 import org.junit.runner.manipulation.Filter
 import org.junit.runner.Description
 
-@RunWith(classOf[SeleniumJSSuiteRunner])
+@RunWith(classOf[JSEnvSuiteRunner])
 class SeleniumJSSuite extends JSEnvSuite(
   JSEnvSuiteConfig(new SeleniumJSEnv(TestCapabilities.fromEnv))
 )
-
-class SeleniumJSSuiteRunner private (
-    root: Class[_], base: JSEnvSuiteRunner)
-    extends Suite(root, Arrays.asList[Runner](base)) {
-
-  /** Constructor for reflective instantiation via `@RunWith`. */
-  def this(suite: Class[_ <: SeleniumJSSuite]) =
-    this(suite, new JSEnvSuiteRunner(suite))
-
-  // Ignore `largeMessageTest` for chrome.
-  if (TestCapabilities.nameFromEnv == "chrome") {
-    base.filter(new Filter {
-      def describe(): String = "Ignore largeMessageTest"
-
-      def shouldRun(description: Description): Boolean = {
-        description.getMethodName == null ||
-        !description.getMethodName.startsWith("largeMessageTest")
-      }
-    })
-  }
-}