FMWK-155 Todos cleanup

src/main/java/org/springframework/data/aerospike/config/AerospikeNamespaceHandler.java

@@ -32,7 +32,6 @@ public class AerospikeNamespaceHandler extends NamespaceHandlerSupport {
     @Override
     public void init() {
 
-        // TODO: add declarations and namespaces for other top-level configuration elements
         registerBeanDefinitionParser("client", new AerospikeClientBeanDefinitionParser());
         registerBeanDefinitionParser("clientPolicy", new ClientPolicyBeanDefinitionParser());
         registerBeanDefinitionParser("readPolicy", new ReadPolicyBeanDefinitionParser());

src/main/java/org/springframework/data/aerospike/config/BatchPolicyBeanDefinitionParser.java

@@ -37,6 +37,5 @@ protected Class<?> getBeanClass(Element element) {
     @Override
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "max-concurrent-threads");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/main/java/org/springframework/data/aerospike/config/ClientPolicyBeanDefinitionParser.java

@@ -39,6 +39,5 @@ protected Class<?> getBeanClass(Element element) {
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "maxThreads");
         ParsingUtils.setPropertyValue(builder, element, "timeOut");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/main/java/org/springframework/data/aerospike/config/QueryPolicyBeanDefinitionParser.java

@@ -37,6 +37,5 @@ protected Class<?> getBeanClass(Element element) {
     @Override
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "max-concurrent-threads");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/main/java/org/springframework/data/aerospike/config/ReadPolicyBeanDefinitionParser.java

@@ -37,6 +37,5 @@ protected Class<?> getBeanClass(Element element) {
     @Override
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "max-concurrent-threads");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/main/java/org/springframework/data/aerospike/config/ScanPolicyBeanDefinitionParser.java

@@ -37,6 +37,5 @@ protected Class<?> getBeanClass(Element element) {
     @Override
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "max-concurrent-threads");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/main/java/org/springframework/data/aerospike/config/WritePolicyBeanDefinitionParser.java

@@ -36,6 +36,5 @@ protected Class<?> getBeanClass(Element element) {
     @Override
     protected void doParse(Element element, BeanDefinitionBuilder builder) {
         ParsingUtils.setPropertyValue(builder, element, "max-concurrent-threads");
-        // TODO: set all properties of the XML element on the builder
     }
 }

src/test/java/org/springframework/data/aerospike/convert/BaseMappingAerospikeConverterTest.java

@@ -62,4 +62,15 @@ private ApplicationContext getApplicationContext() {
 
         return applicationContext;
     }
+
+    protected static Object getBinValue(String name, Collection<Bin> bins) {
+        if (bins == null || bins.isEmpty())
+            return null;
+
+        return bins.stream()
+            .filter(bin -> bin.name.equals(name))
+            .map(bin -> bin.value.getObject())
+            .findFirst().orElse(null);
+    }
+
 }