Merge pull request #4 from GustavoCaso/use-t-fatal-instead-of-t-error

GustavoCaso · web-flow · commit f2679b8ddf5c · 2025-08-11T17:19:41.000+02:00
use t.Fatal or t.Fatalf instead of t.Error or t.Errorf
diff --git a/.golangci.yml b/.golangci.yml
@@ -36,7 +36,7 @@ formatters:
       # with the given prefixes are grouped after 3rd-party packages.
       # Default: []
       local-prefixes:
-        - github.com/my/project
+        - github.com/GustavoCaso/meeseeks
 
     golines:
       # Target maximum line length.
@@ -106,7 +106,7 @@ linters:
     - sqlclosecheck # checks that sql.Rows and sql.Stmt are closed
     - staticcheck # is a go vet on steroids, applying a ton of static analysis checks
     - testableexamples # checks if examples are testable (have an expected output)
-    - testifylint # checks usage of github.com/stretchr/testify
+    # - testifylint # checks usage of github.com/stretchr/testify
     # - testpackage # makes you use a separate _test package
     - tparallel # detects inappropriate usage of t.Parallel() method in your Go test codes
     - unconvert # removes unnecessary type conversions
diff --git a/cmd/meeseeks/main_test.go b/cmd/meeseeks/main_test.go
@@ -20,7 +20,7 @@ func TestMainCommands(t *testing.T) {
 				output := stdoutBuf.String() + stderrBuf.String()
 
 				if exitCode != 1 {
-					t.Errorf("Expected exit code 1, got %d", exitCode)
+					t.Fatalf("Expected exit code 1, got %d", exitCode)
 				}
 
 				expectedMessages := []string{
@@ -34,7 +34,7 @@ func TestMainCommands(t *testing.T) {
 
 				for _, msg := range expectedMessages {
 					if !strings.Contains(output, msg) {
-						t.Errorf("Expected output to contain %q, got %q", msg, output)
+						t.Fatalf("Expected output to contain %q, got %q", msg, output)
 					}
 				}
 			},
@@ -47,11 +47,11 @@ func TestMainCommands(t *testing.T) {
 				output := stdoutBuf.String() + stderrBuf.String()
 
 				if exitCode != 1 {
-					t.Errorf("Expected exit code 1, got %d", exitCode)
+					t.Fatalf("Expected exit code 1, got %d", exitCode)
 				}
 
 				if !strings.Contains(output, "Unknown command: unknown") {
-					t.Errorf("Expected unknown command error, got %q", output)
+					t.Fatalf("Expected unknown command error, got %q", output)
 				}
 			},
 		},
@@ -63,11 +63,11 @@ func TestMainCommands(t *testing.T) {
 				output := stdoutBuf.String() + stderrBuf.String()
 
 				if exitCode != 0 {
-					t.Errorf("Expected exit code %d, got %d", 0, exitCode)
+					t.Fatalf("Expected exit code %d, got %d", 0, exitCode)
 				}
 
 				if !strings.Contains(output, "meeseeks version 1.0.0") {
-					t.Errorf("Expected output to contain %q, got %q", "meeseeks version 1.0.0", output)
+					t.Fatalf("Expected output to contain %q, got %q", "meeseeks version 1.0.0", output)
 				}
 			},
 		},
diff --git a/cmd/meeseeks/run_test.go b/cmd/meeseeks/run_test.go
@@ -35,11 +35,11 @@ func TestRunCommand_ConfigValidation(t *testing.T) {
 			output := stdoutBuf.String() + stderrBuf.String()
 
 			if exitCode != tt.expectedExit {
-				t.Errorf("Expected exit code %d, got %d", tt.expectedExit, exitCode)
+				t.Fatalf("Expected exit code %d, got %d", tt.expectedExit, exitCode)
 			}
 
 			if !strings.Contains(output, tt.errorMessage) {
-				t.Errorf("Expected error message %q, got %q", tt.errorMessage, output)
+				t.Fatalf("Expected error message %q, got %q", tt.errorMessage, output)
 			}
 		})
 	}
