Fix: Check for error being EPIPE only if an error actually occurred

.github/workflows/macos.yaml

@@ -12,13 +12,9 @@ on:
   push:
     branches:
     - master
-    - macOSRunner
-    - ErrnoNotEPIPEServerTest
   pull_request:
     branches:
     - master
-    - macOSRunner
-    - ErrnoNotEPIPEServerTest
 
 defaults:
   run:

tests/http_server_test.cc

@@ -999,6 +999,8 @@ TEST(http_server_test, client_request_timeout_on_delay_in_request_line_send_rais
 
     if (send_failed)
     { // Usually, send does fail; but on macOS occasionally it does not fail
+      // We workaround that here, since of course we can only check for an
+      // error code when there is an actual error
         EXPECT_EQ(client.lastErrno(), EPIPE) << "Errno: " << client.lastErrno();
 
         char recvBuf[1024] = {

tests/tcp_client.h

@@ -16,7 +16,7 @@
 
 // In CLIENT_TRY, note that strerror is allowed to change errno in certain
 // circumstances, so we must save errno in lastErrno_ BEFORE we call strerror
-// 
+//
 // Secondly, if errno has not been set at all then we set lastErrno_ =
 // ECANCELED; the ECANCELED errno is not used in Pistache code as of Aug/2024.
 static const char * strerror_errstr = "<no strerror>";