From cadb15f7afb5e7c88667eb4006209efca17744af Mon Sep 17 00:00:00 2001
From: Julien Dessaux
Date: Mon, 24 Feb 2020 23:05:45 +0100
Subject: Added tests to the main package and completely reworked the code
 around that

---
 job/utils_test.go | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

(limited to 'job/utils_test.go')

diff --git a/job/utils_test.go b/job/utils_test.go
index a3e4dcd..250ae2c 100644
--- a/job/utils_test.go
+++ b/job/utils_test.go
@@ -9,8 +9,13 @@ func TestKeepOldestOnly(t *testing.T) {
 	emptyList := []Job{}
 	oneJob := []Job{{Name: "a", Timestamp: 10, Success: true}}
 	twoJobs := []Job{
+		{Name: "a", Timestamp: 5, Success: true},
 		{Name: "a", Timestamp: 10, Success: true},
+	}
+	threeJobs := []Job{
 		{Name: "a", Timestamp: 5, Success: true},
+		{Name: "a", Timestamp: 10, Success: true},
+		{Name: "a", Timestamp: 8, Success: false},
 	}
 	type args struct {
 		jobs []Job
@@ -20,9 +25,10 @@ func TestKeepOldestOnly(t *testing.T) {
 		args args
 		want []Job
 	}{
-		{"empty list", args{emptyList}, emptyList},
+		{"empty list", args{emptyList}, nil},
 		{"one job", args{oneJob}, oneJob},
 		{"two jobs", args{twoJobs}, oneJob},
+		{"three jobs", args{threeJobs}, oneJob},
 	}
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-- 
cgit v1.2.3