Add test for non retryable error

Author: Quinn-With-Two-Ns

Diff for: temporal-sdk/src/test/java/io/temporal/workflow/nexus/OperationFailMetricTest.java

@@ -337,6 +337,31 @@ public void failHandlerErrorMetrics() {
         });
   }
 
+  @Test
+  public void handlerErrorNonRetryableMetrics() {
+    TestWorkflow1 workflowStub =
+        testWorkflowRule.newWorkflowStubTimeoutOptions(TestWorkflow1.class);
+    WorkflowFailedException workflowException =
+        Assert.assertThrows(
+            WorkflowFailedException.class, () -> workflowStub.execute("handlererror-nonretryable"));
+
+    Map<String, String> execFailedTags =
+        getOperationTags()
+            .put(MetricsTag.TASK_FAILURE_TYPE, "handler_error_INTERNAL")
+            .buildKeepingLast();
+    Eventually.assertEventually(
+        Duration.ofSeconds(3),
+        () -> {
+          reporter.assertTimer(
+              MetricsType.NEXUS_SCHEDULE_TO_START_LATENCY, getBaseTags().buildKeepingLast());
+          reporter.assertTimer(
+              MetricsType.NEXUS_EXEC_LATENCY, getOperationTags().buildKeepingLast());
+          reporter.assertTimer(
+              MetricsType.NEXUS_TASK_E2E_LATENCY, getOperationTags().buildKeepingLast());
+          reporter.assertCounter(MetricsType.NEXUS_EXEC_FAILED_COUNTER, execFailedTags, 1);
+        });
+  }
+
   public static class TestNexus implements TestWorkflow1 {
     @Override
     public String execute(String operation) {
@@ -382,6 +407,11 @@ public OperationHandler<String, String> operation() {
                 throw new HandlerException(
                     HandlerException.ErrorType.BAD_REQUEST,
                     ApplicationFailure.newFailure("intentional failure", "TestFailure", "foo"));
+              case "handlererror-nonretryable":
+                throw new HandlerException(
+                    HandlerException.ErrorType.INTERNAL,
+                    ApplicationFailure.newNonRetryableFailure("intentional failure", "TestFailure"),
+                    HandlerException.RetryBehavior.NON_RETRYABLE);
               case "already-started":
                 throw new WorkflowExecutionAlreadyStarted(
                     WorkflowExecution.getDefaultInstance(), "TestWorkflowType", null);