commit 8078749bfd64d9dcd3372bc3f8965d192747c5b4
author David Pursehouse <dpursehouse@collab.net> Mon May 27 14:29:50 2019 +0900
committer David Pursehouse <dpursehouse@collab.net> Mon May 27 14:30:10 2019 +0900
tree 426850e7fbb41c74ba2f22400e0f79a5ffc6901f
parent f26cfcedc3cc9cc5bfdbd530abfb4d85314c1c21

AbstractEventProcessorTest: Make inner class static

When all error prone warnings are enabled the ClassCanBeStatic
bug pattern is reported:

  plugins/webhooks/src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java:87:
  error: [ClassCanBeStatic] Inner class is non-static but does not reference enclosing class
  private class TestEventProcessor extends AbstractEventProcessor {
          ^
    (see https://errorprone.info/bugpattern/ClassCanBeStatic)

Change-Id: I2059a17858581fbb6215f1cf194e05216ad4a130

src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java

diff --git a/src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java b/src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java
index 807f7b0..e4056b4 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/webhooks/processors/AbstractEventProcessorTest.java
@@ -84,7 +84,7 @@
     assertThat(actual).isFalse();
   }
 
-  private class TestEventProcessor extends AbstractEventProcessor {
+  private static class TestEventProcessor extends AbstractEventProcessor {
     @Override
     public Optional<EventProcessor.Request> doProcess(ProjectEvent event, RemoteConfig config) {
       // do nothing