@@ -97,7 +97,7 @@ func TestRunCommand_Foreground(t *testing.T) {
 
 	err = cmd.Start()
 	if err != nil {
-		t.Errorf("error starting in foreground: %s", err.Error())
+		t.Fatalf("error starting in foreground: %s", err.Error())
 	}
 
 	// Start goroutines to read from pipes and populate buffers
@@ -129,13 +129,13 @@ func TestRunCommand_Foreground(t *testing.T) {
 	if err != nil && !strings.Contains(err.Error(), "signal") &&
 		!strings.Contains(err.Error(), "interrupt") &&
 		!strings.Contains(err.Error(), "context deadline") {
-		t.Errorf("Unexpected error (ignoring interrupt/timeout): %v", err)
+		t.Fatalf("Unexpected error (ignoring interrupt/timeout): %v", err)
 	}
 
 	output := stdout.String() + stderr.String()
 	expected := "Started meeseeks program_count=1"
 	if !strings.Contains(output, expected) {
-		t.Errorf("Expected output to contain %q, got %q", expected, output)
+		t.Fatalf("Expected output to contain %q, got %q", expected, output)
 	}
 }
 
@@ -179,17 +179,17 @@ func TestRunCommand_Detached(t *testing.T) {
 
 	output := stdout.String() + stderr.String()
 	if !strings.Contains(output, "Started meeseeks (detached)") {
-		t.Errorf("Expected `Started meeseeks (detached)`, got: %q", output)
+		t.Fatalf("Expected `Started meeseeks (detached)`, got: %q", output)
 	}
 
 	if _, err := os.Stat(expectedPidFile); os.IsNotExist(err) {
-		t.Errorf("PID file was not created at %s", expectedPidFile)
+		t.Fatalf("PID file was not created at %s", expectedPidFile)
 	}
 
 	time.Sleep(500 * time.Millisecond)
 
 	if _, err := os.Stat(expectedSocketPath); os.IsNotExist(err) {
-		t.Errorf("Socket file was not created at %s", expectedSocketPath)
+		t.Fatalf("Socket file was not created at %s", expectedSocketPath)
 	}
 
 	var stdoutBuf, stderrBuf bytes.Buffer
@@ -201,25 +201,25 @@ func TestRunCommand_Detached(t *testing.T) {
 		5*time.Second,
 	)
 	if exitCode != 0 {
-		t.Errorf("Expected exit code %d, got %d", 0, exitCode)
+		t.Fatalf("Expected exit code %d, got %d", 0, exitCode)
 	}
 	statusOutput := stdoutBuf.String() + stderrBuf.String()
 
 	if strings.Contains(statusOutput, "meeseeks server not running") {
-		t.Errorf("Status command could not connect to daemon: %q", statusOutput)
+		t.Fatalf("Status command could not connect to daemon: %q", statusOutput)
 	}
 
 	exitCode = runCLICommand(t, []string{"exit"}, nil, nil, 5*time.Second)
 	if exitCode != 0 {
-		t.Errorf("Expected exit code %d, got %d", 0, exitCode)
+		t.Fatalf("Expected exit code %d, got %d", 0, exitCode)
 	}
 
 	if _, err := os.Stat(expectedPidFile); !os.IsNotExist(err) {
-		t.Error("PID file still exists. Stoping meeseeks should remove the PID file")
+		t.Fatal("PID file still exists. Stoping meeseeks should remove the PID file")
 	}
 
 	if _, err := os.Stat(expectedSocketPath); !os.IsNotExist(err) {
-		t.Error("Socket file still exists. Stoping meeseeks should remove the Socket file")
+		t.Fatal("Socket file still exists. Stoping meeseeks should remove the Socket file")
 	}
 
 	var stdoutBuf2, stderrBuf2 bytes.Buffer
@@ -231,11 +231,11 @@ func TestRunCommand_Detached(t *testing.T) {
 		5*time.Second,
 	)
 	if exitCode != 1 {
-		t.Errorf("Expected exit code %d, got %d", 1, exitCode)
+		t.Fatalf("Expected exit code %d, got %d", 1, exitCode)
 	}
 	statusOutput = stdoutBuf2.String() + stderrBuf2.String()
 
 	if !strings.Contains(statusOutput, "meeseeks server not running") {
-		t.Error("Status command should not work after exiting meeseeks")
+		t.Fatal("Status command should not work after exiting meeseeks")
 	}
 }
diff --git a/cmd/meeseeks/test_helpers.go b/cmd/meeseeks/test_helpers.go
@@ -141,11 +141,11 @@ func ensureNoDaemonRunning(t *testing.T) {
 	expectedSocketPath := getSocketPath()
 
 	if _, err := os.Stat(expectedPidFile); !os.IsNotExist(err) {
-		t.Error("PID file still exists. Stoping meeseeks should remove the PID file")
+		t.Fatal("PID file still exists. Stoping meeseeks should remove the PID file")
 	}
 
 	if _, err := os.Stat(expectedSocketPath); !os.IsNotExist(err) {
-		t.Error("Socket file still exists. Stoping meeseeks should remove the Socket file")
+		t.Fatal("Socket file still exists. Stoping meeseeks should remove the Socket file")
 	}
 }
 
@@ -166,11 +166,11 @@ func runCommandTests(t *testing.T, tests []commandTestCase) {
 			output := stdoutBuf.String() + stderrBuf.String()
 
 			if exitCode != tt.expectedExit {
-				t.Errorf("Expected exit code %d, got %d", tt.expectedExit, exitCode)
+				t.Fatalf("Expected exit code %d, got %d", tt.expectedExit, exitCode)
 			}
 
 			if !strings.Contains(output, tt.shouldContain) {
-				t.Errorf("Expected output to contain %q, got %q", tt.shouldContain, output)
+				t.Fatalf("Expected output to contain %q, got %q", tt.shouldContain, output)
 			}
 		})
 	}
