From c2c06ae2c947a78d2024881b4e38e2a6f22c53f7 Mon Sep 17 00:00:00 2001
From: Alexander Neumann <alexander@bumpern.de>
Date: Sun, 17 Jun 2018 09:49:03 +0200
Subject: [PATCH] walker: Don't ignore empty trees by default

Closes #1849
---
 internal/walker/walker.go      |  4 +++
 internal/walker/walker_test.go | 56 ++++++++++++++++++++++++++++++++++
 2 files changed, 60 insertions(+)

diff --git a/internal/walker/walker.go b/internal/walker/walker.go
index b679cbce0..fd58dfa98 100644
--- a/internal/walker/walker.go
+++ b/internal/walker/walker.go
@@ -63,6 +63,10 @@ func Walk(ctx context.Context, repo TreeLoader, root restic.ID, ignoreTrees rest
 func walk(ctx context.Context, repo TreeLoader, prefix string, tree *restic.Tree, ignoreTrees restic.IDSet, walkFn WalkFunc) (ignore bool, err error) {
 	var allNodesIgnored = true
 
+	if len(tree.Nodes) == 0 {
+		allNodesIgnored = false
+	}
+
 	sort.Slice(tree.Nodes, func(i, j int) bool {
 		return tree.Nodes[i].Name < tree.Nodes[j].Name
 	})
diff --git a/internal/walker/walker_test.go b/internal/walker/walker_test.go
index 08b4fe405..7da3417e5 100644
--- a/internal/walker/walker_test.go
+++ b/internal/walker/walker_test.go
@@ -400,6 +400,62 @@ func TestWalker(t *testing.T) {
 				),
 			},
 		},
+		{
+			tree: TestTree{
+				"subdir1": TestTree{},
+				"subdir2": TestTree{},
+				"subdir3": TestTree{
+					"file": TestFile{},
+				},
+				"subdir4": TestTree{
+					"file": TestFile{},
+				},
+				"subdir5": TestTree{},
+				"subdir6": TestTree{},
+			},
+			checks: []checkFunc{
+				checkItemOrder([]string{
+					"/",
+					"/subdir1",
+					"/subdir2",
+					"/subdir3",
+					"/subdir3/file",
+					"/subdir4",
+					"/subdir4/file",
+					"/subdir5",
+					"/subdir6",
+				}),
+			},
+		},
+		{
+			tree: TestTree{
+				"subdir1": TestTree{},
+				"subdir2": TestTree{},
+				"subdir3": TestTree{
+					"file": TestFile{},
+				},
+				"subdir4": TestTree{},
+				"subdir5": TestTree{
+					"file": TestFile{},
+				},
+				"subdir6": TestTree{},
+			},
+			checks: []checkFunc{
+				checkIgnore(
+					map[string]struct{}{}, map[string]bool{
+						"/subdir2": true,
+					}, []string{
+						"/",
+						"/subdir1",
+						"/subdir2",
+						"/subdir3",
+						"/subdir3/file",
+						"/subdir5",
+						"/subdir5/file",
+					},
+				),
+			},
+		},
 	}
 
 	for _, test := range tests {