[#30] Improved Component used by tests (line wrapping and missing ret…

…urn)
4finance · Sep 21, 2014 · 146db90 · 146db90
1 parent 380954b
commit 146db90
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/...ring-base/src/test/groovy/com/ofg/infrastructure/ComponentWithMultipleDependencies.groovy b/...ring-base/src/test/groovy/com/ofg/infrastructure/ComponentWithMultipleDependencies.groovy
@@ -24,12 +24,14 @@ class ComponentWithMultipleDependencies {
     }
 
     boolean hasDependenciesInjectedCorrectly() {
-        return serviceRestClient != null &&
-                    restOperations != null &&
-                        restOperationsImplementedBySpring()
+        return nonNullDependencies() && restOperationsImplementedBySpring()
     }
 
     private boolean restOperationsImplementedBySpring() {
-        restOperations.getClass() == RestTemplate
+        return restOperations.getClass() == RestTemplate
+    }
+
+    private boolean nonNullDependencies() {
+        return serviceRestClient != null && restOperations != null
     }
 }