@@ -183,12 +183,12 @@ func testCommandHelp(t *testing.T, command string, expectedMessages []string) {
 	output := stdoutBuf.String() + stderrBuf.String()
 
 	if exitCode != 0 {
-		t.Errorf("Expected exit code 0 for help, got %d", exitCode)
+		t.Fatalf("Expected exit code 0 for help, got %d", exitCode)
 	}
 
 	for _, msg := range expectedMessages {
 		if !strings.Contains(output, msg) {
-			t.Errorf("Expected help output to contain %q, got %q", msg, output)
+			t.Fatalf("Expected help output to contain %q, got %q", msg, output)
 		}
 	}
 }
diff --git a/cmd/meeseeks/utils_test.go b/cmd/meeseeks/utils_test.go
@@ -63,22 +63,20 @@ func TestCreateProgramFromConfig(t *testing.T) {
 
 			if tt.expectError {
 				if err == nil {
-					t.Errorf("Expected error but got none")
-					return
+					t.Fatalf("Expected error but got none")
 				}
 				if !strings.Contains(err.Error(), tt.errorMessage) {
-					t.Errorf("Expected error containing %q, got %q", tt.errorMessage, err.Error())
+					t.Fatalf("Expected error containing %q, got %q", tt.errorMessage, err.Error())
 				}
 				return
 			}
 
 			if err != nil {
-				t.Errorf("Unexpected error: %v", err)
-				return
+				t.Fatalf("Unexpected error: %v", err)
 			}
 
 			if prog.Name() != tt.expectedName {
-				t.Errorf("Expected name %q, got %q", tt.expectedName, prog.Name())
+				t.Fatalf("Expected name %q, got %q", tt.expectedName, prog.Name())
 			}
 		})
 	}
@@ -91,7 +89,7 @@ func TestGetMeeseeksDir(t *testing.T) {
 
 		result := getMeeseeksDir()
 		if result != customDir {
-			t.Errorf("Expected %q, got %q", customDir, result)
+			t.Fatalf("Expected %q, got %q", customDir, result)
 		}
 	})
 
@@ -101,7 +99,7 @@ func TestGetMeeseeksDir(t *testing.T) {
 		expected := filepath.Join(homeDir, ".meeseeks")
 
 		if result != expected {
-			t.Errorf("Expected %q, got %q", expected, result)
+			t.Fatalf("Expected %q, got %q", expected, result)
 		}
 	})
 }
@@ -126,7 +124,7 @@ func TestPathFunctions(t *testing.T) {
 			t.Run(tt.name, func(t *testing.T) {
 				result := tt.function()
 				if result != tt.expected {
-					t.Errorf("Expected %q, got %q", tt.expected, result)
+					t.Fatalf("Expected %q, got %q", tt.expected, result)
 				}
 			})
 		}
@@ -151,7 +149,7 @@ func TestPathFunctions(t *testing.T) {
 			t.Run(tt.name, func(t *testing.T) {
 				result := tt.function()
 				if result != tt.expected {
-					t.Errorf("Expected %q, got %q", tt.expected, result)
+					t.Fatalf("Expected %q, got %q", tt.expected, result)
 				}
 			})
 		}
