diff options
| author | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2022-05-28 11:35:50 -0700 |
|---|---|---|
| committer | Jeff Halter <868228+jhalter@users.noreply.github.com> | 2022-05-28 11:35:50 -0700 |
| commit | aebc4d3647b9823ae8cbb57b21b1af83bfd011fb (patch) | |
| tree | 6fa8871ec45654b68e6b018d6d06969d21b7278f /hotline/transaction_handlers_test.go | |
| parent | decc2fbf5db4a05aec93462ad35d890930bddd04 (diff) | |
Misc cleanup
* Removed some unnecessary user of pointers
* Removed dead cruft
* Reorganized code
Diffstat (limited to 'hotline/transaction_handlers_test.go')
| -rw-r--r-- | hotline/transaction_handlers_test.go | 20 |
1 files changed, 10 insertions, 10 deletions
diff --git a/hotline/transaction_handlers_test.go b/hotline/transaction_handlers_test.go index a83f4cc..5ffe45c 100644 --- a/hotline/transaction_handlers_test.go +++ b/hotline/transaction_handlers_test.go @@ -579,7 +579,7 @@ func TestHandleNewFolder(t *testing.T) { ), }, setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} mfs.On("Mkdir", "/Files/aaa/testFolder", fs.FileMode(0777)).Return(nil) mfs.On("Stat", "/Files/aaa/testFolder").Return(nil, os.ErrNotExist) FS = mfs @@ -613,7 +613,7 @@ func TestHandleNewFolder(t *testing.T) { ), }, setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} mfs.On("Mkdir", "/Files/testFolder", fs.FileMode(0777)).Return(nil) mfs.On("Stat", "/Files/testFolder").Return(nil, os.ErrNotExist) FS = mfs @@ -650,7 +650,7 @@ func TestHandleNewFolder(t *testing.T) { ), }, setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} mfs.On("Mkdir", "/Files/aaa/testFolder", fs.FileMode(0777)).Return(nil) mfs.On("Stat", "/Files/aaa/testFolder").Return(nil, os.ErrNotExist) FS = mfs @@ -675,7 +675,7 @@ func TestHandleNewFolder(t *testing.T) { ), }, setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} mfs.On("Mkdir", "/Files/testFolder", fs.FileMode(0777)).Return(nil) mfs.On("Stat", "/Files/testFolder").Return(nil, os.ErrNotExist) FS = mfs @@ -717,7 +717,7 @@ func TestHandleNewFolder(t *testing.T) { ), }, setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} mfs.On("Mkdir", "/Files/foo/testFolder", fs.FileMode(0777)).Return(nil) mfs.On("Stat", "/Files/foo/testFolder").Return(nil, os.ErrNotExist) FS = mfs @@ -851,9 +851,9 @@ func TestHandleUploadFile(t *testing.T) { t.Errorf("HandleUploadFile() error = %v, wantErr %v", err, tt.wantErr) return } - if !tranAssertEqual(t, tt.wantRes, gotRes) { - t.Errorf("HandleUploadFile() gotRes = %v, want %v", gotRes, tt.wantRes) - } + + tranAssertEqual(t, tt.wantRes, gotRes) + }) } } @@ -873,7 +873,7 @@ func TestHandleMakeAlias(t *testing.T) { { name: "with valid input and required permissions", setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} path, _ := os.Getwd() mfs.On( "Symlink", @@ -924,7 +924,7 @@ func TestHandleMakeAlias(t *testing.T) { { name: "when symlink returns an error", setup: func() { - mfs := MockFileStore{} + mfs := &MockFileStore{} path, _ := os.Getwd() mfs.On( "Symlink", |