internal/daemon: simplify time update tests (#5041)

Using time.After makes the intent clearer

internal/daemon/controller/testing.go

@@ -937,7 +937,7 @@ func (tc *TestController) WaitForNextWorkerStatusUpdate(workerStatusName string)
 			break
 		}
 
-		if waitStatusCurrent.Sub(waitStatusStart) > 0 {
+		if waitStatusCurrent.After(waitStatusStart) {
 			break
 		}
 	}

internal/daemon/worker/status.go

@@ -106,7 +106,7 @@ func (w *Worker) WaitForNextSuccessfulStatusUpdate() error {
 			return ctx.Err()
 		}
 
-		if w.lastSuccessfulStatusTime().Sub(waitStatusStart) > 0 {
+		if w.lastSuccessfulStatusTime().After(waitStatusStart) {
 			break
 		}
 	}

internal/daemon/worker/worker.go

@@ -711,7 +711,7 @@ func (w *Worker) Shutdown() error {
 			break
 		}
 
-		if w.lastSuccessfulStatusTime().Sub(waitStatusStart) > 0 {
+		if w.lastSuccessfulStatusTime().After(waitStatusStart) {
 			break
 		}