commit c62488f297e8b090b7578109da2c071ffb6c4d3a
author Christian Aistleitner <christian@quelltextlich.at> Sat Jul 04 00:33:43 2015 +0200
committer Christian Aistleitner <christian@quelltextlich.at> Sat Jul 04 12:39:43 2015 +0200
tree 37fa93b519171f7265bdef1aac538f1c8100d6a0
parent 025114b1861cdf14461f775fabf4ae93f51050d2

Harmonizing test method names in tests of ItsConfig

Now all tests in that class follow the pattern

  "test" + name of tested method + tested aspect

Change-Id: I3f81f267e46196d5eadd34b70b536451a92d3b6c

src/test/java/com/googlesource/gerrit/plugins/hooks/its/ItsConfigTest.java

diff --git a/src/test/java/com/googlesource/gerrit/plugins/hooks/its/ItsConfigTest.java b/src/test/java/com/googlesource/gerrit/plugins/hooks/its/ItsConfigTest.java
index 88b99fd..70d3480 100644
--- a/src/test/java/com/googlesource/gerrit/plugins/hooks/its/ItsConfigTest.java
+++ b/src/test/java/com/googlesource/gerrit/plugins/hooks/its/ItsConfigTest.java
@@ -408,7 +408,7 @@
     assertFalse(itsConfig.isEnabled(event));
   }
 
-  public void testPatternNullMatch() {
+  public void testGetIssuePatternNullMatch() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -422,7 +422,7 @@
         itsConfig.getIssuePattern());
   }
 
-  public void testPatternNullMatchWCommentLink() {
+  public void testGetIssuePatternNullMatchWCommentLink() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -436,7 +436,7 @@
         itsConfig.getIssuePattern());
   }
 
-  public void testPattern() {
+  public void testGetIssuePattern() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -450,7 +450,7 @@
         "TestPattern", itsConfig.getIssuePattern().pattern());
   }
 
-  public void testPatternWCommentLink() {
+  public void testGetIssuePatternWCommentLink() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -465,7 +465,7 @@
 
   }
 
-  public void testPatternGroupIndexGroupDefault() {
+  public void testGetIssuePatternGroupIndexGroupDefault() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -481,7 +481,7 @@
         1, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testPatternGroupIndexGroupDefaultGroupless() {
+  public void testGetIssuePatternGroupIndexGroupDefaultGroupless() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -497,7 +497,7 @@
         0, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testPatternGroupIndexGroup1() {
+  public void testGetIssuePatternGroupIndexGroup1() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -513,7 +513,7 @@
         1, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testPatternGroupIndexGroup3() {
+  public void testGetIssuePatternGroupIndexGroup3() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -529,7 +529,7 @@
         3, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testPatternGroupIndexGroupTooHigh() {
+  public void testGetIssuePatternGroupIndexGroupTooHigh() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -545,7 +545,7 @@
         1, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testPatternGroupIndexGroupTooHighGroupless() {
+  public void testGetIssuePatternGroupIndexGroupTooHighGroupless() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -561,7 +561,7 @@
         0, itsConfig.getIssuePatternGroupIndex());
   }
 
-  public void testItsAssociationPolicyOptional() {
+  public void testGetItsAssociationPolicyOptional() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -577,7 +577,7 @@
         ItsAssociationPolicy.OPTIONAL, itsConfig.getItsAssociationPolicy());
   }
 
-  public void testItsAssociationPolicyOptionalWCommentLink() {
+  public void testGetItsAssociationPolicyOptionalWCommentLink() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -593,7 +593,7 @@
         ItsAssociationPolicy.OPTIONAL, itsConfig.getItsAssociationPolicy());
   }
 
-  public void testItsAssociationPolicySuggested() {
+  public void testGetItsAssociationPolicySuggested() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -609,7 +609,7 @@
         ItsAssociationPolicy.SUGGESTED, itsConfig.getItsAssociationPolicy());
   }
 
-  public void testItsAssociationPolicySuggestedWCommentLink() {
+  public void testGetItsAssociationPolicySuggestedWCommentLink() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -625,7 +625,7 @@
         ItsAssociationPolicy.SUGGESTED, itsConfig.getItsAssociationPolicy());
   }
 
-  public void testItsAssociationPolicyMandatory() {
+  public void testGetItsAssociationPolicyMandatory() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))
@@ -641,7 +641,7 @@
         ItsAssociationPolicy.MANDATORY, itsConfig.getItsAssociationPolicy());
   }
 
-  public void testItsAssociationPolicyMandatoryWCommentLink() {
+  public void testGetItsAssociationPolicyMandatoryWCommentLink() {
     ItsConfig itsConfig = createItsConfig();
 
     expect(serverConfig.getString("ItsTestName", null, "commentlink"))