From 4a5cee2279fb7b3cb8a9fd4a7827e8018a38047d Mon Sep 17 00:00:00 2001
From: Parker Holladay <parker.holladay@gmail.com>
Date: Thu, 31 Aug 2023 22:44:29 -0600
Subject: [PATCH] Count success based on error and status code

---
 lib/metrics.go      | 2 +-
 lib/metrics_test.go | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/metrics.go b/lib/metrics.go
index 5d7dbf9b..96e7ef52 100644
--- a/lib/metrics.go
+++ b/lib/metrics.go
@@ -69,7 +69,7 @@ func (m *Metrics) Add(r *Result) {
 		m.End = end
 	}
 
-	if r.Code >= 200 && r.Code < 400 {
+	if r.Code >= 200 && r.Code < 400 && (r.Error == "") {
 		m.success++
 	}
 
diff --git a/lib/metrics_test.go b/lib/metrics_test.go
index 8d22f1a4..0465acce 100644
--- a/lib/metrics_test.go
+++ b/lib/metrics_test.go
@@ -62,8 +62,8 @@ func TestMetrics_Add(t *testing.T) {
 		Wait:        duration("10ms"),
 		Requests:    10000,
 		Rate:        1.000100010001,
-		Throughput:  0.6667660098349737,
-		Success:     0.6667,
+		Throughput:  0.3333329999669967,
+		Success:     0.3333,
 		StatusCodes: map[string]int{"500": 3333, "200": 3334, "302": 3333},
 		Errors:      []string{"Internal server error"},