some more editorial changes

4 files changed, 28 insertions, 13 deletions

diff --git a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/QaaLevelVerifierTest.java b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/QaaLevelVerifierTest.java
index 44cdf111..c2530004 100644
--- a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/QaaLevelVerifierTest.java
+++ b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/QaaLevelVerifierTest.java
@@ -27,7 +27,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
     try {
@@ -35,7 +35,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
   }
@@ -54,7 +54,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
   }
@@ -69,7 +69,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
     QaaLevelVerifier.verifyQaaLevel(EaafConstants.EIDAS_LOA_SUBSTANTIAL, requiredLoAs, matchingMode);
@@ -80,7 +80,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
   }
@@ -95,7 +95,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
     try {
@@ -103,7 +103,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
     
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
     QaaLevelVerifier.verifyQaaLevel(EaafConstants.EIDAS_LOA_HIGH, requiredLoAs, matchingMode);
@@ -113,7 +113,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
   }
@@ -128,7 +128,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
         
     QaaLevelVerifier.verifyQaaLevel(EaafConstants.EIDAS_LOA_LOW, requiredLoAs, matchingMode);
@@ -139,7 +139,7 @@ public class QaaLevelVerifierTest {
       Assert.fail("LoA should not be allowed");
       
     } catch (QaaNotAllowedException e) {
-      
+      Assert.assertNotNull("No errorMsg", e.getMessage());
     }
     
   }
diff --git a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/binding/PostBindingTest.java b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/binding/PostBindingTest.java
index 2449c73c..f14a9093 100644
--- a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/binding/PostBindingTest.java
+++ b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/binding/PostBindingTest.java
@@ -457,7 +457,9 @@ public class PostBindingTest {
     try {
       Assert.assertNotNull("AuthnReq is null", msg.getInboundMessage());
 
-    } catch (final RuntimeException e) { }
+    } catch (final RuntimeException e) { 
+      Assert.assertNotNull("No errorMsg", e.getMessage());
+    }
 
   }
 
diff --git a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/dummy/DummyCredentialProvider.java b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/dummy/DummyCredentialProvider.java
index 7418e1b3..1fe9afcf 100644
--- a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/dummy/DummyCredentialProvider.java
+++ b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/dummy/DummyCredentialProvider.java
@@ -1,13 +1,13 @@
 package at.gv.egiz.eaaf.modules.pvp2.test.dummy;
 
+import org.springframework.beans.factory.annotation.Autowired;
+
 import at.gv.egiz.eaaf.core.api.idp.IConfiguration;
 import at.gv.egiz.eaaf.core.exceptions.EaafConfigurationException;
 import at.gv.egiz.eaaf.core.impl.credential.KeyStoreConfiguration;
 import at.gv.egiz.eaaf.core.impl.credential.KeyStoreConfiguration.KeyStoreType;
 import at.gv.egiz.eaaf.modules.pvp2.impl.utils.AbstractCredentialProvider;
 
-import org.springframework.beans.factory.annotation.Autowired;
-
 public class DummyCredentialProvider extends AbstractCredentialProvider {
 
   @Autowired IConfiguration basicConfig;
@@ -49,6 +49,11 @@ public class DummyCredentialProvider extends AbstractCredentialProvider {
 
   }
 
+  /**
+   * Get Path to keystore.
+   * 
+   * @return
+   */
   public String getKeyStoreFilePath() {
     final String path = basicConfig.getBasicConfiguration(KEYSTORE_PATH);
     return path;
diff --git a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/metadata/MetadataBuilderTest.java b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/metadata/MetadataBuilderTest.java
index 5b09d50d..3cc0a908 100644
--- a/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/metadata/MetadataBuilderTest.java
+++ b/eaaf_modules/eaaf_module_pvp2_core/src/test/java/at/gv/egiz/eaaf/modules/pvp2/test/metadata/MetadataBuilderTest.java
@@ -159,6 +159,14 @@ public class MetadataBuilderTest {
     return entity;
   }
 
+  /**
+   * Dummy Metadata builder configuration.
+   * 
+   * @param credentialProvider Credentialprovider
+   * @param buildSpInfos Sp metadata flag
+   * @param buildIdpInfos IDP metadata flag
+   * @return
+   */
   public static IPvpMetadataBuilderConfiguration idpMetadataConfig(
       IPvp2CredentialProvider credentialProvider, boolean buildSpInfos, boolean buildIdpInfos) {
     return new IPvpMetadataBuilderConfiguration() {