switch to VisibleForTesting instead of hacky setAccessible

Author: 3vilhamster

src/main/java/com/uber/cadence/internal/replay/ReplayDecider.java

@@ -19,6 +19,7 @@
 
 import static com.uber.cadence.worker.NonDeterministicWorkflowPolicy.FailWorkflow;
 
+import com.google.common.annotations.VisibleForTesting;
 import com.uber.cadence.EventType;
 import com.uber.cadence.GetWorkflowExecutionHistoryRequest;
 import com.uber.cadence.GetWorkflowExecutionHistoryResponse;
@@ -634,6 +635,7 @@ private Duration decisionTaskRemainingTime() {
     private Iterator<HistoryEvent> current;
     private byte[] nextPageToken;
 
+    @VisibleForTesting
     DecisionTaskWithHistoryIteratorImpl(
         PollForDecisionTaskResponse task, Duration decisionTaskStartToCloseTimeout) {
       this.task = Objects.requireNonNull(task);

src/main/java/com/uber/cadence/internal/sync/TestActivityEnvironmentInternal.java

@@ -17,6 +17,7 @@
 
 package com.uber.cadence.internal.sync;
 
+import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Defaults;
 import com.uber.cadence.*;
 import com.uber.cadence.GetTaskListsByDomainRequest;
@@ -133,6 +134,7 @@ private class TestActivityExecutor extends WorkflowInterceptorBase {
     @SuppressWarnings("UnusedVariable")
     private final IWorkflowService workflowService;
 
+    @VisibleForTesting
     TestActivityExecutor(IWorkflowService workflowService, WorkflowInterceptorBase next) {
       super(next);
       this.workflowService = workflowService;
@@ -334,6 +336,7 @@ public CompletableFuture<Boolean> isHealthy() {
       return impl.isHealthy();
     }
 
+    @VisibleForTesting
     private WorkflowServiceWrapper(IWorkflowService impl) {
       if (impl == null) {
         // Create empty implementation that just ignores all requests.