fix: updating error handling for non tar files

Signed-off-by: Adnan Gulegulzar <[email protected]>

content/oci/readonlyoci_test.go

@@ -36,6 +36,7 @@ import (
 	"oras.land/oras-go/v2"
 	"oras.land/oras-go/v2/content"
 	"oras.land/oras-go/v2/content/memory"
+	"oras.land/oras-go/v2/errdef"
 	"oras.land/oras-go/v2/internal/docker"
 	"oras.land/oras-go/v2/internal/spec"
 	"oras.land/oras-go/v2/registry"
@@ -836,3 +837,15 @@ func Test_deleteAnnotationRefName(t *testing.T) {
 		})
 	}
 }
+
+func TestNewFromTar_NonTarFile(t *testing.T) {
+
+	emptyfile := "testdata/emptyfile"
+
+	_, err := NewFromTar(context.Background(), emptyfile)
+
+	if want := errdef.ErrNotTarFile; !errors.Is(err, want) {
+		t.Errorf("OCI.NewFromTar(%s) error = %v, wantErr %v", emptyfile, err, want)
+	}
+
+}

errdef/errors.go

@@ -28,4 +28,5 @@ var (
 	ErrSizeExceedsLimit   = errors.New("size exceeds limit")
 	ErrUnsupported        = errors.New("unsupported")
 	ErrUnsupportedVersion = errors.New("unsupported version")
+	ErrNotTarFile         = errors.New("tar file type expected")
 )

internal/fs/tarfs/tarfs.go

@@ -48,6 +48,13 @@ func New(path string) (*TarFS, error) {
 	if err != nil {
 		return nil, fmt.Errorf("failed to resolve absolute path for %s: %w", path, err)
 	}
+
+	extension := filepath.Ext(pathAbs)
+
+	if extension != ".tar" {
+		return nil, errdef.ErrNotTarFile
+	}
+
 	tarfs := &TarFS{
 		path:    pathAbs,
 		entries: make(map[string]*entry),

internal/fs/tarfs/tarfs_test.go

@@ -268,3 +268,15 @@ func TestTarFS_Stat_Unsupported(t *testing.T) {
 		}
 	}
 }
+
+func TestTarFS_New_NonTarFile(t *testing.T) {
+
+	emptyfile := "testdata/emptyfile"
+
+	_, err := New(emptyfile)
+
+	if want := errdef.ErrNotTarFile; !errors.Is(err, want) {
+		t.Errorf("TarFS.New(%s) error = %v, wantErr %v", emptyfile, err, want)
+	}
+
+}