fix: [#554] Storage can't judge nesting folder correctly (#49)

cos.go

@@ -340,14 +340,8 @@ func (r *Cos) Path(file string) string {
 func (r *Cos) Put(file string, content string) error {
 	// If the file is created in a folder directly, we can't check if the folder exists.
 	// So we need to create the top folder first.
-	if !strings.HasSuffix(file, "/") {
-		index := strings.Index(file, "/")
-		if index != -1 {
-			folder := file[:index+1]
-			if err := r.MakeDirectory(folder); err != nil {
-				return err
-			}
-		}
+	if err := r.makeDirectories(file); err != nil {
+		return err
 	}
 
 	tempFile, err := r.tempFile(content)
@@ -375,14 +369,8 @@ func (r *Cos) PutFileAs(filePath string, source filesystem.File, name string) (s
 
 	// If the file is created in a folder directly, we can't check if the folder exists.
 	// So we need to create the top folder first.
-	if !strings.HasSuffix(fullPath, "/") {
-		index := strings.Index(fullPath, "/")
-		if index != -1 {
-			folder := fullPath[:index+1]
-			if err := r.MakeDirectory(folder); err != nil {
-				return "", err
-			}
-		}
+	if err := r.makeDirectories(str.Of(filePath).Finish("/").String()); err != nil {
+		return "", err
 	}
 
 	if _, _, err := r.instance.Object.Upload(
@@ -436,6 +424,18 @@ func (r *Cos) Url(file string) string {
 	return objectUrl.String()
 }
 
+func (r *Cos) makeDirectories(path string) error {
+	folders := strings.Split(path, "/")
+	for i := 1; i < len(folders); i++ {
+		folder := strings.Join(folders[:i], "/")
+		if err := r.MakeDirectory(folder); err != nil {
+			return err
+		}
+	}
+
+	return nil
+}
+
 func (r *Cos) tempFile(content string) (*os.File, error) {
 	tempFile, err := os.CreateTemp(os.TempDir(), "goravel-")
 	if err != nil {

cos_test.go

@@ -50,6 +50,10 @@ func TestStorage(t *testing.T) {
 				assert.True(t, driver.Exists("AllDirectories/2.txt"))
 				assert.True(t, driver.Exists("AllDirectories/3/3.txt"))
 				assert.True(t, driver.Exists("AllDirectories/3/4/"))
+				assert.True(t, driver.Exists("AllDirectories/"))
+				assert.True(t, driver.Exists("AllDirectories/3/"))
+				assert.True(t, driver.Exists("AllDirectories/3/5/"))
+				assert.True(t, driver.Exists("AllDirectories/3/5/6/"))
 				assert.True(t, driver.Exists("AllDirectories/3/5/6/6.txt"))
 				files, err := driver.AllDirectories("AllDirectories")
 				assert.Nil(t, err)
@@ -267,9 +271,11 @@ func TestStorage(t *testing.T) {
 		{
 			name: "Put",
 			setup: func() {
-				assert.Nil(t, driver.Put("Put/1.txt", "Goravel"))
-				assert.True(t, driver.Exists("Put/1.txt"))
+				assert.Nil(t, driver.Put("Put/a/b/1.txt", "Goravel"))
 				assert.True(t, driver.Exists("Put/"))
+				assert.True(t, driver.Exists("Put/a/"))
+				assert.True(t, driver.Exists("Put/a/b/"))
+				assert.True(t, driver.Exists("Put/a/b/1.txt"))
 				assert.True(t, driver.Missing("Put/2.txt"))
 				assert.Nil(t, driver.DeleteDirectory("Put"))
 			},