Fix tests

jfrog · Dec 24, 2024 · d021068 · d021068
1 parent b2f2cd1
commit d021068
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/artifactory/utils/commandsummary/utils_test.go b/artifactory/utils/commandsummary/utils_test.go
@@ -1,9 +1,9 @@
 package commandsummary
 
 import (
-	"os"
 	"testing"
 
+	testsutils "github.com/jfrog/jfrog-client-go/utils/tests"
 	"github.com/stretchr/testify/assert"
 )
 
@@ -103,8 +103,8 @@ func TestAddGitHubTrackingToUrl(t *testing.T) {
 	for _, test := range tests {
 		t.Run(test.name, func(t *testing.T) {
 			// Set up the environment variable
-			os.Setenv("GITHUB_WORKFLOW", test.envValue)
-			defer os.Unsetenv("GITHUB_WORKFLOW")
+			cleanup := testsutils.SetEnvWithCallbackAndAssert(t, "GITHUB_WORKFLOW", test.envValue)
+			defer cleanup()
 
 			// Call the function
 			result, err := addGitHubTrackingToUrl(test.url, test.section)

diff --git a/artifactory/utils/testdata/command_summaries/extended/maven-nested-module.md b/artifactory/utils/testdata/command_summaries/extended/maven-nested-module.md
@@ -12,7 +12,7 @@
 📦 libs-release
 └── 📁 path
     └── 📁 to
-        └── <a href='https://myplatform.com/ui/repos/tree/General/libs-release/path/to/artifact2?clearFilter=true' target="_blank">artifact2</a>
+        └── <a href='https://myplatform.com/ui/repos/tree/General/libs-release/path/to/artifact2?clearFilter=true&gh_job_id=JFrog+CLI+Core+Tests&gh_section=packages' target="_blank">artifact2</a>
 
 </details></pre>