fix: windows test ci (#941)

I've fixed the windows build test CI mentioned in issue #938, I adjusted
the workflow to run on my branch to make sure it works, all tests
passed.

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit


## Summary by CodeRabbit

* **Tests**
* Improved file compression and extraction tests with additional
assertions and logging for better validation and debugging.
* Enhanced test setup to skip empty directory paths, preventing
unnecessary directory creation.
* Refined shell substitution test to use a valid inner command while
maintaining expected error handling.
* **Chores**
* Updated build and test workflow to use a runner-specific temporary
directory for Go module caching, improving environment isolation.

<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Nitin Kumar <nknitinkumar12384@gmail.com>

Author: claykom

.github/workflows/superfile-build-test.yml

@@ -31,19 +31,25 @@ jobs:
         with:
           path: |
             ~/.cache/go-build
-            ~/go/pkg/mod
+            ${{ runner.temp }}/gomodcache
           key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
           restore-keys: |
             ${{ runner.os }}-go-
 
       - name: Install dependencies
         run: go mod download
+        env:
+          GOMODCACHE: ${{ runner.temp }}/gomodcache
 
       - name: Build
-        run: go build ./...
+        run: go build -v ./...
+        env:
+          GOMODCACHE: ${{ runner.temp }}/gomodcache
 
       - name: Test
         run: go test -v ./...
+        env:
+          GOMODCACHE: ${{ runner.temp }}/gomodcache
 
       - name: Check gofmt (skip on Windows)
         if: runner.os != 'Windows'

src/internal/handle_file_operation_test.go

@@ -125,16 +125,27 @@ func TestCompressSelectedFiles(t *testing.T) {
 				return err == nil
 			}, time.Second, 10*time.Millisecond)
 
+			// Assert zip file exists right after compression
+			require.FileExists(t, zipFile, "Expected zip file does not exist after compression")
+
 			// No-op update to get the filepanel updated
 			// TODO - This should not be needed. Only operation finish SPF should refresh
 			// on its own
 			TeaUpdateWithErrCheck(t, &m, nil)
 
 			require.Greater(t, len(m.getFocusedFilePanel().element), tt.cursorIndexForZip)
-			assert.Equal(t, zipFile, m.getFocusedFilePanel().element[tt.cursorIndexForZip].location,
+			selectedItemLocation := m.getFocusedFilePanel().element[tt.cursorIndexForZip].location
+			// Debug output for panel element
+			t.Logf("Panel element at cursorIndexForZip: %s", selectedItemLocation)
+			assert.Equal(t, zipFile, selectedItemLocation,
 				"%s does not exists at index %d among %v", zipFile, tt.cursorIndexForZip,
 				m.getFocusedFilePanel().element)
 
+			// Ensure we are extracting the zip file, not a directory
+			fileInfo, err := os.Stat(selectedItemLocation)
+			require.NoError(t, err, "Failed to stat panel location before extraction")
+			require.False(t, fileInfo.IsDir(), "Panel location for extraction is a directory, expected a zip file: %s", selectedItemLocation)
+
 			m.getFocusedFilePanel().cursor = tt.cursorIndexForZip
 
 			TeaUpdateWithErrCheck(t, &m, utils.TeaRuneKeyMsg(common.Hotkeys.ExtractFile[0]))

src/internal/test_utils.go

@@ -19,6 +19,9 @@ var SampleDataBytes = []byte("This is sample") //nolint: gochecknoglobals // Eff
 func setupDirectories(t *testing.T, dirs ...string) {
 	t.Helper()
 	for _, dir := range dirs {
+		if dir == "" {
+			continue
+		}
 		err := os.MkdirAll(dir, 0755)
 		require.NoError(t, err)
 	}

src/internal/ui/prompt/tokenize_test.go

@@ -128,7 +128,7 @@ func Test_resolveShellSubstitution(t *testing.T) {
 		},
 		{
 			name:            "Ill formed command 2",
-			command:         "abc $(abc) syt ${ sdfc ( {)}",
+			command:         "abc $(echo abc) syt ${ sdfc ( {)}",
 			expectedResult:  "",
 			isErrorExpected: true,
 			errorToMatch:    curlyBracketMatchError(),