mirror of https://github.com/cloudreve/Cloudreve
				
				
				
			Fix: failed test due to database type
							parent
							
								
									a13530f969
								
							
						
					
					
						commit
						5424115e51
					
				| 
						 | 
				
			
			@ -272,7 +272,7 @@ func TestDeleteFileByIDs(t *testing.T) {
 | 
			
		|||
	// 出错
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnError(errors.New("error"))
 | 
			
		||||
		mock.ExpectRollback()
 | 
			
		||||
		err := DeleteFileByIDs([]uint{1, 2, 3})
 | 
			
		||||
| 
						 | 
				
			
			@ -282,7 +282,7 @@ func TestDeleteFileByIDs(t *testing.T) {
 | 
			
		|||
	// 成功
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
		err := DeleteFileByIDs([]uint{1, 2, 3})
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -3,6 +3,7 @@ package model
 | 
			
		|||
import (
 | 
			
		||||
	"errors"
 | 
			
		||||
	"github.com/DATA-DOG/go-sqlmock"
 | 
			
		||||
	"github.com/HFO4/cloudreve/pkg/conf"
 | 
			
		||||
	"github.com/HFO4/cloudreve/pkg/util"
 | 
			
		||||
	"github.com/jinzhu/gorm"
 | 
			
		||||
	"github.com/stretchr/testify/assert"
 | 
			
		||||
| 
						 | 
				
			
			@ -75,20 +76,21 @@ func TestFolder_GetChildFolder(t *testing.T) {
 | 
			
		|||
}
 | 
			
		||||
 | 
			
		||||
func TestGetRecursiveChildFolder(t *testing.T) {
 | 
			
		||||
	conf.DatabaseConfig.Type = "mysql"
 | 
			
		||||
	asserts := assert.New(t)
 | 
			
		||||
	dirs := []string{"/目录1", "/目录2"}
 | 
			
		||||
 | 
			
		||||
	// 正常
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectQuery("SELECT(.+)folders(.+)").
 | 
			
		||||
			WithArgs(1, util.BuildRegexp(dirs, "^", "/", "|"), "/目录1", "/目录2").
 | 
			
		||||
			WithArgs(1, util.BuildRegexp(dirs, "^", "/", "|"), 1, "/目录1", "/目录2").
 | 
			
		||||
			WillReturnRows(
 | 
			
		||||
				sqlmock.NewRows([]string{"id", "name"}).
 | 
			
		||||
					AddRow(1, "sub1").
 | 
			
		||||
					AddRow(2, "sub2").
 | 
			
		||||
					AddRow(3, "sub3"),
 | 
			
		||||
			)
 | 
			
		||||
		subs, err := GetRecursiveChildFolder(dirs, 1)
 | 
			
		||||
		subs, err := GetRecursiveChildFolder(dirs, 1, true)
 | 
			
		||||
		asserts.NoError(mock.ExpectationsWereMet())
 | 
			
		||||
		asserts.NoError(err)
 | 
			
		||||
		asserts.Len(subs, 3)
 | 
			
		||||
| 
						 | 
				
			
			@ -96,9 +98,9 @@ func TestGetRecursiveChildFolder(t *testing.T) {
 | 
			
		|||
	// 出错
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectQuery("SELECT(.+)folders(.+)").
 | 
			
		||||
			WithArgs(1, util.BuildRegexp(dirs, "^", "/", "|"), "/目录1", "/目录2").
 | 
			
		||||
			WithArgs(1, util.BuildRegexp(dirs, "^", "/", "|"), 1, "/目录1", "/目录2").
 | 
			
		||||
			WillReturnError(errors.New("233"))
 | 
			
		||||
		subs, err := GetRecursiveChildFolder(dirs, 1)
 | 
			
		||||
		subs, err := GetRecursiveChildFolder(dirs, 1, true)
 | 
			
		||||
		asserts.NoError(mock.ExpectationsWereMet())
 | 
			
		||||
		asserts.Error(err)
 | 
			
		||||
		asserts.Len(subs, 0)
 | 
			
		||||
| 
						 | 
				
			
			@ -111,7 +113,7 @@ func TestDeleteFolderByIDs(t *testing.T) {
 | 
			
		|||
	// 出错
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnError(errors.New("error"))
 | 
			
		||||
		mock.ExpectRollback()
 | 
			
		||||
		err := DeleteFolderByIDs([]uint{1, 2, 3})
 | 
			
		||||
| 
						 | 
				
			
			@ -121,7 +123,7 @@ func TestDeleteFolderByIDs(t *testing.T) {
 | 
			
		|||
	// 成功
 | 
			
		||||
	{
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
		err := DeleteFolderByIDs([]uint{1, 2, 3})
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
| 
						 | 
				
			
			@ -5,6 +5,7 @@ import (
 | 
			
		|||
	"errors"
 | 
			
		||||
	"github.com/DATA-DOG/go-sqlmock"
 | 
			
		||||
	model "github.com/HFO4/cloudreve/models"
 | 
			
		||||
	"github.com/HFO4/cloudreve/pkg/conf"
 | 
			
		||||
	"github.com/HFO4/cloudreve/pkg/serializer"
 | 
			
		||||
	"github.com/jinzhu/gorm"
 | 
			
		||||
	"github.com/stretchr/testify/assert"
 | 
			
		||||
| 
						 | 
				
			
			@ -175,6 +176,7 @@ func TestFileSystem_CreateDirectory(t *testing.T) {
 | 
			
		|||
}
 | 
			
		||||
 | 
			
		||||
func TestFileSystem_ListDeleteFiles(t *testing.T) {
 | 
			
		||||
	conf.DatabaseConfig.Type = "mysql"
 | 
			
		||||
	asserts := assert.New(t)
 | 
			
		||||
	fs := &FileSystem{User: &model.User{
 | 
			
		||||
		Model: gorm.Model{
 | 
			
		||||
| 
						 | 
				
			
			@ -201,6 +203,7 @@ func TestFileSystem_ListDeleteFiles(t *testing.T) {
 | 
			
		|||
}
 | 
			
		||||
 | 
			
		||||
func TestFileSystem_ListDeleteDirs(t *testing.T) {
 | 
			
		||||
	conf.DatabaseConfig.Type = "mysql"
 | 
			
		||||
	asserts := assert.New(t)
 | 
			
		||||
	fs := &FileSystem{User: &model.User{
 | 
			
		||||
		Model: gorm.Model{
 | 
			
		||||
| 
						 | 
				
			
			@ -260,6 +263,7 @@ func TestFileSystem_ListDeleteDirs(t *testing.T) {
 | 
			
		|||
}
 | 
			
		||||
 | 
			
		||||
func TestFileSystem_Delete(t *testing.T) {
 | 
			
		||||
	conf.DatabaseConfig.Type = "mysql"
 | 
			
		||||
	asserts := assert.New(t)
 | 
			
		||||
	fs := &FileSystem{User: &model.User{
 | 
			
		||||
		Model: gorm.Model{
 | 
			
		||||
| 
						 | 
				
			
			@ -272,6 +276,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
 | 
			
		||||
	// 全部未成功
 | 
			
		||||
	{
 | 
			
		||||
		// 列出要删除的目录
 | 
			
		||||
		mock.ExpectQuery("SELECT(.+)").
 | 
			
		||||
			WillReturnRows(
 | 
			
		||||
				sqlmock.NewRows([]string{"id"}).
 | 
			
		||||
| 
						 | 
				
			
			@ -285,6 +290,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
				sqlmock.NewRows([]string{"id", "name", "source_name", "policy_id", "size"}).
 | 
			
		||||
					AddRow(4, "1.txt", "1.txt", 2, 1),
 | 
			
		||||
			)
 | 
			
		||||
		// 查询顶级的文件
 | 
			
		||||
		mock.ExpectQuery("SELECT(.+)").WillReturnRows(sqlmock.NewRows([]string{"id", "name", "source_name", "policy_id", "size"}).AddRow(1, "1.txt", "1.txt", 1, 2))
 | 
			
		||||
		mock.ExpectQuery("SELECT(.+)files(.+)").
 | 
			
		||||
			WillReturnRows(sqlmock.NewRows([]string{"id", "policy_id", "source_name"}))
 | 
			
		||||
| 
						 | 
				
			
			@ -293,7 +299,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
		mock.ExpectQuery("SELECT(.+)").WillReturnRows(sqlmock.NewRows([]string{"id", "type"}).AddRow(1, "local"))
 | 
			
		||||
		// 删除文件记录
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
		// 归还容量
 | 
			
		||||
| 
						 | 
				
			
			@ -303,7 +309,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
		mock.ExpectCommit()
 | 
			
		||||
		// 删除目录
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			@ -340,7 +346,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
		mock.ExpectQuery("SELECT(.+)").WillReturnRows(sqlmock.NewRows([]string{"id", "type"}).AddRow(1, "local"))
 | 
			
		||||
		// 删除文件记录
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
		// 归还容量
 | 
			
		||||
| 
						 | 
				
			
			@ -350,7 +356,7 @@ func TestFileSystem_Delete(t *testing.T) {
 | 
			
		|||
		mock.ExpectCommit()
 | 
			
		||||
		// 删除目录
 | 
			
		||||
		mock.ExpectBegin()
 | 
			
		||||
		mock.ExpectExec("UPDATE(.+)delete(.+)").
 | 
			
		||||
		mock.ExpectExec("DELETE(.+)").
 | 
			
		||||
			WillReturnResult(sqlmock.NewResult(0, 3))
 | 
			
		||||
		mock.ExpectCommit()
 | 
			
		||||
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in New Issue