[TEST] Name every constructor argument in the new cases

HttpOperation keeps references to its ssl options, headers, body, compression and
retry policy. The short constructor form materialises the defaulted ones as
temporaries that die at the end of the full expression, and Setup() reads them
from inside Send(): the Bazel asan job reported stack-use-after-scope in
HttpOperation::Setup() with the frame belonging to the case.

RetryPolicyEnabled in this file passes all twelve by name for that reason. The
case I patterned on, RetryJitterIsNotSharedAcrossThreads, uses the short form and
never calls Send(), so it never sees it.

Verified with bazel test --config=asan on the same target: no sanitizer report,
and the new cases run.

Signed-off-by: thc1006 <84045975+thc1006@users.noreply.github.com>

Author: thc1006

ext/test/http/curl_http_test.cc

@@ -521,8 +521,14 @@ TEST_F(BasicCurlHttpTests, ACaCertificateStringIsPassedAsABlob)
   http_client::Body body;
   http_client::Headers headers;
 
+  http_client::Compression compression = http_client::Compression::kNone;
+  http_client::RetryPolicy retry_policy;
+
+  // Every argument is named. The defaulted ones would be temporaries, and the operation keeps
+  // references to them past the end of this expression.
   curl::HttpOperation operation(http_client::Method::Get, "http://127.0.0.1:19000/get/",
-                                ssl_options, &handler, headers, body);
+                                ssl_options, &handler, headers, body, compression, false,
+                                curl::kDefaultHttpConnTimeout, false, false, retry_policy);
 
   ASSERT_EQ(CURLE_OK, operation.Send());
   ASSERT_EQ(200, operation.GetResponseCode());