diff --git a/internal/config/config_test.go b/internal/config/config_test.go
@@ -186,22 +186,22 @@ func TestLoadConfig(t *testing.T) {
 
 			if tt.wantErr {
 				if err == nil {
-					t.Errorf("LoadConfig() expected error but got none")
+					t.Fatalf("LoadConfig() expected error but got none")
 					return
 				}
 				if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {
-					t.Errorf("LoadConfig() error = %q, want error containing %q", err.Error(), tt.errContains)
+					t.Fatalf("LoadConfig() error = %q, want error containing %q", err.Error(), tt.errContains)
 				}
 				return
 			}
 
 			if err != nil {
-				t.Errorf("LoadConfig() unexpected error = %v", err)
+				t.Fatalf("LoadConfig() unexpected error = %v", err)
 				return
 			}
 
 			if !configsEqual(config, tt.expected) {
-				t.Errorf("LoadConfig() = %+v, want %+v", config, tt.expected)
+				t.Fatalf("LoadConfig() = %+v, want %+v", config, tt.expected)
 			}
 		})
 	}
@@ -261,22 +261,20 @@ func TestProgramConfig_GetInterval(t *testing.T) {
 
 			if tt.wantErr {
 				if err == nil {
-					t.Errorf("GetInterval() expected error but got none")
-					return
+					t.Fatalf("GetInterval() expected error but got none")
 				}
 				if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {
-					t.Errorf("GetInterval() error = %q, want error containing %q", err.Error(), tt.errContains)
+					t.Fatalf("GetInterval() error = %q, want error containing %q", err.Error(), tt.errContains)
 				}
 				return
 			}
 
 			if err != nil {
-				t.Errorf("GetInterval() unexpected error = %v", err)
-				return
+				t.Fatalf("GetInterval() unexpected error = %v", err)
 			}
 
 			if duration != tt.expected {
-				t.Errorf("GetInterval() = %v, want %v", duration, tt.expected)
+				t.Fatalf("GetInterval() = %v, want %v", duration, tt.expected)
 			}
 		})
 	}
@@ -361,17 +359,16 @@ func TestConfig_Validate(t *testing.T) {
 
 			if tt.wantErr {
 				if err == nil {
-					t.Errorf("Validate() expected error but got none")
-					return
+					t.Fatalf("Validate() expected error but got none")
 				}
 				if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {
-					t.Errorf("Validate() error = %q, want error containing %q", err.Error(), tt.errContains)
+					t.Fatalf("Validate() error = %q, want error containing %q", err.Error(), tt.errContains)
 				}
 				return
 			}
 
 			if err != nil {
-				t.Errorf("Validate() unexpected error = %v", err)
+				t.Fatalf("Validate() unexpected error = %v", err)
 			}
 		})
 	}
diff --git a/internal/server/server_test.go b/internal/server/server_test.go
diff --git a/pkg/meeseeks/meeseeks_test.go b/pkg/meeseeks/meeseeks_test.go
diff --git a/pkg/program/program_test.go b/pkg/program/program_test.go

Original file line number	Diff line number	Diff line change
`@@ -20,7 +20,7 @@ func TestMainCommands(t *testing.T) {`
`20`	`20`	`output := stdoutBuf.String() + stderrBuf.String()`
`21`	`21`
`22`	`22`	`if exitCode != 1 {`
`23`		`- t.Errorf("Expected exit code 1, got %d", exitCode)`
	`23`	`+ t.Fatalf("Expected exit code 1, got %d", exitCode)`
`24`	`24`	`}`
`25`	`25`
`26`	`26`	`expectedMessages := []string{`
`@@ -34,7 +34,7 @@ func TestMainCommands(t *testing.T) {`
`34`	`34`
`35`	`35`	`for _, msg := range expectedMessages {`
`36`	`36`	`if !strings.Contains(output, msg) {`
`37`		`- t.Errorf("Expected output to contain %q, got %q", msg, output)`
	`37`	`+ t.Fatalf("Expected output to contain %q, got %q", msg, output)`
`38`	`38`	`}`
`39`	`39`	`}`
`40`	`40`	`},`
`@@ -47,11 +47,11 @@ func TestMainCommands(t *testing.T) {`
`47`	`47`	`output := stdoutBuf.String() + stderrBuf.String()`
`48`	`48`
`49`	`49`	`if exitCode != 1 {`
`50`		`- t.Errorf("Expected exit code 1, got %d", exitCode)`
	`50`	`+ t.Fatalf("Expected exit code 1, got %d", exitCode)`
`51`	`51`	`}`
`52`	`52`
`53`	`53`	`if !strings.Contains(output, "Unknown command: unknown") {`
`54`		`- t.Errorf("Expected unknown command error, got %q", output)`
	`54`	`+ t.Fatalf("Expected unknown command error, got %q", output)`
