Reverse order of getHistory for a correct cleanup

repo/history.go

@@ -50,7 +50,7 @@ func (h *history) getHistory() (files []string, err error) {
 			}
 		}
 	}
-	sort.Strings(files)
+	sort.Sort(sort.Reverse(sort.StringSlice(files)))
 	return
 }
 

repo/history_test.go

@@ -7,6 +7,7 @@ import (
 	"os"
 	"testing"
 	"time"
+	"github.com/stretchr/testify/assert"
 )
 
 func testHistory() *history {
@@ -45,3 +46,16 @@ func TestHistoryCleanup(t *testing.T) {
 		t.Fatal("history too long", len(files), "instead of", maxHistoryVersions)
 	}
 }
+
+func TestHistoryOrder(t *testing.T) {
+	h := testHistory()
+	h.varDir = "testdata/order"
+
+	files, err := h.getHistory()
+
+	assert.NoError(t, err)
+	assert.Len(t, files, 3)
+	assert.Equal(t, "testdata/order/contentserver-repo-2017-10-23.json", files[0])
+	assert.Equal(t, "testdata/order/contentserver-repo-2017-10-22.json", files[1])
+	assert.Equal(t, "testdata/order/contentserver-repo-2017-10-21.json", files[2])
+}