test: update TarDir unit test

The current TestTarDir does not check if an empty archive is produced. Update
the test case to check for empty tar archives.

The include test case does not correctly check for *.mod, files. Update the
assertion to expect files matching either pattern.

The test is currently failing and will be fixed in the next few commits.

Signed-off-by: Aaron D Borden <[email protected]>

Author: adborden

pkg/utils/utils_test.go

@@ -2,6 +2,7 @@ package utils
 
 import (
 	"archive/tar"
+	"fmt"
 	"io"
 	"os"
 	"path"
@@ -126,19 +127,22 @@ func TestTarDir(t *testing.T) {
 
 	_ = TarDir(testSrcDir, tarPath, []string{}, []string{"*.mod"})
 	fileNames, _ := getTarFileNames(tarPath)
+	assert.Greater(t, len(fileNames), 0, "expected tar to have greater than 0 files")
 	for _, fileName := range fileNames {
 		flag, _ := filepath.Match(getNewPattern("*.mod"), fileName)
-		assert.Equal(t, flag, false)
+		assert.Equal(t, flag, false, "expected *.mod to be excluded")
 	}
 	_, err = os.Stat(tarPath)
 	assert.Equal(t, err, nil)
 	os.Remove(tarPath)
 
 	_ = TarDir(testSrcDir, tarPath, []string{"*/*.lock", "*.mod"}, []string{})
 	fileNames, _ = getTarFileNames(tarPath)
+	assert.Greater(t, len(fileNames), 0, "expected tar to have greater than 0 files")
 	for _, fileName := range fileNames {
-		flag, _ := filepath.Match(getNewPattern("*/*.lock"), fileName)
-		assert.Equal(t, flag, true)
+		matchedLockPattern, _ := filepath.Match(getNewPattern("*/*.lock"), fileName)
+		matchedModPattern, _ := filepath.Match(getNewPattern("*.mod"), fileName)
+		assert.True(t, matchedLockPattern || matchedModPattern, fmt.Sprintf("expected \"%s\" to match one of */*.lock or *.mod", fileName))
 	}
 	_, err = os.Stat(tarPath)
 	assert.Equal(t, err, nil)