`55`	`55`	`}`
`56`	`56`	`},`
`57`	`57`	`},`
`@@ -63,11 +63,11 @@ func TestMainCommands(t *testing.T) {`
`63`	`63`	`output := stdoutBuf.String() + stderrBuf.String()`
`64`	`64`
`65`	`65`	`if exitCode != 0 {`
`66`		`- t.Errorf("Expected exit code %d, got %d", 0, exitCode)`
	`66`	`+ t.Fatalf("Expected exit code %d, got %d", 0, exitCode)`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`if !strings.Contains(output, "meeseeks version 1.0.0") {`
`70`		`- t.Errorf("Expected output to contain %q, got %q", "meeseeks version 1.0.0", output)`
	`70`	`+ t.Fatalf("Expected output to contain %q, got %q", "meeseeks version 1.0.0", output)`
`71`	`71`	`}`
`72`	`72`	`},`
`73`	`73`	`},`
Original file line number	Diff line number	Diff line change
`@@ -35,11 +35,11 @@ func TestRunCommand_ConfigValidation(t *testing.T) {`
`35`	`35`	`output := stdoutBuf.String() + stderrBuf.String()`
`36`	`36`
`37`	`37`	`if exitCode != tt.expectedExit {`
`38`		`- t.Errorf("Expected exit code %d, got %d", tt.expectedExit, exitCode)`
	`38`	`+ t.Fatalf("Expected exit code %d, got %d", tt.expectedExit, exitCode)`
`39`	`39`	`}`
`40`	`40`
`41`	`41`	`if !strings.Contains(output, tt.errorMessage) {`
`42`		`- t.Errorf("Expected error message %q, got %q", tt.errorMessage, output)`
	`42`	`+ t.Fatalf("Expected error message %q, got %q", tt.errorMessage, output)`
`43`	`43`	`}`
`44`	`44`	`})`
`45`	`45`	`}`
`@@ -97,7 +97,7 @@ func TestRunCommand_Foreground(t *testing.T) {`
`97`	`97`
`98`	`98`	`err = cmd.Start()`
`99`	`99`	`if err != nil {`
`100`		`- t.Errorf("error starting in foreground: %s", err.Error())`
	`100`	`+ t.Fatalf("error starting in foreground: %s", err.Error())`
`101`	`101`	`}`
`102`	`102`
`103`	`103`	`// Start goroutines to read from pipes and populate buffers`
`@@ -129,13 +129,13 @@ func TestRunCommand_Foreground(t *testing.T) {`
`129`	`129`	`if err != nil && !strings.Contains(err.Error(), "signal") &&`
`130`	`130`	`!strings.Contains(err.Error(), "interrupt") &&`
`131`	`131`	`!strings.Contains(err.Error(), "context deadline") {`
`132`		`- t.Errorf("Unexpected error (ignoring interrupt/timeout): %v", err)`
	`132`	`+ t.Fatalf("Unexpected error (ignoring interrupt/timeout): %v", err)`
`133`	`133`	`}`
`134`	`134`
`135`	`135`	`output := stdout.String() + stderr.String()`
`136`	`136`	`expected := "Started meeseeks program_count=1"`
`137`	`137`	`if !strings.Contains(output, expected) {`
`138`		`- t.Errorf("Expected output to contain %q, got %q", expected, output)`
	`138`	`+ t.Fatalf("Expected output to contain %q, got %q", expected, output)`
`139`	`139`	`}`
`140`	`140`	`}`
`141`	`141`
`@@ -179,17 +179,17 @@ func TestRunCommand_Detached(t *testing.T) {`
`179`	`179`
`180`	`180`	`output := stdout.String() + stderr.String()`
`181`	`181`	`if !strings.Contains(output, "Started meeseeks (detached)") {`
`182`		- t.Errorf("Expected `Started meeseeks (detached)`, got: %q", output)
	`182`	+ t.Fatalf("Expected `Started meeseeks (detached)`, got: %q", output)
`183`	`183`	`}`
`184`	`184`
`185`	`185`	`if _, err := os.Stat(expectedPidFile); os.IsNotExist(err) {`
`186`		`- t.Errorf("PID file was not created at %s", expectedPidFile)`
	`186`	`+ t.Fatalf("PID file was not created at %s", expectedPidFile)`
`187`	`187`	`}`
`188`	`188`
`189`	`189`	`time.Sleep(500 * time.Millisecond)`
`190`	`190`
`191`	`191`	`if _, err := os.Stat(expectedSocketPath); os.IsNotExist(err) {`
`192`		`- t.Errorf("Socket file was not created at %s", expectedSocketPath)`
	`192`	`+ t.Fatalf("Socket file was not created at %s", expectedSocketPath)`
