move

backend/services/move.go

@@ -12,6 +12,10 @@ func (s *Services) MoveDocumentIntoDocument(from *entities.Document, to *entitie
 		return fmt.Errorf(entities.ERROR_DOCUMENT_NOT_PERCISTED)
 	}
 
+	if to.DocumentType != entities.DIRECTORY {
+		return fmt.Errorf("The destination document must be a directory")
+	}
+
 	if canEdit, _ := UserCanEditDocument(s.ConnectedUser, from); canEdit == false {
 		return fmt.Errorf("You can not edit this folder")
 	}

backend/services/move_test.go

@@ -13,12 +13,40 @@ func TestMoveDocument(t *testing.T) {
 		t.Errorf(`Error creating antoine user %v`, err)
 	}
 
+	fakeUUID := "dsqdsq"
 	err, document := entities.NewDocument("test.pdf", entities.FILE)
+	document.DocumentUUID = &fakeUUID
 
 	err, folder := entities.NewDocument("orga", entities.DIRECTORY)
+	folder.DocumentUUID = &fakeUUID
 
 	service := New(userAntoine)
 
-	err := service.MoveDocumentIntoDocument(document, folder)
+	err = service.MoveDocumentIntoDocument(document, folder)
+
+	if err != nil {
+		t.Error(err)
+	}
+
+}
+
+func TestMoveDocumentToAFile(t *testing.T) {
+
+	err, userAntoine := entities.NewUser("antoine")
+	if err != nil {
+		t.Errorf(`Error creating antoine user %v`, err)
+	}
+
+	err, document := entities.NewDocument("test.pdf", entities.FILE)
+
+	err, notAFolder := entities.NewDocument("orga", entities.FILE)
+
+	service := New(userAntoine)
+
+	err = service.MoveDocumentIntoDocument(document, notAFolder)
+
+	if err == nil {
+		t.Error(err)
+	}
 
 }