Merge #1321

1321: Run all unit tests in buildkite r=charleskawczynski a=charleskawczynski This PR: - moves all of the unit tests into buildkite. - Deletes the `test/gpu/device.jl` file, since it's from a deprecated module - Moves `test/gpu/data.jl → test/DataLayouts/cuda.jl` - Attempts to improve the grouping and labeling of the tests in the buildkite pipeline I've also incorporated #1319. This is a pretty big step towards #837. Making bors not depend on GHA would effectively close the issue. Co-authored-by: Charles Kawczynski <[email protected]>
CliMA · Jun 16, 2023 · 046c63f · 046c63f
2 parents 534c678 + b164a82
commit 046c63f
Show file tree

Hide file tree

Showing 12 changed files with 572 additions and 358 deletions.
diff --git a/.buildkite/pipeline.yml b/.buildkite/pipeline.yml
diff --git a/perf/allocs.jl b/perf/allocs.jl
diff --git a/perf/allocs_per_case.jl b/perf/allocs_per_case.jl
diff --git a/test/gpu/data.jl → test/DataLayouts/cuda.jl b/test/gpu/data.jl → test/DataLayouts/cuda.jl
diff --git a/test/Limiters/distributed.jl b/test/Limiters/distributed.jl
diff --git a/test/Operators/finitedifference/column_benchmark_utils.jl b/test/Operators/finitedifference/column_benchmark_utils.jl
@@ -414,8 +414,8 @@ end
 
 function test_results(t_ave)
     # If these tests fail, just update the numbers (or the
-    # buffer) so long its not an agregious regression.
-    buffer = 1.7
+    # buffer) so long its not an egregious regression.
+    buffer = 2
     ns = 1
     μs = 10^3
     @test t_ave[(:no_h_space, op_GradientF2C!, :none)] < 559.8423*ns*buffer

diff --git a/test/Operators/spectralelement/run_sphere_geometry_distributed.jl b/test/Operators/spectralelement/run_sphere_geometry_distributed.jl
diff --git a/test/recursive.jl → test/RecursiveApply/recursive_apply.jl b/test/recursive.jl → test/RecursiveApply/recursive_apply.jl
diff --git a/test/Spaces/distributed.jl b/test/Spaces/distributed.jl
diff --git a/test/Topologies/distributed.jl b/test/Topologies/distributed.jl
diff --git a/test/gpu/device.jl b/test/gpu/device.jl
diff --git a/test/runtests.jl b/test/runtests.jl
@@ -11,7 +11,7 @@ Stacktrace:
  [1] pkgerror(msg::String)
 =#
 if !Sys.iswindows()
-    @safetestset "Recursive" begin @time include("recursive.jl") end
+    @safetestset "Recursive" begin @time include("RecursiveApply/recursive_apply.jl") end
     @safetestset "PlusHalf" begin @time include("Utilities/plushalf.jl") end
 
     @safetestset "DataLayouts 0D" begin @time include("DataLayouts/data0d.jl") end
@@ -99,8 +99,7 @@ if !Sys.iswindows()
 end
 if "CUDA" in ARGS
     @safetestset "GPU - cuda" begin @time include("gpu/cuda.jl") end
-    @safetestset "GPU - data" begin @time include("gpu/data.jl") end
-    @safetestset "GPU - device" begin @time include("gpu/device.jl") end
+    @safetestset "GPU - data" begin @time include("test/DataLayouts/cuda.jl") end
     @safetestset "Spaces - serial CUDA DSS" begin @time include("Spaces/ddss1.jl") end
     @safetestset "Spaces - serial CUDA DSS on CubedSphere" begin @time include("Spaces/ddss1_cs.jl") end
     @safetestset "Operators - spectral element CUDA" begin @time include("Operators/spectralelement/rectilinear_cuda.jl") end