`193`	`193`	`}`
`194`	`194`
`195`	`195`	`var stdoutBuf, stderrBuf bytes.Buffer`
`@@ -201,25 +201,25 @@ func TestRunCommand_Detached(t *testing.T) {`
`201`	`201`	`5*time.Second,`
`202`	`202`	`)`
`203`	`203`	`if exitCode != 0 {`
`204`		`- t.Errorf("Expected exit code %d, got %d", 0, exitCode)`
	`204`	`+ t.Fatalf("Expected exit code %d, got %d", 0, exitCode)`
`205`	`205`	`}`
`206`	`206`	`statusOutput := stdoutBuf.String() + stderrBuf.String()`
`207`	`207`
`208`	`208`	`if strings.Contains(statusOutput, "meeseeks server not running") {`
`209`		`- t.Errorf("Status command could not connect to daemon: %q", statusOutput)`
	`209`	`+ t.Fatalf("Status command could not connect to daemon: %q", statusOutput)`
`210`	`210`	`}`
`211`	`211`
`212`	`212`	`exitCode = runCLICommand(t, []string{"exit"}, nil, nil, 5*time.Second)`
`213`	`213`	`if exitCode != 0 {`
`214`		`- t.Errorf("Expected exit code %d, got %d", 0, exitCode)`
	`214`	`+ t.Fatalf("Expected exit code %d, got %d", 0, exitCode)`
`215`	`215`	`}`
`216`	`216`
`217`	`217`	`if _, err := os.Stat(expectedPidFile); !os.IsNotExist(err) {`
`218`		`- t.Error("PID file still exists. Stoping meeseeks should remove the PID file")`
	`218`	`+ t.Fatal("PID file still exists. Stoping meeseeks should remove the PID file")`
`219`	`219`	`}`
`220`	`220`
`221`	`221`	`if _, err := os.Stat(expectedSocketPath); !os.IsNotExist(err) {`
`222`		`- t.Error("Socket file still exists. Stoping meeseeks should remove the Socket file")`
	`222`	`+ t.Fatal("Socket file still exists. Stoping meeseeks should remove the Socket file")`
`223`	`223`	`}`
`224`	`224`
`225`	`225`	`var stdoutBuf2, stderrBuf2 bytes.Buffer`
`@@ -231,11 +231,11 @@ func TestRunCommand_Detached(t *testing.T) {`
`231`	`231`	`5*time.Second,`
`232`	`232`	`)`
`233`	`233`	`if exitCode != 1 {`
`234`		`- t.Errorf("Expected exit code %d, got %d", 1, exitCode)`
	`234`	`+ t.Fatalf("Expected exit code %d, got %d", 1, exitCode)`
`235`	`235`	`}`
`236`	`236`	`statusOutput = stdoutBuf2.String() + stderrBuf2.String()`
`237`	`237`
`238`	`238`	`if !strings.Contains(statusOutput, "meeseeks server not running") {`
`239`		`- t.Error("Status command should not work after exiting meeseeks")`
	`239`	`+ t.Fatal("Status command should not work after exiting meeseeks")`
`240`	`240`	`}`
`241`	`241`	`}`
Original file line number	Diff line number	Diff line change
`@@ -141,11 +141,11 @@ func ensureNoDaemonRunning(t *testing.T) {`
`141`	`141`	`expectedSocketPath := getSocketPath()`
`142`	`142`
`143`	`143`	`if _, err := os.Stat(expectedPidFile); !os.IsNotExist(err) {`
`144`		`- t.Error("PID file still exists. Stoping meeseeks should remove the PID file")`
	`144`	`+ t.Fatal("PID file still exists. Stoping meeseeks should remove the PID file")`
`145`	`145`	`}`
`146`	`146`
`147`	`147`	`if _, err := os.Stat(expectedSocketPath); !os.IsNotExist(err) {`
`148`		`- t.Error("Socket file still exists. Stoping meeseeks should remove the Socket file")`
	`148`	`+ t.Fatal("Socket file still exists. Stoping meeseeks should remove the Socket file")`
`149`	`149`	`}`
`150`	`150`	`}`
`151`	`151`
`@@ -166,11 +166,11 @@ func runCommandTests(t *testing.T, tests []commandTestCase) {`
`166`	`166`	`output := stdoutBuf.String() + stderrBuf.String()`
`167`	`167`
`168`	`168`	`if exitCode != tt.expectedExit {`
`169`		`- t.Errorf("Expected exit code %d, got %d", tt.expectedExit, exitCode)`
	`169`	`+ t.Fatalf("Expected exit code %d, got %d", tt.expectedExit, exitCode)`
