Winhttp flaky test

sdk/core/azure-core/src/http/winhttp/win_http_transport.cpp

@@ -522,10 +522,20 @@ namespace Azure { namespace Core { namespace Http { namespace _detail {
   }
   void WinHttpAction::CompleteActionWithError(DWORD_PTR stowedErrorInformation, DWORD stowedError)
   {
-    // Note that the order of scope_exit and lock is important - this ensures that scope_exit is
-    // destroyed *after* lock is destroyed, ensuring that the event is not set to the signalled
-    // state before the lock is released.
-    auto scope_exit{m_actionCompleteEvent.SetEvent_scope_exit()};
+    if (m_expectedStatus != WINHTTP_CALLBACK_STATUS_HANDLE_CLOSING)
+    {
+      // Note that the order of scope_exit and lock is important - this ensures that scope_exit is
+      // destroyed *after* lock is destroyed, ensuring that the event is not set to the signalled
+      // state before the lock is released.
+      auto scope_exit{m_actionCompleteEvent.SetEvent_scope_exit()};
+    }
+    else
+    {
+      Log::Write(
+          Logger::Level::Verbose,
+          "Received error while closing: " + std::to_string(stowedError));
+    }
+
     std::unique_lock<std::mutex> lock(m_actionCompleteMutex);
     m_stowedErrorInformation = stowedErrorInformation;
     m_stowedError = stowedError;

sdk/core/azure-core/test/ut/transport_adapter_base_test.cpp

@@ -383,14 +383,17 @@ namespace Azure { namespace Core { namespace Test {
 
   TEST_P(TransportAdapter, cancelRequest)
   {
-    Azure::Core::Url hostPath(AzureSdkHttpbinServer::Delay() + "/10"); // 10 seconds delay on server
-    Azure::Core::Context cancelThis = Azure::Core::Context::ApplicationContext.WithDeadline(
-        std::chrono::system_clock::now() + std::chrono::seconds(3));
+    Azure::Core::Url hostPath(AzureSdkHttpbinServer::Delay() + "/2"); // 2 seconds delay on server
+    for (int i = 0; i < 10; i++)
+    {
+      Azure::Core::Context cancelThis = Azure::Core::Context::ApplicationContext.WithDeadline(
+          std::chrono::system_clock::now() + std::chrono::milliseconds(500));
 
-    auto request = Azure::Core::Http::Request(Azure::Core::Http::HttpMethod::Get, hostPath);
+      auto request = Azure::Core::Http::Request(Azure::Core::Http::HttpMethod::Get, hostPath);
 
-    // Request will be cancelled 3 seconds after sending the request.
-    EXPECT_THROW(m_pipeline->Send(request, cancelThis), Azure::Core::OperationCancelledException);
+      // Request will be canceled 500ms after sending the request.
+      EXPECT_THROW(m_pipeline->Send(request, cancelThis), Azure::Core::OperationCancelledException);
+    }
   }
 
   TEST_P(TransportAdapter, cancelTransferDownload)