From 0eec00fd58cebe0f3c2e1d5e7ee6e8752d9f2827 Mon Sep 17 00:00:00 2001
From: David Pursehouse <dpursehouse@digital.ai>
Date: Thu, 18 Jun 2020 15:53:30 +0900
Subject: [PATCH] TestSubmitType: Enable logErrors when debug logging is
 enabled

Change-Id: Ib46367271496bb35e702b6400e988b3095c190f1
---
 .../google/gerrit/server/restapi/change/TestSubmitType.java  | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/java/com/google/gerrit/server/restapi/change/TestSubmitType.java b/java/com/google/gerrit/server/restapi/change/TestSubmitType.java
index c1be1cea00..e67a319bac 100644
--- a/java/com/google/gerrit/server/restapi/change/TestSubmitType.java
+++ b/java/com/google/gerrit/server/restapi/change/TestSubmitType.java
@@ -15,6 +15,7 @@
 package com.google.gerrit.server.restapi.change;
 
 import com.google.common.base.MoreObjects;
+import com.google.common.flogger.FluentLogger;
 import com.google.gerrit.common.data.SubmitTypeRecord;
 import com.google.gerrit.extensions.client.SubmitType;
 import com.google.gerrit.extensions.common.TestSubmitRuleInput;
@@ -35,6 +36,8 @@ import com.google.inject.Provider;
 import org.kohsuke.args4j.Option;
 
 public class TestSubmitType implements RestModifyView<RevisionResource, TestSubmitRuleInput> {
+  private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
   private final Provider<ReviewDb> db;
   private final ChangeData.Factory changeDataFactory;
   private final RulesCache rules;
@@ -68,7 +71,7 @@ public class TestSubmitType implements RestModifyView<RevisionResource, TestSubm
 
     SubmitRuleOptions opts =
         SubmitRuleOptions.builder()
-            .logErrors(false)
+            .logErrors(logger.atFine().isEnabled())
             .skipFilters(input.filters == Filters.SKIP)
             .rule(input.rule)
             .build();