`170`	`170`	`}`
`171`	`171`
`172`	`172`	`if !strings.Contains(output, tt.shouldContain) {`
`173`		`- t.Errorf("Expected output to contain %q, got %q", tt.shouldContain, output)`
	`173`	`+ t.Fatalf("Expected output to contain %q, got %q", tt.shouldContain, output)`
`174`	`174`	`}`
`175`	`175`	`})`
`176`	`176`	`}`
`@@ -183,12 +183,12 @@ func testCommandHelp(t *testing.T, command string, expectedMessages []string) {`
`183`	`183`	`output := stdoutBuf.String() + stderrBuf.String()`
`184`	`184`
`185`	`185`	`if exitCode != 0 {`
`186`		`- t.Errorf("Expected exit code 0 for help, got %d", exitCode)`
	`186`	`+ t.Fatalf("Expected exit code 0 for help, got %d", exitCode)`
`187`	`187`	`}`
`188`	`188`
`189`	`189`	`for _, msg := range expectedMessages {`
`190`	`190`	`if !strings.Contains(output, msg) {`
`191`		`- t.Errorf("Expected help output to contain %q, got %q", msg, output)`
	`191`	`+ t.Fatalf("Expected help output to contain %q, got %q", msg, output)`
`192`	`192`	`}`
`193`	`193`	`}`
`194`	`194`	`}`
Original file line number	Diff line number	Diff line change
`@@ -63,22 +63,20 @@ func TestCreateProgramFromConfig(t *testing.T) {`
`63`	`63`
`64`	`64`	`if tt.expectError {`
`65`	`65`	`if err == nil {`
`66`		`- t.Errorf("Expected error but got none")`
`67`		`- return`
	`66`	`+ t.Fatalf("Expected error but got none")`
`68`	`67`	`}`
`69`	`68`	`if !strings.Contains(err.Error(), tt.errorMessage) {`
`70`		`- t.Errorf("Expected error containing %q, got %q", tt.errorMessage, err.Error())`
	`69`	`+ t.Fatalf("Expected error containing %q, got %q", tt.errorMessage, err.Error())`
`71`	`70`	`}`
`72`	`71`	`return`
`73`	`72`	`}`
`74`	`73`
`75`	`74`	`if err != nil {`
`76`		`- t.Errorf("Unexpected error: %v", err)`
`77`		`- return`
	`75`	`+ t.Fatalf("Unexpected error: %v", err)`
`78`	`76`	`}`
`79`	`77`
`80`	`78`	`if prog.Name() != tt.expectedName {`
`81`		`- t.Errorf("Expected name %q, got %q", tt.expectedName, prog.Name())`
	`79`	`+ t.Fatalf("Expected name %q, got %q", tt.expectedName, prog.Name())`
`82`	`80`	`}`
`83`	`81`	`})`
`84`	`82`	`}`
`@@ -91,7 +89,7 @@ func TestGetMeeseeksDir(t *testing.T) {`
`91`	`89`
`92`	`90`	`result := getMeeseeksDir()`
`93`	`91`	`if result != customDir {`
`94`		`- t.Errorf("Expected %q, got %q", customDir, result)`
	`92`	`+ t.Fatalf("Expected %q, got %q", customDir, result)`
`95`	`93`	`}`
`96`	`94`	`})`
`97`	`95`
`@@ -101,7 +99,7 @@ func TestGetMeeseeksDir(t *testing.T) {`
`101`	`99`	`expected := filepath.Join(homeDir, ".meeseeks")`
`102`	`100`
`103`	`101`	`if result != expected {`
`104`		`- t.Errorf("Expected %q, got %q", expected, result)`
	`102`	`+ t.Fatalf("Expected %q, got %q", expected, result)`
`105`	`103`	`}`
`106`	`104`	`})`
`107`	`105`	`}`
`@@ -126,7 +124,7 @@ func TestPathFunctions(t *testing.T) {`
`126`	`124`	`t.Run(tt.name, func(t *testing.T) {`
`127`	`125`	`result := tt.function()`
`128`	`126`	`if result != tt.expected {`
`129`		`- t.Errorf("Expected %q, got %q", tt.expected, result)`
	`127`	`+ t.Fatalf("Expected %q, got %q", tt.expected, result)`
