Merge "Add TestWatcher to log starting/finished messages"
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/FindOwnersIT.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/FindOwnersIT.java
index d50a1c0..3233024 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/FindOwnersIT.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/FindOwnersIT.java
@@ -20,6 +20,7 @@
import static java.nio.charset.StandardCharsets.UTF_8;
import com.google.common.collect.Multimap;
+import com.google.common.flogger.FluentLogger;
import com.google.gerrit.acceptance.LightweightPluginDaemonTest;
import com.google.gerrit.acceptance.PushOneCommit;
import com.google.gerrit.acceptance.RestResponse;
@@ -50,11 +51,28 @@
import org.eclipse.jgit.treewalk.TreeWalk;
import org.eclipse.jgit.treewalk.filter.PathFilterGroup;
import org.junit.Before;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
+import org.junit.runner.Description;
/** Test find-owners plugin API. */
@TestPlugin(name = "find-owners", sysModule = "com.googlesource.gerrit.plugins.findowners.Module")
public class FindOwnersIT extends LightweightPluginDaemonTest {
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
@Inject private Emails emails;
@Inject private ProjectOperations projectOperations;
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnerWeightsTest.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnerWeightsTest.java
index 183a222..65abbe7 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnerWeightsTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnerWeightsTest.java
@@ -16,17 +16,35 @@
import static com.google.common.truth.Truth.assertThat;
+import com.google.common.flogger.FluentLogger;
import com.googlesource.gerrit.plugins.findowners.OwnerWeights.WeightComparator;
import java.util.HashMap;
import java.util.List;
import java.util.Map;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
+import org.junit.runner.Description;
/** Test OwnerWeights class */
@RunWith(JUnit4.class)
public class OwnerWeightsTest {
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
private static OwnerWeights createOwnerWeights(int[] counts) {
OwnerWeights obj = new OwnerWeights();
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnersValidatorTest.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnersValidatorTest.java
index 769ddbd..b0444bc 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnersValidatorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/OwnersValidatorTest.java
@@ -24,6 +24,7 @@
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.ImmutableSetMultimap;
import com.google.common.collect.Lists;
+import com.google.common.flogger.FluentLogger;
import com.google.gerrit.reviewdb.client.Account;
import com.google.gerrit.server.account.Emails;
import com.google.gerrit.server.config.PluginConfig;
@@ -47,13 +48,30 @@
import org.eclipse.jgit.storage.file.FileRepositoryBuilder;
import org.junit.After;
import org.junit.Before;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
+import org.junit.runner.Description;
/** Test OwnersValidator, which checks syntax of changed OWNERS files. */
@RunWith(JUnit4.class)
public class OwnersValidatorTest {
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
private static class MockedEmails extends Emails {
Set<String> registered;
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/ParserTest.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/ParserTest.java
index a26e952..15431fe 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/ParserTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/ParserTest.java
@@ -16,14 +16,33 @@
import static com.google.common.truth.Truth.assertThat;
+import com.google.common.flogger.FluentLogger;
import java.util.Arrays;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
+import org.junit.runner.Description;
/** Test Parser class */
@RunWith(JUnit4.class)
public class ParserTest {
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
+
private static String mockedTestDir() {
return "./d1/d2/";
}
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/PrologIT.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/PrologIT.java
index 3da1794..258e708 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/PrologIT.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/PrologIT.java
@@ -16,6 +16,7 @@
import static com.google.common.truth.Truth.assertThat;
+import com.google.common.flogger.FluentLogger;
import com.google.gerrit.acceptance.GitUtil;
import com.google.gerrit.acceptance.LightweightPluginDaemonTest;
import com.google.gerrit.acceptance.PushOneCommit;
@@ -31,7 +32,10 @@
import find_owners.PRED_submit_rule_1;
import find_owners.PRED_submit_rule_2;
import org.eclipse.jgit.revwalk.RevCommit;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
+import org.junit.runner.Description;
/** Test existence of compiled find-owners plugin Prolog predicates. */
@TestPlugin(name = "find-owners", sysModule = "com.googlesource.gerrit.plugins.findowners.Module")
@@ -39,6 +43,21 @@
// This class now only tests existence of compiled Prolog predicates.
// Later we can test the functionality of the predicates.
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
+
@Test
public void predefinedPredicateTest() throws Exception {
Object term = new PRED_check_owner_approval_2(null, null, null);
diff --git a/src/test/java/com/googlesource/gerrit/plugins/findowners/UtilTest.java b/src/test/java/com/googlesource/gerrit/plugins/findowners/UtilTest.java
index 1ddd4ec..6411e3a 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/findowners/UtilTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/findowners/UtilTest.java
@@ -16,18 +16,36 @@
import static com.google.common.truth.Truth.assertThat;
+import com.google.common.flogger.FluentLogger;
import java.io.IOException;
import java.util.HashMap;
import java.util.HashSet;
import java.util.Map;
import java.util.Set;
+import org.junit.Rule;
import org.junit.Test;
+import org.junit.rules.TestWatcher;
import org.junit.runner.RunWith;
import org.junit.runners.JUnit4;
+import org.junit.runner.Description;
/** Test Util class */
@RunWith(JUnit4.class)
public class UtilTest {
+ private static final FluentLogger logger = FluentLogger.forEnclosingClass();
+
+ @Rule
+ public TestWatcher watcher = new TestWatcher() {
+ @Override
+ public void starting(final Description method) {
+ logger.atInfo().log("Test starting: " + method.getMethodName());
+ }
+
+ @Override
+ public void finished(final Description method) {
+ logger.atInfo().log("Test finished: " + method.getMethodName());
+ }
+ };
@Test
public void getOwner2WeightsTest() {