XMLHandler tests covered at 91%

3 files changed, 27 insertions, 27 deletions

diff --git a/src/test/java/musichub/util/LogHandlerTest.java b/src/test/java/musichub/util/LogHandlerTest.java
index 17b47f6..3965520 100644
--- a/src/test/java/musichub/util/LogHandlerTest.java
+++ b/src/test/java/musichub/util/LogHandlerTest.java
@@ -12,6 +12,16 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 
 public class LogHandlerTest {
     @Test
+    void testLogHandlerClass() {
+        try {
+            LogHandler logHandlerClass = new LogHandler();
+        } catch (Error e) {
+            assertTrue(e instanceof AssertionError);
+            assertEquals("You just can't instantiate this class.", e.getMessage());
+        }
+    }
+
+    @Test
     void testWrite() {
         try {
             write("JUnit test", "INFO");
@@ -28,14 +38,4 @@ public class LogHandlerTest {
             e.printStackTrace();
         }
     }
-
-    @Test
-    void testLogHandlerClass() {
-        try {
-            LogHandler logHandlerClass = new LogHandler();
-        } catch (Error ex) {
-            assertTrue(ex instanceof AssertionError);
-            assertEquals("You just can't instantiate this class.", ex.getMessage());
-        }
-    }
 }
diff --git a/src/test/java/musichub/util/PathValidationTest.java b/src/test/java/musichub/util/PathValidationTest.java
index 4c7a3d6..b0e0c38 100644
--- a/src/test/java/musichub/util/PathValidationTest.java
+++ b/src/test/java/musichub/util/PathValidationTest.java
@@ -8,19 +8,19 @@ import static org.junit.jupiter.api.Assertions.*;
 
 public class PathValidationTest {
     @Test
-    void testIsPathValid() {
-        assertTrue(isPathValid("Song/Side_To_Side.wav")); //the right path
-        assertFalse(isPathValid("wrong_path/Side_To_Side.wav")); //wrong path
-        assertFalse(isPathValid("Song/Side_To_Side.mp3")); //wrong extension
-    }
-
-    @Test
     void testPathValidationClass() {
         try {
             PathValidation pathValidationClass = new PathValidation();
-        } catch (Error ex) {
-            assertTrue(ex instanceof AssertionError);
-            assertEquals("You just can't instantiate this class.", ex.getMessage());
+        } catch (Error e) {
+            assertTrue(e instanceof AssertionError);
+            assertEquals("You just can't instantiate this class.", e.getMessage());
         }
     }
+
+    @Test
+    void testIsPathValid() {
+        assertTrue(isPathValid("Song/Side_To_Side.wav")); //the right path
+        assertFalse(isPathValid("wrong_path/Side_To_Side.wav")); //wrong path
+        assertFalse(isPathValid("Song/Side_To_Side.mp3")); //wrong extension
+    }
 }
diff --git a/src/test/java/musichub/util/XMLHandlerTest.java b/src/test/java/musichub/util/XMLHandlerTest.java
index 1be2258..7b5a791 100644
--- a/src/test/java/musichub/util/XMLHandlerTest.java
+++ b/src/test/java/musichub/util/XMLHandlerTest.java
@@ -10,11 +10,6 @@ public class XMLHandlerTest {
     final String DIR = System.getProperty("user.dir");
     final String JUNIT_FILE_PATH = DIR + "/files/JUnit.xml";
 
-/*    @Test
-    void testXMLHandlerClass() {
-            XMLHandler xmlHandler = new XMLHandler();
-    }*/
-
     @Test
     void testCreateXML() {
         XMLHandler xmlHandler = new XMLHandler(); //XML class
@@ -24,7 +19,12 @@ public class XMLHandlerTest {
 
     @Test
     void testParseXMLFile() {
+        final String PARSE_DIR = System.getProperty("user.dir");
+        final String PARSE_FILE_PATH = PARSE_DIR + "/files/parse_JUnit.xml";
         XMLHandler xmlHandler = new XMLHandler();
-        NodeList albumNodes = xmlHandler.parseXMLFile(JUNIT_FILE_PATH);
+        //wrong content of file resulting in an exception, will print it during test but it's normal
+        NodeList junitNodes = xmlHandler.parseXMLFile(JUNIT_FILE_PATH);
+        //right content of file
+        NodeList parseNodes = xmlHandler.parseXMLFile(PARSE_FILE_PATH);
     }
-}
+}
\ No newline at end of file