Update tests to match the new project style

Author: 0x2b3bfa0

iterative/utils/runner_test.go

@@ -1,23 +1,25 @@
 package utils
 
-import "testing"
+import (
+	"testing"
 
-func TestPositiveReadinessCheck(t *testing.T) {
-	result := IsReady(`
-        -- Logs begin at Wed 2021-01-20 00:25:37 UTC, end at Fri 2021-02-26 15:37:56 UTC. --
-        Feb 26 15:37:20 ip-172-31-6-188 cml.sh[2203]: {"level":"info","time":"···","repo":"···","status":"ready"}
-    `)
-	if !result {
-		t.Errorf("Positive readiness check")
-	}
-}
+	"github.com/stretchr/testify/assert"
+)
+
+func TestReadinessCheck(t *testing.T) {
+	t.Run("Runner log containing a readiness message should be considered ready", func(t *testing.T) {
+		logString := `
+			-- Logs begin at Wed 2021-01-20 00:25:37 UTC, end at Fri 2021-02-26 15:37:56 UTC. --
+			Feb 26 15:37:20 ip-172-31-6-188 cml.sh[2203]: {"level":"info","time":"···","repo":"···","status":"ready"}
+		`
+		assert.True(t, IsReady(logString))
+	})
 
-func TestNegativeReadinessCheck(t *testing.T) {
-	result := IsReady(`
-        -- Logs begin at Wed 2021-01-20 00:25:37 UTC, end at Fri 2021-02-26 15:37:56 UTC. --
-        Feb 26 15:37:20 ip-172-31-6-188 cml.sh[2203]: {"level":"info","time":"···","repo":"···"}
-    `)
-	if result {
-		t.Errorf("Negative readiness check")
-	}
+	t.Run("Runner log not containing a readiness message should not be considered ready", func(t *testing.T) {
+		logString := `
+			-- Logs begin at Wed 2021-01-20 00:25:37 UTC, end at Fri 2021-02-26 15:37:56 UTC. --
+			Feb 26 15:37:20 ip-172-31-6-188 cml.sh[2203]: {"level":"info","time":"···","repo":"···"}
+		`
+		assert.False(t, IsReady(logString))
+	})
 }