test(agent/reaper): Fix restructure issue (#7168)

mafredri · web-flow · commit b3b26a62f2d4 · 2023-04-17T17:39:10.000Z
In #7164 we accidentally removed the "in CI" check, this fixes it.
diff --git a/agent/reaper/reaper_test.go b/agent/reaper/reaper_test.go
@@ -18,7 +18,11 @@ import (
 	"github.com/coder/coder/testutil"
 )
 
-//nolint:paralleltest // Non-parallel subtest.
+// TestReap checks that's the reaper is successfully reaping
+// exited processes and passing the PIDs through the shared
+// channel.
+//
+//nolint:paralleltest
 func TestReap(t *testing.T) {
 	// Don't run the reaper test in CI. It does weird
 	// things like forkexecing which may have unintended
@@ -27,13 +31,6 @@ func TestReap(t *testing.T) {
 		t.Skip("Detected CI, skipping reaper tests")
 	}
 
-	// OK checks that's the reaper is successfully reaping
-	// exited processes and passing the PIDs through the shared
-	// channel.
-}
-
-//nolint:paralleltest // Signal handling.
-func TestReap_OK(t *testing.T) {
 	pids := make(reap.PidCh, 1)
 	err := reaper.ForkReap(
 		reaper.WithPIDCallback(pids),
