Reduce timeout for unit tests.

jefchien · jefchien · commit 58d31cc5c5c5 · 2025-10-16T17:40:01.000-04:00
diff --git a/Makefile b/Makefile
@@ -199,7 +199,7 @@ lint: install-golangci-lint simple-lint
 	${LINTER} run ./...
 
 test:
-	CGO_ENABLED=0 go test -timeout 15m -coverprofile coverage.txt -failfast ./...
+	CGO_ENABLED=0 go test -timeout 10m -coverprofile coverage.txt -failfast ./...
 
 # List of existing packages with data races
 # TODO: Fix each
diff --git a/plugins/inputs/logfile/tail/tail_test.go b/plugins/inputs/logfile/tail/tail_test.go
@@ -377,6 +377,7 @@ func TestConcurrentLinePoolAccess(t *testing.T) {
 	wg.Add(1)
 	go func() {
 		defer wg.Done()
+		defer close(linesChan)
 		for i := 0; i < numLines; i++ {
 			select {
 			case line := <-tail.Lines:
@@ -386,7 +387,6 @@ func TestConcurrentLinePoolAccess(t *testing.T) {
 				return
 			}
 		}
-		close(linesChan)
 	}()
 
 	numWorkers := 5

Original file line number	Diff line number	Diff line change
`@@ -377,6 +377,7 @@ func TestConcurrentLinePoolAccess(t *testing.T) {`
`377`	`377`	`wg.Add(1)`
`378`	`378`	`go func() {`
`379`	`379`	`defer wg.Done()`
	`380`	`+ defer close(linesChan)`
`380`	`381`	`for i := 0; i < numLines; i++ {`
`381`	`382`	`select {`
`382`	`383`	`case line := <-tail.Lines:`
`@@ -386,7 +387,6 @@ func TestConcurrentLinePoolAccess(t *testing.T) {`
`386`	`387`	`return`
`387`	`388`	`}`
`388`	`389`	`}`
`389`		`- close(linesChan)`
`390`	`390`	`}()`
`391`	`391`
`392`	`392`	`numWorkers := 5`