diff --git a/pkg/filesystem/hooks_test.go b/pkg/filesystem/hooks_test.go index b6f3fbb..bf82d4c 100644 --- a/pkg/filesystem/hooks_test.go +++ b/pkg/filesystem/hooks_test.go @@ -117,11 +117,9 @@ func TestGenericAfterUpload(t *testing.T) { WithArgs(1, "我的文件"). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) // 1 - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("我的文件", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) - mock.ExpectCommit() mock.ExpectQuery("SELECT(.+)files(.+)").WillReturnError(errors.New("not found")) mock.ExpectBegin() mock.ExpectExec("INSERT(.+)files(.+)").WillReturnResult(sqlmock.NewResult(1, 1)) @@ -140,11 +138,9 @@ func TestGenericAfterUpload(t *testing.T) { WithArgs(1, "我的文件"). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) // 1 - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("我的文件", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) - mock.ExpectCommit() mock.ExpectQuery("SELECT(.+)files(.+)").WillReturnRows( mock.NewRows([]string{"name"}).AddRow("test.txt"), ) @@ -160,11 +156,9 @@ func TestGenericAfterUpload(t *testing.T) { WithArgs(1, "我的文件"). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) // 1 - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("我的文件", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) - mock.ExpectCommit() mock.ExpectQuery("SELECT(.+)files(.+)").WillReturnRows( mock.NewRows([]string{"name", "upload_session_id"}).AddRow("test.txt", "1"), ) @@ -180,11 +174,9 @@ func TestGenericAfterUpload(t *testing.T) { WithArgs(1, "我的文件"). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) // 1 - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("我的文件", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) - mock.ExpectCommit() mock.ExpectQuery("SELECT(.+)files(.+)").WillReturnError(errors.New("not found")) mock.ExpectBegin() diff --git a/pkg/filesystem/manage_test.go b/pkg/filesystem/manage_test.go index eb78f1e..584460b 100644 --- a/pkg/filesystem/manage_test.go +++ b/pkg/filesystem/manage_test.go @@ -224,12 +224,10 @@ func TestFileSystem_CreateDirectory(t *testing.T) { WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id", "name"})) - mock.ExpectBegin() // ab mock.ExpectQuery("SELECT(.+)"). WithArgs("ab", 2, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(3, 1)) - mock.ExpectCommit() res, err := fs.CreateDirectory(ctx, "/ad/ab") asserts.NoError(err) asserts.EqualValues(3, res.ID) @@ -245,10 +243,10 @@ func TestFileSystem_CreateDirectory(t *testing.T) { WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(2, 1)) mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id", "name"})) - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("ab", 2, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) + mock.ExpectBegin() mock.ExpectExec("INSERT(.+)").WillReturnResult(sqlmock.NewResult(1, 1)) mock.ExpectCommit() _, err = fs.CreateDirectory(ctx, "/ad/ab") @@ -270,18 +268,18 @@ func TestFileSystem_CreateDirectory(t *testing.T) { WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(1, 1)) mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id", "name"})) // 创建ad - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("ad", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) + mock.ExpectBegin() mock.ExpectExec("INSERT(.+)").WillReturnResult(sqlmock.NewResult(2, 1)) mock.ExpectCommit() mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id", "name"})) // 创建ab - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("ab", 2, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) + mock.ExpectBegin() mock.ExpectExec("INSERT(.+)").WillReturnResult(sqlmock.NewResult(1, 1)) mock.ExpectCommit() _, err = fs.CreateDirectory(ctx, "/ad/ab") @@ -303,12 +301,14 @@ func TestFileSystem_CreateDirectory(t *testing.T) { WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"}).AddRow(1, 1)) mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id", "name"})) // 创建ad - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)"). WithArgs("ad", 1, 1). WillReturnRows(sqlmock.NewRows([]string{"id", "owner_id"})) + mock.ExpectBegin() mock.ExpectExec("INSERT(.+)").WillReturnResult(sqlmock.NewResult(2, 1)).WillReturnError(errors.New("error")) mock.ExpectRollback() + mock.ExpectQuery("SELECT(.+)"). + WillReturnError(errors.New("error")) _, err = fs.CreateDirectory(ctx, "/ad/ab") asserts.Error(err) asserts.NoError(mock.ExpectationsWereMet()) diff --git a/pkg/task/import_test.go b/pkg/task/import_test.go index 63b01a2..a17123d 100644 --- a/pkg/task/import_test.go +++ b/pkg/task/import_test.go @@ -177,8 +177,8 @@ func TestImportTask_Do(t *testing.T) { // 查找同名文件,不存在 mock.ExpectQuery("SELECT(.+)files").WillReturnRows(sqlmock.NewRows([]string{"id"})) // 创建目录 - mock.ExpectBegin() mock.ExpectQuery("SELECT(.+)folders").WillReturnRows(sqlmock.NewRows([]string{"id"})) + mock.ExpectBegin() mock.ExpectExec("INSERT(.+)folders(.+)").WillReturnResult(sqlmock.NewResult(2, 1)) mock.ExpectCommit() // 插入文件记录