commit 78280dc391f27e3ec41e8902f51a7d32a944a362
author Chetan Mehrotra <chetanm@adobe.com> Mon Feb 03 21:53:55 2014 +0530
committer Chetan Mehrotra <chetanm@adobe.com> Mon Feb 03 21:53:55 2014 +0530
tree fa9313f05efd344bca806f91acc4f5a6ea2d9ca5
parent a21401d25e0948fffb9a63bcdfb5ccfa6942e306

Minor changes around method names

slf4j-api/src/test/java/org/slf4j/helpers/SubstitutableLoggerTest.java

diff --git a/slf4j-api/src/test/java/org/slf4j/helpers/SubstitutableLoggerTest.java b/slf4j-api/src/test/java/org/slf4j/helpers/SubstitutableLoggerTest.java
index f3f5524..67d3547 100644
--- a/slf4j-api/src/test/java/org/slf4j/helpers/SubstitutableLoggerTest.java
+++ b/slf4j-api/src/test/java/org/slf4j/helpers/SubstitutableLoggerTest.java
@@ -47,7 +47,7 @@
     SubstitutableLogger log = new SubstitutableLogger("foo");
     assertTrue(log.delegate() instanceof NOPLogger);
 
-    Set<String> methodSignatures = determinemethodSignatures(Logger.class);
+    Set<String> expectedMethodSignatures = determineMethodSignatures(Logger.class);
     LoggerInvocationHandler ih = new LoggerInvocationHandler();
     Logger proxyLogger = (Logger) Proxy.newProxyInstance(getClass().getClassLoader(),
         new Class[]{Logger.class}, ih);
@@ -56,9 +56,9 @@
     invokeMethods(log);
 
     //Assert that all methods are delegated
-    methodSignatures.removeAll(ih.getMethodSignatures());
-    if (!methodSignatures.isEmpty()) {
-      fail("Following methods are not delegated " + methodSignatures.toString());
+    expectedMethodSignatures.removeAll(ih.getInvokedMethodSignatures());
+    if (!expectedMethodSignatures.isEmpty()) {
+      fail("Following methods are not delegated " + expectedMethodSignatures.toString());
     }
   }
 
@@ -71,22 +71,22 @@
   }
 
   private class LoggerInvocationHandler implements InvocationHandler {
-    private final Set<String> methodSignatures = new HashSet<String>();
+    private final Set<String> invokedMethodSignatures = new HashSet<String>();
 
     public Object invoke(Object proxy, Method method, Object[] args) throws Throwable {
-      methodSignatures.add(getMethodSignature(method));
+      invokedMethodSignatures.add(getMethodSignature(method));
       if (method.getName().startsWith("is")) {
         return true;
       }
       return null;
     }
 
-    public Set<String> getMethodSignatures() {
-      return methodSignatures;
+    public Set<String> getInvokedMethodSignatures() {
+      return invokedMethodSignatures;
     }
   }
 
-  private static Set<String> determinemethodSignatures(Class<Logger> loggerClass) {
+  private static Set<String> determineMethodSignatures(Class<Logger> loggerClass) {
     Set<String> methodSignatures = new HashSet<String>();
     for (Method m : loggerClass.getDeclaredMethods()) {
       if (!EXCLUDED_METHODS.contains(m.getName())) {