Rework command-line-settings tests

* The original test only coincidentally passed because the broken arg
  parsing didn't actually modify the command-line:

  * The -D argument _should_ have been consumed during argument parsing,
    but wasn't, so remained in residual.
  * The test app would not have outputted the -D argument had argument
    parsing been working, since it only output the app args, not the
    system properties.

* There is now a specific test each for the system properties and the
  residual args.

Signed-off-by: Greg Symons <[email protected]>

Author: gregsymons

src/sbt-test/ash/command-line-settings/build.sbt

@@ -4,11 +4,20 @@ name := "command-line-app"
 
 version := "0.1.0-SNAPSHOT"
 
-TaskKey[Unit]("runCheck") := {
-  val configArg = "-Dconfig.resource=/config.conf"
+TaskKey[Unit]("checkSystemProperty") := {
+  val configArg = "config.resource=/config.conf"
   val cwd = (stagingDirectory in Universal).value
-  val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath, configArg)
+  val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath, s"-D$configArg")
 
   val output = (sys.process.Process(cmd, cwd).!!).replaceAll("\n", "")
-  assert(output.contains(configArg), s"Application did not receive command line configuration resource $configArg")
+  assert(output.contains(configArg), s"Application did not receive system property arg '$configArg'")
+}
+
+TaskKey[Unit]("checkResidual") := {
+  val arg = "residualArg"
+  val cwd = (stagingDirectory in Universal).value
+  val cmd = Seq((cwd / "bin" / packageName.value).getAbsolutePath, arg)
+
+  val output = (sys.process.Process(cmd, cwd).!!).replaceAll("\n", "")
+  assert(output.contains(arg), s"Application did not receive residual arg '$arg'")
 }

src/sbt-test/ash/command-line-settings/src/main/scala/MainApp.scala

@@ -1,3 +1,4 @@
-object MainApp extends App {
+object MainApp extends App {  
+  println(sys.props.collect { case (k, v) => s"$k=$v" } mkString "\n")
   println(args.mkString("|"))
 }

src/sbt-test/ash/command-line-settings/test

@@ -1,4 +1,5 @@
 # Run the staging and check the script.
 > stage
 $ exists target/universal/stage/bin/command-line-app
-> runCheck
+> checkSystemProperty
+> checkResidual