[skip ci] clarify what these tests are attempting

Author: bvogelzang

test/client_test.rb

@@ -135,11 +135,14 @@ class ClientTest < TinyTds::TestCase
     it 'raises TinyTds exception with tcp socket network failure' do
       skip if ENV['CI'] && ENV['APPVEYOR_BUILD_FOLDER'] # only CI using docker
       begin
-        client = new_connection timeout: 1, port: 1234
+        client = new_connection timeout: 2, port: 1234
         assert_client_works(client)
-        action = lambda { client.execute("waitfor delay '00:00:02'").do }
-        # Delay the TCP socket from closing until delay has elapsed.
-        Toxiproxy[:sqlserver_test].toxic(:slow_close, delay: 500).apply do
+        action = lambda { client.execute("waitfor delay '00:00:05'").do }
+
+        # Use toxiproxy to close the TCP socket after 1 second.
+        # TinyTds shuuld be able to execute the statement, hit the timeout configured above, and then not be able to use the network to cancel
+        # the network connection needs to close after the sql batch is sent and before the timeout above is hit
+        Toxiproxy[:sqlserver_test].toxic(:slow_close, delay: 1000).apply do
           assert_raise_tinytds_error(action) do |e|
             assert_equal 20003, e.db_error_number
             assert_equal 6, e.severity
@@ -154,11 +157,14 @@ class ClientTest < TinyTds::TestCase
     it 'raises TinyTds exception with dead connection network failure' do
       skip if ENV['CI'] && ENV['APPVEYOR_BUILD_FOLDER'] # only CI using docker
       begin
-        client = new_connection timeout: 1, port: 1234
+        client = new_connection timeout: 2, port: 1234
         assert_client_works(client)
-        action = lambda { client.execute("waitfor delay '00:00:02'").do }
-        # Stops all data from getting through, and closes the connection after timeout
-        Toxiproxy[:sqlserver_test].toxic(:timeout, timeout: 500).apply do
+        action = lambda { client.execute("waitfor delay '00:00:05'").do }
+
+        # Use toxiproxy to close the network connection after 1 second.
+        # We want TinyTds to execute the statement, hit the timeout configured above, and then not be able to use the network to cancel
+        # the network connection needs to close after the sql batch is sent and before the timeout above is hit
+        Toxiproxy[:sqlserver_test].toxic(:timeout, timeout: 1000).apply do
           assert_raise_tinytds_error(action) do |e|
             assert_equal 20047, e.db_error_number
             assert_includes [1,9], e.severity