From 1f7e571fb91498fd1105f2b60549a607473d6a2e Mon Sep 17 00:00:00 2001 From: Slug-Boi Date: Tue, 3 Jun 2025 19:59:00 +0200 Subject: [PATCH] fix: env var test updated for panic fix: recover in test --- src/cmd/utils/util_test.go | 25 ++++++++++++++++--------- 1 file changed, 16 insertions(+), 9 deletions(-) diff --git a/src/cmd/utils/util_test.go b/src/cmd/utils/util_test.go index 4a624d9..433adf6 100644 --- a/src/cmd/utils/util_test.go +++ b/src/cmd/utils/util_test.go @@ -173,16 +173,23 @@ func Test_FindAuthorFileEnv(t *testing.T) { // Test Find_authorfile with env var setup() defer teardown() + + // Save original environment variable + originalAuthorFile := os.Getenv("author_file") + + defer func() { + // Reset environment variable + os.Setenv("author_file", originalAuthorFile) + + if r := recover(); r == nil { + t.Errorf("Find_authorfile() did not panic") + } + }() + + // Set an invalid environment variable to trigger panic os.Setenv("author_file", "") - authorfile := utils.Find_authorfile() - configdir, err := os.UserConfigDir() - if err != nil { - t.Fatalf("Failed to get user config directory: %v", err) - } - if authorfile != configdir+"/cocommit/authors.json" { - t.Errorf("Find_authorfile() = %v; want %v", authorfile, configdir+"/cocommit/authors.json") - } - + + utils.Find_authorfile() } func Test_CreateAuthorPanicOnFileError(t *testing.T) {