Fix linter issues

cs3org · Mar 22, 2021 · e6789a8 · e6789a8
1 parent bc90c4e
commit e6789a8
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/pkg/storage/utils/decomposedfs/tree/tree_test.go b/pkg/storage/utils/decomposedfs/tree/tree_test.go
@@ -145,6 +145,7 @@ var _ = Describe("Tree", func() {
 					Expect(restoreFunc()).To(Succeed())
 
 					originalNode, err := env.Lookup.NodeFromPath(env.Ctx, originalPath)
+					Expect(err).ToNot(HaveOccurred())
 					Expect(originalNode.Exists).To(BeTrue())
 				})
 
@@ -155,9 +156,11 @@ var _ = Describe("Tree", func() {
 					Expect(restoreFunc()).To(Succeed())
 
 					newNode, err := env.Lookup.NodeFromPath(env.Ctx, "dir1/newLocation")
+					Expect(err).ToNot(HaveOccurred())
 					Expect(newNode.Exists).To(BeTrue())
 
 					originalNode, err := env.Lookup.NodeFromPath(env.Ctx, originalPath)
+					Expect(err).ToNot(HaveOccurred())
 					Expect(originalNode.Exists).To(BeFalse())
 				})
 

diff --git a/pkg/storage/utils/localfs/localfs.go b/pkg/storage/utils/localfs/localfs.go
@@ -1191,11 +1191,12 @@ func (fs *localfs) RestoreRecycleItem(ctx context.Context, restoreKey, restorePa
 	}
 
 	var localRestorePath string
-	if restorePath != "" {
+	switch {
+	case restorePath != "":
 		localRestorePath = fs.wrap(ctx, restorePath)
-	} else if fs.isShareFolder(ctx, filePath) {
+	case fs.isShareFolder(ctx, filePath):
 		localRestorePath = fs.wrapReferences(ctx, filePath)
-	} else {
+	default:
 		localRestorePath = fs.wrap(ctx, filePath)
 	}