fix: use StatusProto.fromThrowable for ApiException in test-proxy

This should fix the test proxy from prefixing error messages with
"io.grpc.StatusRuntimeException: "

Author: Patrick Wrobel

test-proxy/src/main/java/com/google/cloud/bigtable/testproxy/CbtTestProxy.java

@@ -284,21 +284,13 @@ public void mutateRow(
       return;
     }
 
-    // TODO(developer): evaluate if we want to manually unpack the proto into a model, instead of
-    // using fromProto. Same for the other methods.
     RowMutation mutation = RowMutation.fromProto(request.getRequest());
     try {
       // This response is empty.
       client.dataClient().mutateRow(mutation);
     } catch (ApiException e) {
       responseObserver.onNext(
-          MutateRowResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          MutateRowResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -351,13 +343,7 @@ public void bulkMutateRows(
       return;
     } catch (ApiException e) {
       responseObserver.onNext(
-          MutateRowsResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          MutateRowsResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -410,13 +396,7 @@ public void readRow(ReadRowRequest request, StreamObserver<RowResult> responseOb
       }
     } catch (ApiException e) {
       responseObserver.onNext(
-          RowResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          RowResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -461,13 +441,7 @@ public void readRows(ReadRowsRequest request, StreamObserver<RowsResult> respons
           resultBuilder.setStatus(com.google.rpc.Status.getDefaultInstance()).build());
     } catch (ApiException e) {
       responseObserver.onNext(
-          RowsResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          RowsResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -585,13 +559,7 @@ public void sampleRowKeys(
       keyOffsets = client.dataClient().sampleRowKeys(tableId);
     } catch (ApiException e) {
       responseObserver.onNext(
-          SampleRowKeysResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          SampleRowKeysResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -631,13 +599,7 @@ public void checkAndMutateRow(
       matched = client.dataClient().checkAndMutateRow(mutation);
     } catch (ApiException e) {
       responseObserver.onNext(
-          CheckAndMutateRowResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          CheckAndMutateRowResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -682,13 +644,7 @@ public void readModifyWriteRow(
       }
     } catch (ApiException e) {
       responseObserver.onNext(
-          RowResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          RowResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {
@@ -745,13 +701,7 @@ public void executeQuery(
       return;
     } catch (ApiException e) {
       responseObserver.onNext(
-          ExecuteQueryResult.newBuilder()
-              .setStatus(
-                  com.google.rpc.Status.newBuilder()
-                      .setCode(e.getStatusCode().getCode().ordinal())
-                      .setMessage(e.getMessage())
-                      .build())
-              .build());
+          ExecuteQueryResult.newBuilder().setStatus(StatusProto.fromThrowable(e)).build());
       responseObserver.onCompleted();
       return;
     } catch (StatusRuntimeException e) {