Merge pull request #38 from DilumAluthge-forks/dpa/info-statements

Test suite: Change some `println()` statements to `@info` statements

Author: kleinhenz

test/runtests.jl

@@ -18,41 +18,42 @@ end
 @testset "SlurmClusterManager.jl" begin
   # test that slurm is available
   @test !(Sys.which("sinfo") === nothing)
-  
+
   # submit job
   # project should point to top level dir so that SlurmClusterManager is available to script.jl
   project_path = abspath(joinpath(@__DIR__, ".."))
-  println("project_path = $project_path")
+  @info "" project_path
   jobid = withenv("JULIA_PROJECT"=>project_path) do
     strip(read(`sbatch --export=ALL --parsable -n 4 -o test.out script.jl`, String))
   end
-  println("jobid = $jobid")
-  
+  @info "" jobid
+
   # get job state from jobid
   getjobstate = jobid -> begin
     cmd = Cmd(`scontrol show jobid=$jobid`, ignorestatus=true)
     info = read(cmd, String)
     state = match(r"JobState=(\S*)", info)
     return state === nothing ? nothing : state.captures[1]
   end
-  
+
   # wait for job to complete
   status = timedwait(60.0, pollint=1.0) do
     state = getjobstate(jobid)
     state == nothing && return false
-    println("jobstate = $state")
+    @info "jobstate=$(state)"
     return state == "COMPLETED" || state == "FAILED"
   end
-  
+
   state = getjobstate(jobid)
-  
+
   # check that job finished running within timelimit (either completed or failed)
   @test status == :ok
   @test state == "COMPLETED"
-  
+
   # print job output
   output = read("test.out", String)
-  println("script output:")
+  println("# BEGIN script output")
   println(output)
+  println("# END script output")
 
 end # testset "SlurmClusterManager.jl"