diff --git a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateDTD.java b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateDTD.java index b04e8e0cd956ca22505084892bcbdb8a4f72db71..5279f0b5b987c9de8781304b1c18c277e0769325 100644 --- a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateDTD.java +++ b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateDTD.java @@ -50,8 +50,8 @@ public class TestSLUBValidateDTD { } catch ( Exception e) { has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("dtd validation should not raise an exception", has_exception ); + assertFalse( "dtd validation should return false for failed validation", res); } @org.junit.Test public void dtd_invalid2() { @@ -72,8 +72,8 @@ public class TestSLUBValidateDTD { } catch ( Exception e) { has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse( "dtd validation should not raise an exception", has_exception ); + assertFalse( "dtd validation should return false for failed validation", res); } @@ -100,8 +100,8 @@ public class TestSLUBValidateDTD { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertTrue( res); + assertFalse("dtd validation should not raise an exception", has_exception ); + assertTrue("dtd validation should return true for sucessful validation", res); } @org.junit.Test @@ -127,8 +127,8 @@ public class TestSLUBValidateDTD { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertTrue( res); + assertFalse("dtd validation should not raise an exception", has_exception ); + assertTrue("dtd validation should return true for sucessful validation", res); } @@ -151,8 +151,8 @@ public class TestSLUBValidateDTD { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("dtd validation should not raise an exception", has_exception ); + assertFalse("dtd validation should return false for failed validation", res); } @org.junit.Test @@ -176,7 +176,7 @@ public class TestSLUBValidateDTD { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("dtd validation should not raise an exception", has_exception ); + assertFalse("dtd validation should return false for failed validation", res); } } diff --git a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateRelaxNG.java b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateRelaxNG.java index 783cef6c9b9cde2da624241ce8fba6ee69e266f7..85ef1920fe4c30ed5ddaad841f9c6bd07dc931fc 100644 --- a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateRelaxNG.java +++ b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateRelaxNG.java @@ -59,8 +59,8 @@ public class TestSLUBValidateRelaxNG { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertTrue( res); + assertFalse("relaxNG validation should not raise an exception", has_exception ); + assertTrue("relaxNG validation should return true for sucessful validation", res); } @org.junit.Test public void relaxNG_invalid() { @@ -76,7 +76,7 @@ public class TestSLUBValidateRelaxNG { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("relaxNG validation should not raise an exception", has_exception ); + assertFalse( "relaxNG validation should return false for failed validation", res); } } diff --git a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateSchema.java b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateSchema.java index 51ff0bd9e5d3c8bbbcbc2078791591a0b07b36b7..77d0f588b5aabd5f1f55afd9309d0e9c67db21e1 100644 --- a/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateSchema.java +++ b/java/org/slub/rosetta/dps/repository/plugin/Validation/TestSLUBValidateSchema.java @@ -64,8 +64,8 @@ public class TestSLUBValidateSchema { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertTrue( res); + assertFalse("schema validation should not raise an exception", has_exception ); + assertTrue("schema validation should return true for sucessful validation", res); } @org.junit.Test @@ -83,8 +83,8 @@ public class TestSLUBValidateSchema { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("schema validation should not raise an exception", has_exception ); + assertFalse("schema validation should return false for failed validation", res); } @org.junit.Test @@ -93,7 +93,7 @@ public class TestSLUBValidateSchema { Set<ValidationSchema> old = mock.getNamespaceSchemaMap(); for (ValidationSchema val : old) { // filter Atoim out to trigger error System.out.println("###### " + val.nameSpace); - if (!val.nameSpace.equals("http://www.w3.org/2005/Atom")) { + if (!"http://www.w3.org/2005/Atom".equals(val.nameSpace)) { nssm.add(val); } } @@ -110,7 +110,7 @@ public class TestSLUBValidateSchema { e.printStackTrace(); has_exception = true; } - assertFalse( has_exception ); - assertFalse( res); + assertFalse("schema validation should not raise an exception", has_exception ); + assertFalse("schema validation should return false for failed validation", res); } }