HDDS-15199. Fix StartQuotaRepair client ignoring bucket list

hadoop-ozone/common/src/main/java/org/apache/hadoop/ozone/om/protocolPB/OzoneManagerProtocolClientSideTranslatorPB.java

@@ -2676,6 +2676,7 @@ public String getQuotaRepairStatus() throws IOException {
   public void startQuotaRepair(List<String> buckets) throws IOException {
     OzoneManagerProtocolProtos.StartQuotaRepairRequest startQuotaRepairRequest =
         OzoneManagerProtocolProtos.StartQuotaRepairRequest.newBuilder()
+            .addAllBuckets(buckets)
             .build();
     OMRequest omRequest = createOMRequest(Type.StartQuotaRepair)
         .setStartQuotaRepairRequest(startQuotaRepairRequest).build();

hadoop-ozone/common/src/test/java/org/apache/hadoop/ozone/om/protocolPB/TestOzoneManagerProtocolClientSideTranslatorPB.java

@@ -0,0 +1,92 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
+ * contributor license agreements. See the NOTICE file distributed with
+ * this work for additional information regarding copyright ownership.
+ * The ASF licenses this file to You under the Apache License, Version 2.0
+ * (the "License"); you may not use this file except in compliance with
+ * the License. You may obtain a copy of the License at
+ *
+ *      http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package org.apache.hadoop.ozone.om.protocolPB;
+
+import static org.assertj.core.api.Assertions.assertThat;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import java.io.IOException;
+import java.util.Collections;
+import java.util.List;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.OMRequest;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.OMResponse;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.StartQuotaRepairRequest;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.StartQuotaRepairResponse;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.Status;
+import org.apache.hadoop.ozone.protocol.proto.OzoneManagerProtocolProtos.Type;
+import org.junit.jupiter.api.Test;
+import org.mockito.ArgumentCaptor;
+
+class TestOzoneManagerProtocolClientSideTranslatorPB {
+
+  private final OmTransport omTransport = mock(OmTransport.class);
+  private final OzoneManagerProtocolClientSideTranslatorPB pb = new OzoneManagerProtocolClientSideTranslatorPB(
+      omTransport, "test-client-id");
+
+  @Test
+  void testStartQuotaRepair() throws IOException {
+    StartQuotaRepairResponse response = StartQuotaRepairResponse.newBuilder().build();
+    when(omTransport.submitRequest(any(OMRequest.class))).thenReturn(
+        OMResponse.newBuilder()
+            .setCmdType(Type.StartQuotaRepair)
+            .setStatus(Status.OK)
+            .setStartQuotaRepairResponse(response)
+            .build());
+
+    ArgumentCaptor<OMRequest> captor = ArgumentCaptor.forClass(OMRequest.class);
+
+    pb.startQuotaRepair(Collections.emptyList());
+
+    verify(omTransport).submitRequest(captor.capture());
+
+    OMRequest request = captor.getValue();
+    assertThat(request.getCmdType()).isEqualTo(Type.StartQuotaRepair);
+
+    StartQuotaRepairRequest startQuotaRepairRequest = request.getStartQuotaRepairRequest();
+    assertThat(startQuotaRepairRequest.getBucketsList()).isEmpty();
+  }
+
+  @Test
+  void testStartQuotaRepairWithSpecifiedBuckets() throws IOException {
+    StartQuotaRepairResponse response = StartQuotaRepairResponse.newBuilder().build();
+    when(omTransport.submitRequest(any(OMRequest.class))).thenReturn(
+        OMResponse.newBuilder()
+            .setCmdType(Type.StartQuotaRepair)
+            .setStatus(Status.OK)
+            .setStartQuotaRepairResponse(response)
+            .build());
+
+    ArgumentCaptor<OMRequest> captor = ArgumentCaptor.forClass(OMRequest.class);
+
+    List<String> buckets = Collections.singletonList("Bucket1");
+
+    pb.startQuotaRepair(buckets);
+
+    verify(omTransport).submitRequest(captor.capture());
+
+    OMRequest request = captor.getValue();
+    assertThat(request.getCmdType()).isEqualTo(Type.StartQuotaRepair);
+
+    StartQuotaRepairRequest startQuotaRepairRequest = request.getStartQuotaRepairRequest();
+    assertThat(startQuotaRepairRequest.getBucketsList()).isEqualTo(buckets);
+  }
+
+}