Skip to content

Commit b3e3cd3

Browse files
authored
Simplify the way that the LSF tests are launched (#21)
Partially addresses @bjarthur's comment here: #10 (comment)
1 parent 28cc49f commit b3e3cd3

File tree

1 file changed

+6
-17
lines changed

1 file changed

+6
-17
lines changed

test/runtests.jl

Lines changed: 6 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -13,23 +13,12 @@ using Test: @testset, @test, @test_skip
1313
# LSF:
1414
using LSFClusterManager: addprocs_lsf, LSFManager
1515

16-
const test_args = lowercase.(strip.(ARGS))
17-
18-
@info "" test_args
19-
2016
lsf_is_installed() = !isnothing(Sys.which("bsub"))
2117

2218
@testset "LSFClusterManager.jl" begin
23-
if lsf_is_installed()
24-
@info "Running the LSF tests..." Sys.which("bsub")
25-
include("lsf.jl")
26-
else
27-
if "lsf" in test_args
28-
@error "ERROR: The LSF tests were explicitly requested in ARGS, but bsub was not found, so the LSF tests cannot be run" Sys.which("bsub") test_args
29-
@test false
30-
else
31-
@warn "bsub was not found - LSF tests will be skipped" Sys.which("bsub")
32-
@test_skip false
33-
end
34-
end
35-
end # @testset
19+
@info "" Sys.which("bsub")
20+
21+
@test lsf_is_installed()
22+
23+
include("lsf.jl")
24+
end

0 commit comments

Comments
 (0)