impl(bigtable): changed test to use AsyncPrepareQuery to increase coverage (#15736)

Author: scotthart

google/cloud/bigtable/client.cc

@@ -28,13 +28,14 @@ StatusOr<PreparedQuery> Client::PrepareQuery(InstanceResource const& instance,
                                              SqlStatement const& statement,
                                              Options opts) {
   OptionsSpan span(MergeOptions(std::move(opts), opts_));
-  return conn_->PrepareQuery({std::move(instance), std::move(statement)});
+  return conn_->PrepareQuery({instance, statement});
 }
 
 future<StatusOr<PreparedQuery>> Client::AsyncPrepareQuery(
-    // NOLINTNEXTLINE(performance-unnecessary-value-param)
-    InstanceResource const& instance, SqlStatement const& statement, Options) {
-  return conn_->AsyncPrepareQuery({std::move(instance), std::move(statement)});
+    InstanceResource const& instance, SqlStatement const& statement,
+    Options opts) {
+  OptionsSpan span(MergeOptions(std::move(opts), opts_));
+  return conn_->AsyncPrepareQuery({instance, statement});
 }
 
 RowStream Client::ExecuteQuery(BoundQuery&& bound_query, Options opts) {

google/cloud/bigtable/tests/data_integration_test.cc

@@ -798,7 +798,8 @@ TEST_P(DataIntegrationTest, SingleColumnQueryWithHistory) {
              WHERE _key = '%s')sql",
       quoted_table_name, row_key);
   auto prepared_query =
-      client.PrepareQuery(instance_resource, SqlStatement(query_string));
+      client.AsyncPrepareQuery(instance_resource, SqlStatement(query_string))
+          .get();
   ASSERT_TRUE(prepared_query.ok()) << prepared_query.status().message();
 
   auto bound_query = (*prepared_query).BindParameters({});