Fix incorrect exception handling in WorkflowTchannelClient and add unit test coverage

build.gradle

@@ -84,6 +84,7 @@ dependencies {
     testCompile group: 'io.grpc', name: 'grpc-testing', version: '1.54.2'
     testImplementation 'io.opentracing:opentracing-mock:0.33.0'
     testImplementation group: 'org.mockito', name: 'mockito-core', version: '4.5.1'
+    testImplementation "org.mockito:mockito-inline:4.5.1" // for mocking final classes
 }
 
 license {

src/main/java/com/uber/cadence/serviceclient/WorkflowServiceTChannel.java

@@ -776,6 +776,9 @@ private GetWorkflowExecutionHistoryResponse getWorkflowExecutionHistory(
       if (result.isSetServiceBusyError()) {
         throw result.getServiceBusyError();
       }
+      if (result.isSetClientVersionNotSupportedError()) {
+        throw result.getClientVersionNotSupportedError();
+      }
       throw new TException("GetWorkflowExecutionHistory failed with unknown error:" + result);
     } finally {
       if (response != null) {
@@ -2090,6 +2093,9 @@ private QueryWorkflowResponse queryWorkflow(QueryWorkflowRequest queryRequest) t
       if (result.isSetBadRequestError()) {
         throw result.getBadRequestError();
       }
+      if (result.isSetServiceBusyError()) {
+        throw result.getServiceBusyError();
+      }
       if (result.isSetEntityNotExistError()) {
         throw result.getEntityNotExistError();
       }
@@ -2265,6 +2271,9 @@ private ClusterInfo getClusterInfo() throws TException {
       if (result.isSetServiceBusyError()) {
         throw result.getServiceBusyError();
       }
+      if (result.isSetInternalServiceError()) {
+        throw result.getInternalServiceError();
+      }
       throw new TException("GetClusterInfo failed with unknown error:" + result);
     } finally {
       if (response != null) {
@@ -2295,6 +2304,9 @@ public void RefreshWorkflowTasks(RefreshWorkflowTasksRequest refreshWorkflowTask
       response = doRemoteCall(request);
       WorkflowService.RefreshWorkflowTasks_result result =
           response.getBody(WorkflowService.RefreshWorkflowTasks_result.class);
+      if (response.getResponseCode() == ResponseCode.OK) {
+        return;
+      }
       if (result.isSetBadRequestError()) {
         throw result.getBadRequestError();
       }
@@ -2307,6 +2319,7 @@ public void RefreshWorkflowTasks(RefreshWorkflowTasksRequest refreshWorkflowTask
       if (result.isSetEntityNotExistError()) {
         throw result.getEntityNotExistError();
       }
+      throw new TException("RefreshWorkflowTasks failed with unknown error:" + result);
     } finally {
       if (response != null) {
         response.release();