Centralize browser indicator, simplify

Author: armanbilge

build.sbt

@@ -66,10 +66,10 @@ ThisBuild / githubWorkflowBuild := Seq(WorkflowStep.Sbt(List("${{ matrix.ci }}")
 
 replaceCommandAlias("ci", ciVariants.mkString("; ", "; ", ""))
 
-addCommandAlias("ciNode", "; set useJSEnv := JSEnv.NodeJS; core/test; core/doc")
-addCommandAlias("ciFirefox", "; set useJSEnv := JSEnv.Firefox; all core/test webworker/test; set useJSEnv := JSEnv.NodeJS")
-addCommandAlias("ciChrome", "; set useJSEnv := JSEnv.Chrome; all core/test webworker/test; set useJSEnv := JSEnv.NodeJS")
-addCommandAlias("ciJSDOMNodeJS", "; set useJSEnv := JSEnv.JSDOMNodeJS; core/test; set useJSEnv := JSEnv.NodeJS")
+addCommandAlias("ciNode", "; set useJSEnv := JSEnv.NodeJS; test; core/doc")
+addCommandAlias("ciFirefox", "; set useJSEnv := JSEnv.Firefox; test; set useJSEnv := JSEnv.NodeJS")
+addCommandAlias("ciChrome", "; set useJSEnv := JSEnv.Chrome; test; set useJSEnv := JSEnv.NodeJS")
+addCommandAlias("ciJSDOMNodeJS", "; set useJSEnv := JSEnv.JSDOMNodeJS; test; set useJSEnv := JSEnv.NodeJS")
 
 // release configuration
 
@@ -161,6 +161,6 @@ lazy val webworker = project
       "org.scalameta" %%% "munit" % MUnitVersion % Test,
     ),
     (Test / test) := (Test / test).dependsOn(Compile / fastOptJS).value,
-    buildInfoKeys := Seq(scalaVersion, baseDirectory, BuildInfoKey("jsEnv" -> useJSEnv.value.toString)),
+    buildInfoKeys := Seq(scalaVersion, baseDirectory, BuildInfoKey("isBrowser" -> useJSEnv.value.isBrowser)),
     buildInfoPackage := "org.scalajs.macrotaskexecutor")
   .enablePlugins(ScalaJSPlugin, BuildInfoPlugin, NoPublishPlugin)

project/JSEnv.scala

@@ -1,7 +1,7 @@
-sealed abstract class JSEnv
+sealed abstract class JSEnv(val isBrowser: Boolean)
 object JSEnv {
-  case object Chrome extends JSEnv
-  case object Firefox extends JSEnv
-  case object JSDOMNodeJS extends JSEnv
-  case object NodeJS extends JSEnv
+  case object Chrome extends JSEnv(true)
+  case object Firefox extends JSEnv(true)
+  case object JSDOMNodeJS extends JSEnv(false)
+  case object NodeJS extends JSEnv(false)
 }

webworker/src/test/scala/org/scalajs/macrotaskexecutor/WebWorkerMacrotaskSuite.scala

@@ -32,7 +32,7 @@ class WebWorkerMacrotaskSuite extends FunSuite {
 
   def targetDir = s"${BuildInfo.baseDirectory}/target/scala-${scalaVersion}"
 
-  override def munitIgnore = !Set("Firefox", "Chrome").contains(BuildInfo.jsEnv)
+  override def munitIgnore = !BuildInfo.isBrowser
 
   test("pass the MacrotaskSuite in a web worker") {
     val p = Promise[Boolean]()