From 46757fce6e3f2f6d11d4cb60e40a86f2efaaa056 Mon Sep 17 00:00:00 2001
From: Irine Sistiana <49315432+IrineSistiana@users.noreply.github.com>
Date: Mon, 14 Feb 2022 21:43:20 +0800
Subject: [PATCH] tools: don't test idle timeout if previous test failed

---
 tools/tools.go | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/tools/tools.go b/tools/tools.go
index e98e9f9c1..6357e196f 100644
--- a/tools/tools.go
+++ b/tools/tools.go
@@ -110,7 +110,7 @@ func ProbServerTimeout(addr string) error {
 			}
 			_, err = conn.ReadMsg()
 			if err != nil {
-				return fmt.Errorf("failed to read probe #%d msg response: %v", i, err)
+				return fmt.Errorf("failed to read #%d probe msg response: %v", i, err)
 			}
 		}
 		return nil
@@ -151,7 +151,7 @@ func ProbServerTimeout(addr string) error {
 			conn.SetDeadline(time.Now().Add(time.Second * 10))
 			m, err := conn.ReadMsg()
 			if err != nil {
-				return false, fmt.Errorf("failed to read probe #%d msg response: %v", i, err)
+				return false, fmt.Errorf("failed to read #%d probe msg response: %v", i, err)
 			}
 
 			mlog.S().Infof("#%d response received, latency: %d ms", m.Id, time.Since(start).Milliseconds())
@@ -195,6 +195,7 @@ func ProbServerTimeout(addr string) error {
 	mlog.S().Info("testing basic connection reuse")
 	if err := testBasicReuse(); err != nil {
 		mlog.S().Infof("× test failed: %v", err)
+		return nil // bypass the following tests if the basic test failed.
 	} else {
 		mlog.S().Info("√ basic connection reuse test passed")
 	}