Merge pull request #35 from DilumAluthge-forks/dpa/more-testsuite-tweaks

More test suite tweaks

Author: kleinhenz

test/runtests.jl

@@ -1,6 +1,19 @@
-#!/usr/bin/env julia
-
-using Distributed, Test, SlurmClusterManager
+# We don't use `using Foo` here.
+# We either use `using Foo: hello, world`, or we use `import Foo`.
+# https://github.com/JuliaLang/julia/pull/42080
+import SlurmClusterManager
+import Distributed
+import Test
+
+# Bring some names into scope, just for convenience:
+using Test: @testset, @test
+
+const original_JULIA_DEBUG = strip(get(ENV, "JULIA_DEBUG", ""))
+if isempty(original_JULIA_DEBUG)
+  ENV["JULIA_DEBUG"] = "SlurmClusterManager"
+else
+  ENV["JULIA_DEBUG"] = original_JULIA_DEBUG * ",SlurmClusterManager"
+end
 
 # test that slurm is available
 @test !(Sys.which("sinfo") === nothing)

test/script.jl

@@ -20,7 +20,7 @@ const hosts = map(workers()) do id
   remotecall_fetch(() -> gethostname(), id)
 end
 sort!(hosts)
-println(hosts)
+println("List of hosts: ", hosts)
 
 # We don't use `@assert` here, for reason described above.
 if hosts != ["c1", "c1", "c2", "c2"]