combine enable/disable fn into single fn

Author: nareshz

google-cloud-spanner/src/main/java/com/google/cloud/spanner/SpannerOptions.java

@@ -1408,22 +1408,13 @@ public Builder setEnableExtendedTracing(boolean enableExtendedTracing) {
     }
 
     /**
-     * Enable spanner server side tracing. Enabling this option will create the trace spans at the
-     * Spanner layer. By default, server side tracing is disabled. Enabling server side tracing
-     * requires OpenTelemetry to be set up properly. Simply enabling this option won't generate
-     * server side traces.
+     * Sets whether to enable Spanner server side tracing. Enabling this option will create the
+     * trace spans at the Spanner layer. By default, server side tracing is disabled. Enabling
+     * server side tracing requires OpenTelemetry to be set up properly. Simply enabling this option
+     * won't generate server side traces.
      */
-    public Builder enableServerSideTracing() {
-      this.enableServerSideTracing = true;
-      return this;
-    }
-
-    /**
-     * Disable spanner server side tracing. If server side is disabled, trace spans won't be created
-     * at the Spanner layer.
-     */
-    public Builder disableServerSideTracing() {
-      this.enableServerSideTracing = false;
+    public Builder setEnableServerSideTracing(boolean enableServerSideTracing) {
+      this.enableServerSideTracing = enableServerSideTracing;
       return this;
     }
 

google-cloud-spanner/src/test/java/com/google/cloud/spanner/SpannerOptionsTest.java

@@ -738,13 +738,13 @@ public void testServerSideTracingEnablement() {
     assertTrue(
         SpannerOptions.newBuilder()
             .setProjectId("p")
-            .enableServerSideTracing()
+            .setEnableServerSideTracing(true)
             .build()
             .isServerSideTracingEnabled());
     assertFalse(
         SpannerOptions.newBuilder()
             .setProjectId("p")
-            .disableServerSideTracing()
+            .setEnableServerSideTracing(false)
             .build()
             .isServerSideTracingEnabled());
   }

google-cloud-spanner/src/test/java/com/google/cloud/spanner/spi/v1/GapicSpannerRpcTest.java

@@ -493,7 +493,10 @@ public void testNewCallContextWithRouteToLeaderHeaderAndLarDisabled() {
   @Test
   public void testNewCallContextWithServerSideTracingHeader() {
     SpannerOptions options =
-        SpannerOptions.newBuilder().setProjectId("some-project").enableServerSideTracing().build();
+        SpannerOptions.newBuilder()
+            .setProjectId("some-project")
+            .setEnableServerSideTracing(true)
+            .build();
     GapicSpannerRpc rpc = new GapicSpannerRpc(options, false);
     GrpcCallContext callContext =
         rpc.newCallContext(
@@ -514,7 +517,10 @@ public void testNewCallContextWithServerSideTracingHeader() {
   @Test
   public void testNewCallContextWithoutServerSideTracingHeader() {
     SpannerOptions options =
-        SpannerOptions.newBuilder().setProjectId("some-project").disableServerSideTracing().build();
+        SpannerOptions.newBuilder()
+            .setProjectId("some-project")
+            .setEnableServerSideTracing(false)
+            .build();
     GapicSpannerRpc rpc = new GapicSpannerRpc(options, false);
     GrpcCallContext callContext =
         rpc.newCallContext(
@@ -530,7 +536,7 @@ public void testNewCallContextWithoutServerSideTracingHeader() {
   @Test
   public void testServerSideTracingHeaderWithEnabledTracing() {
     final SpannerOptions options =
-        createSpannerOptions().toBuilder().enableServerSideTracing().build();
+        createSpannerOptions().toBuilder().setEnableServerSideTracing(true).build();
     try (Spanner spanner = options.getService()) {
       final DatabaseClient databaseClient =
           spanner.getDatabaseClient(DatabaseId.of("[PROJECT]", "[INSTANCE]", "[DATABASE]"));
@@ -547,7 +553,7 @@ public void testServerSideTracingHeaderWithEnabledTracing() {
   @Test
   public void testServerSideTracingHeaderWithDisabledTracing() {
     final SpannerOptions options =
-        createSpannerOptions().toBuilder().disableServerSideTracing().build();
+        createSpannerOptions().toBuilder().setEnableServerSideTracing(false).build();
     try (Spanner spanner = options.getService()) {
       final DatabaseClient databaseClient =
           spanner.getDatabaseClient(DatabaseId.of("[PROJECT]", "[INSTANCE]", "[DATABASE]"));
@@ -644,7 +650,7 @@ public void testTraceContextHeaderWithOpenTelemetryAndServerSideTracingEnabled()
         createSpannerOptions()
             .toBuilder()
             .setOpenTelemetry(openTelemetry)
-            .enableServerSideTracing()
+            .setEnableServerSideTracing(true)
             .build();
     try (Spanner spanner = options.getService()) {
       final DatabaseClient databaseClient =
@@ -670,7 +676,7 @@ public void testTraceContextHeaderWithOpenTelemetryAndServerSideTracingDisabled(
         createSpannerOptions()
             .toBuilder()
             .setOpenTelemetry(openTelemetry)
-            .disableServerSideTracing()
+            .setEnableServerSideTracing(false)
             .build();
     try (Spanner spanner = options.getService()) {
       final DatabaseClient databaseClient =