`130`	`128`	`}`
`131`	`129`	`})`
`132`	`130`	`}`
`@@ -151,7 +149,7 @@ func TestPathFunctions(t *testing.T) {`
`151`	`149`	`t.Run(tt.name, func(t *testing.T) {`
`152`	`150`	`result := tt.function()`
`153`	`151`	`if result != tt.expected {`
`154`		`- t.Errorf("Expected %q, got %q", tt.expected, result)`
	`152`	`+ t.Fatalf("Expected %q, got %q", tt.expected, result)`
`155`	`153`	`}`
`156`	`154`	`})`
`157`	`155`	`}`
Original file line number	Diff line number	Diff line change
`@@ -186,22 +186,22 @@ func TestLoadConfig(t *testing.T) {`
`186`	`186`
`187`	`187`	`if tt.wantErr {`
`188`	`188`	`if err == nil {`
`189`		`- t.Errorf("LoadConfig() expected error but got none")`
	`189`	`+ t.Fatalf("LoadConfig() expected error but got none")`
`190`	`190`	`return`
`191`	`191`	`}`
`192`	`192`	`if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {`
`193`		`- t.Errorf("LoadConfig() error = %q, want error containing %q", err.Error(), tt.errContains)`
	`193`	`+ t.Fatalf("LoadConfig() error = %q, want error containing %q", err.Error(), tt.errContains)`
`194`	`194`	`}`
`195`	`195`	`return`
`196`	`196`	`}`
`197`	`197`
`198`	`198`	`if err != nil {`
`199`		`- t.Errorf("LoadConfig() unexpected error = %v", err)`
	`199`	`+ t.Fatalf("LoadConfig() unexpected error = %v", err)`
`200`	`200`	`return`
`201`	`201`	`}`
`202`	`202`
`203`	`203`	`if !configsEqual(config, tt.expected) {`
`204`		`- t.Errorf("LoadConfig() = %+v, want %+v", config, tt.expected)`
	`204`	`+ t.Fatalf("LoadConfig() = %+v, want %+v", config, tt.expected)`
`205`	`205`	`}`
`206`	`206`	`})`
`207`	`207`	`}`
`@@ -261,22 +261,20 @@ func TestProgramConfig_GetInterval(t *testing.T) {`
`261`	`261`
`262`	`262`	`if tt.wantErr {`
`263`	`263`	`if err == nil {`
`264`		`- t.Errorf("GetInterval() expected error but got none")`
`265`		`- return`
	`264`	`+ t.Fatalf("GetInterval() expected error but got none")`
`266`	`265`	`}`
`267`	`266`	`if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {`
`268`		`- t.Errorf("GetInterval() error = %q, want error containing %q", err.Error(), tt.errContains)`
	`267`	`+ t.Fatalf("GetInterval() error = %q, want error containing %q", err.Error(), tt.errContains)`
`269`	`268`	`}`
`270`	`269`	`return`
`271`	`270`	`}`
`272`	`271`
`273`	`272`	`if err != nil {`
`274`		`- t.Errorf("GetInterval() unexpected error = %v", err)`
`275`		`- return`
	`273`	`+ t.Fatalf("GetInterval() unexpected error = %v", err)`
`276`	`274`	`}`
`277`	`275`
`278`	`276`	`if duration != tt.expected {`
`279`		`- t.Errorf("GetInterval() = %v, want %v", duration, tt.expected)`
	`277`	`+ t.Fatalf("GetInterval() = %v, want %v", duration, tt.expected)`
`280`	`278`	`}`
`281`	`279`	`})`
`282`	`280`	`}`
`@@ -361,17 +359,16 @@ func TestConfig_Validate(t *testing.T) {`
`361`	`359`
`362`	`360`	`if tt.wantErr {`
`363`	`361`	`if err == nil {`
`364`		`- t.Errorf("Validate() expected error but got none")`
`365`		`- return`
	`362`	`+ t.Fatalf("Validate() expected error but got none")`
`366`	`363`	`}`
`367`	`364`	`if tt.errContains != "" && !containsString(err.Error(), tt.errContains) {`
`368`		`- t.Errorf("Validate() error = %q, want error containing %q", err.Error(), tt.errContains)`
	`365`	`+ t.Fatalf("Validate() error = %q, want error containing %q", err.Error(), tt.errContains)`
`369`	`366`	`}`
`370`	`367`	`return`
`371`	`368`	`}`
`372`	`369`
`373`	`370`	`if err != nil {`
`374`		`- t.Errorf("Validate() unexpected error = %v", err)`
	`371`	`+ t.Fatalf("Validate() unexpected error = %v", err)`
`375`	`372`	`}`
`376`	`373`	`})`
`377`	`374`	`}`