Remove winhttp (#3832)

* remove refs to winhttp * this tsst only curl
Azure · Jul 15, 2022 · 7bf6735 · 7bf6735
1 parent 593437b
commit 7bf6735
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/sdk/core/azure-core/CMakeLists.txt b/sdk/core/azure-core/CMakeLists.txt
@@ -204,7 +204,10 @@ if(BUILD_TESTING)
     add_subdirectory(test/nlohmann-json-test)
   endif()
   add_subdirectory(test/fault-injector)
-  add_subdirectory(test/libcurl-stress-test)
+
+  if(BUILD_TRANSPORT_CURL)
+    add_subdirectory(test/libcurl-stress-test)
+  endif()
 endif()
 
 if (BUILD_PERFORMANCE_TESTS)

diff --git a/sdk/core/azure-core/test/libcurl-stress-test/libcurl_stress_test.cpp b/sdk/core/azure-core/test/libcurl-stress-test/libcurl_stress_test.cpp
@@ -27,16 +27,12 @@ void SendRequest(std::string target)
 {
   std::cout << target << std::endl;
   /* The transport adapter must allow insecure SSL certs.
-  If both curl and winHttp are available, curl is preferred for this test.for*/
-#if defined(BUILD_CURL_HTTP_TRANSPORT_ADAPTER)
+  If both curl and winHttp are available, curl is preferred for this test*/
+
   Azure::Core::Http::CurlTransportOptions curlOptions;
   curlOptions.SslVerifyPeer = false;
   auto implementationClient = std::make_shared<Azure::Core::Http::CurlTransport>(curlOptions);
 
-#elif (BUILD_TRANSPORT_WINHTTP_ADAPTER)
-  Azure::Core::Http::WinHttpTransportOptions winHttpOptions;
-  auto implementationClient = std::make_shared<Azure::Core::Http::WinHttpTransport>(winHttpOptions);
-#endif
   try
   {