fix(server): remove the root file from asset files (#1134)

* fix: filter the root file when finding file by id

* fix: error checking if statement

* fix: error checking in aws file

* fix: update the commented code

* move the filter logic to pkg and add a test

server/internal/infrastructure/aws/file.go

@@ -48,8 +48,7 @@ func NewFile(ctx context.Context, bucketName, baseURL, cacheControl string) (gat
 		baseURL = fmt.Sprintf("https://%s.s3.amazonaws.com/", bucketName)
 	}
 
-	var err error
-	u, _ = url.Parse(baseURL)
+	u, err := url.Parse(baseURL)
 	if err != nil {
 		return nil, rerror.NewE(i18n.T("invalid base URL"))
 	}

server/internal/infrastructure/gcp/file.go

@@ -42,8 +42,7 @@ func NewFile(bucketName, base, cacheControl string) (gateway.File, error) {
 		base = fmt.Sprintf("https://storage.googleapis.com/%s", bucketName)
 	}
 
-	var err error
-	u, _ = url.Parse(base)
+	u, err := url.Parse(base)
 	if err != nil {
 		return nil, rerror.NewE(i18n.T("invalid base URL"))
 	}

server/pkg/asset/file.go

@@ -61,7 +61,9 @@ func (f *File) Files() []*File {
 }
 
 func (f *File) SetFiles(s []*File) {
-	f.files = slices.Clone(s)
+	f.files = lo.Filter(s, func(af *File, _ int) bool {
+		return af.Path() != f.Path()
+	})
 }
 
 func (f *File) FilePaths() []string {

server/pkg/asset/file_test.go

@@ -70,6 +70,19 @@ func TestFile_Files(t *testing.T) {
 	}, f.FlattenChildren())
 }
 
+func TestFile_SetFiles(t *testing.T) {
+	root := NewFile().Build()
+	files := []*File{NewFile().Path("aaa/a/a.txt").Build(), NewFile().Path("aaa/b.txt").Build()}
+	root.SetFiles(files)
+	assert.Equal(t, files, root.files)
+
+	root2 := NewFile().Path("aaa.zip").Build()
+	files2 := []*File{NewFile().Path("aaa.zip").Build(), NewFile().Path("aaa/a/a.txt").Build(), NewFile().Path("aaa/b.txt").Build()}
+	expected := []*File{NewFile().Path("aaa/a/a.txt").Build(), NewFile().Path("aaa/b.txt").Build()}
+	root2.SetFiles(files2)
+	assert.Equal(t, expected, root2.files)
+}
+
 func Test_FoldFiles(t *testing.T) {
 	assert.Equal(t,
 		&File{