From a846c037cbe24aac76faeee1f4c3e3091e6ca8d4 Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Fri, 30 Aug 2019 00:34:11 +0200
Subject: [PATCH 1/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 .../java/com/baeldung/sax/SaxParserMain.java  | 161 ++++++++++++++++++
 xml/src/main/resources/sax/baeldung.xml       |  16 ++
 2 files changed, 177 insertions(+)
 create mode 100644 xml/src/main/java/com/baeldung/sax/SaxParserMain.java
 create mode 100644 xml/src/main/resources/sax/baeldung.xml

diff --git a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
new file mode 100644
index 0000000000..fdf9b524d0
--- /dev/null
+++ b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
@@ -0,0 +1,161 @@
+package com.baeldung.sax;
+
+import org.xml.sax.Attributes;
+import org.xml.sax.SAXException;
+import org.xml.sax.helpers.DefaultHandler;
+
+import javax.xml.parsers.ParserConfigurationException;
+import javax.xml.parsers.SAXParser;
+import javax.xml.parsers.SAXParserFactory;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+import java.util.Objects;
+
+public class SaxParserMain {
+    public static void main(String[] args) throws ParserConfigurationException, SAXException, IOException {
+        SAXParserFactory factory = SAXParserFactory.newInstance();
+        SAXParser saxParser = factory.newSAXParser();
+
+        BaeldungHandler baeldungHandler = new BaeldungHandler();
+        saxParser.parse("xml\\src\\main\\resources\\sax\\baeldung.xml", baeldungHandler);
+        System.out.println(baeldungHandler.getWebsite());
+    }
+
+    public static class BaeldungHandler extends DefaultHandler {
+        private Baeldung website;
+        private String elementValue;
+
+        @Override
+        public void characters(char[] ch, int start, int length) throws SAXException {
+            elementValue = new String(ch, start, length);
+        }
+
+        @Override
+        public void startDocument() throws SAXException {
+            website = new Baeldung();
+        }
+
+        @Override
+        public void startElement(String uri, String localName, String qName, Attributes attributes) throws SAXException {
+            switch (qName) {
+                case ARTICLES:
+                    website.setArticleList(new ArrayList<>());
+                    break;
+                case ARTICLE:
+                    website.getArticleList().add(new BaeldungArticle());
+            }
+        }
+
+        @Override
+        public void endElement(String uri, String localName, String qName) throws SAXException {
+            switch (qName) {
+                case TITLE:
+                    latestArticle().setTitle(elementValue);
+                    break;
+                case CONTENT:
+                    latestArticle().setContent(elementValue);
+                    break;
+            }
+        }
+
+        private BaeldungArticle latestArticle() {
+            List<BaeldungArticle> articleList = website.getArticleList();
+            int latestArticleIndex = articleList.size() - 1;
+            return articleList.get(latestArticleIndex);
+        }
+
+        public Baeldung getWebsite() {
+            return website;
+        }
+
+        @Override
+        public String toString() {
+            return "BaeldungHandler{" +
+                    "website=" + website +
+                    ", elementValue='" + elementValue + '\'' +
+                    '}';
+        }
+
+        private static final String ARTICLES = "articles";
+        private static final String ARTICLE = "article";
+        private static final String TITLE = "title";
+        private static final String CONTENT = "content";
+    }
+
+
+    public static class Baeldung {
+        private List<BaeldungArticle> articleList;
+
+        public List<BaeldungArticle> getArticleList() {
+            return articleList;
+        }
+
+        public void setArticleList(List<BaeldungArticle> articleList) {
+            this.articleList = articleList;
+        }
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) return true;
+            if (o == null || getClass() != o.getClass()) return false;
+            Baeldung baeldung = (Baeldung) o;
+            return Objects.equals(articleList, baeldung.articleList);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(articleList);
+        }
+
+        @Override
+        public String toString() {
+            return "Baeldung{" +
+                    "articleList=" + articleList +
+                    '}';
+        }
+    }
+
+    public static class BaeldungArticle {
+        private String title;
+        private String content;
+
+        public String getTitle() {
+            return title;
+        }
+
+        public void setTitle(String title) {
+            this.title = title;
+        }
+
+        public String getContent() {
+            return content;
+        }
+
+        public void setContent(String content) {
+            this.content = content;
+        }
+
+        @Override
+        public boolean equals(Object o) {
+            if (this == o) return true;
+            if (o == null || getClass() != o.getClass()) return false;
+            BaeldungArticle that = (BaeldungArticle) o;
+            return Objects.equals(title, that.title) &&
+                    Objects.equals(content, that.content);
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(title, content);
+        }
+
+        @Override
+        public String toString() {
+            return "BaeldungArticle{" +
+                    "title='" + title + '\'' +
+                    ", content='" + content + '\'' +
+                    '}';
+        }
+    }
+}
diff --git a/xml/src/main/resources/sax/baeldung.xml b/xml/src/main/resources/sax/baeldung.xml
new file mode 100644
index 0000000000..4b839f35c7
--- /dev/null
+++ b/xml/src/main/resources/sax/baeldung.xml
@@ -0,0 +1,16 @@
+<baeldung>
+    <articles>
+        <article>
+            <title>Parsing an XML File Using SAX Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+        <article>
+            <title>Parsing an XML File Using DOM Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+        <article>
+            <title>Parsing an XML File Using StAX Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+    </articles>
+</baeldung>
\ No newline at end of file

From 6e77eef313253f92b4c9643e1e75f72cce41e7ba Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sat, 31 Aug 2019 13:38:40 +0200
Subject: [PATCH 2/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 .../com/baeldung/sax/SaxParserMainTest.java   | 44 +++++++++++++++++++
 xml/src/test/resources/sax/baeldung.xml       | 16 +++++++
 2 files changed, 60 insertions(+)
 create mode 100644 xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java
 create mode 100644 xml/src/test/resources/sax/baeldung.xml

diff --git a/xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java b/xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java
new file mode 100644
index 0000000000..af31bc3bd2
--- /dev/null
+++ b/xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java
@@ -0,0 +1,44 @@
+package com.baeldung.sax;
+
+import org.junit.Test;
+import org.xml.sax.SAXException;
+
+import javax.xml.parsers.ParserConfigurationException;
+import javax.xml.parsers.SAXParser;
+import javax.xml.parsers.SAXParserFactory;
+import java.io.IOException;
+import java.util.List;
+
+import static org.junit.Assert.*;
+
+public class SaxParserMainTest {
+
+    @Test
+    void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
+        SAXParserFactory factory = SAXParserFactory.newInstance();
+        SAXParser saxParser = factory.newSAXParser();
+
+        SaxParserMain.BaeldungHandler baeldungHandler = new SaxParserMain.BaeldungHandler();
+        saxParser.parse("xml\\src\\test\\resources\\sax\\baeldung.xml", baeldungHandler);
+
+        SaxParserMain.Baeldung result = baeldungHandler.getWebsite();
+
+        assertNotNull(result);
+        List<SaxParserMain.BaeldungArticle> articles = result.getArticleList();
+
+        assertNotNull(articles);
+        assertEquals(3, articles.size());
+
+        SaxParserMain.BaeldungArticle articleOne = articles.get(0);
+        assertEquals("Parsing an XML File Using SAX Parser", articleOne.getTitle());
+        assertEquals("Lorem ipsum...", articleOne.getContent());
+
+        SaxParserMain.BaeldungArticle articleTwo = articles.get(1);
+        assertEquals("Parsing an XML File Using DOM Parser", articleTwo.getTitle());
+        assertEquals("Lorem ipsum...", articleTwo.getContent());
+
+        SaxParserMain.BaeldungArticle articleThree = articles.get(2);
+        assertEquals("Parsing an XML File Using StAX Parser", articleThree.getTitle());
+        assertEquals("Lorem ipsum...", articleThree.getContent());
+    }
+}
diff --git a/xml/src/test/resources/sax/baeldung.xml b/xml/src/test/resources/sax/baeldung.xml
new file mode 100644
index 0000000000..4b839f35c7
--- /dev/null
+++ b/xml/src/test/resources/sax/baeldung.xml
@@ -0,0 +1,16 @@
+<baeldung>
+    <articles>
+        <article>
+            <title>Parsing an XML File Using SAX Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+        <article>
+            <title>Parsing an XML File Using DOM Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+        <article>
+            <title>Parsing an XML File Using StAX Parser</title>
+            <content>Lorem ipsum...</content>
+        </article>
+    </articles>
+</baeldung>
\ No newline at end of file

From 21804927e3b2bdd1edcad9af94581edd8b445fe3 Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sat, 31 Aug 2019 15:40:44 +0200
Subject: [PATCH 3/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 .../{SaxParserMainTest.java => SaxParserMainUnitTest.java}  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
 rename xml/src/test/java/com/baeldung/sax/{SaxParserMainTest.java => SaxParserMainUnitTest.java} (84%)

diff --git a/xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
similarity index 84%
rename from xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java
rename to xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
index af31bc3bd2..d9351aed03 100644
--- a/xml/src/test/java/com/baeldung/sax/SaxParserMainTest.java
+++ b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
@@ -11,15 +11,15 @@ import java.util.List;
 
 import static org.junit.Assert.*;
 
-public class SaxParserMainTest {
+public class SaxParserMainUnitTest {
 
     @Test
-    void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
+    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
         SAXParserFactory factory = SAXParserFactory.newInstance();
         SAXParser saxParser = factory.newSAXParser();
 
         SaxParserMain.BaeldungHandler baeldungHandler = new SaxParserMain.BaeldungHandler();
-        saxParser.parse("xml\\src\\test\\resources\\sax\\baeldung.xml", baeldungHandler);
+        saxParser.parse("src\\test\\resources\\sax\\baeldung.xml", baeldungHandler);
 
         SaxParserMain.Baeldung result = baeldungHandler.getWebsite();
 

From 08da9be66649c6056a6490f25ac7d446172e23a6 Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sat, 31 Aug 2019 21:48:19 +0200
Subject: [PATCH 4/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 .../java/com/baeldung/sax/SaxParserMainUnitTest.java   | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
index d9351aed03..d94857dbb5 100644
--- a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
+++ b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
@@ -7,6 +7,11 @@ import javax.xml.parsers.ParserConfigurationException;
 import javax.xml.parsers.SAXParser;
 import javax.xml.parsers.SAXParserFactory;
 import java.io.IOException;
+import java.net.URI;
+import java.net.URISyntaxException;
+import java.net.URL;
+import java.nio.file.Path;
+import java.nio.file.Paths;
 import java.util.List;
 
 import static org.junit.Assert.*;
@@ -14,12 +19,13 @@ import static org.junit.Assert.*;
 public class SaxParserMainUnitTest {
 
     @Test
-    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
+    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException, URISyntaxException {
         SAXParserFactory factory = SAXParserFactory.newInstance();
         SAXParser saxParser = factory.newSAXParser();
 
         SaxParserMain.BaeldungHandler baeldungHandler = new SaxParserMain.BaeldungHandler();
-        saxParser.parse("src\\test\\resources\\sax\\baeldung.xml", baeldungHandler);
+        Path path = Paths.get(new URL("src\\test\\resources\\sax\\baeldung.xml").toURI());
+        saxParser.parse(path.toFile(), baeldungHandler);
 
         SaxParserMain.Baeldung result = baeldungHandler.getWebsite();
 

From 6f4957a63d07358c4bfa9acc4819d0164b15b6dd Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sat, 31 Aug 2019 21:57:51 +0200
Subject: [PATCH 5/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 .../java/com/baeldung/sax/SaxParserMainUnitTest.java   | 10 ++--------
 1 file changed, 2 insertions(+), 8 deletions(-)

diff --git a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
index d94857dbb5..8a8aa62dd2 100644
--- a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
+++ b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
@@ -7,11 +7,6 @@ import javax.xml.parsers.ParserConfigurationException;
 import javax.xml.parsers.SAXParser;
 import javax.xml.parsers.SAXParserFactory;
 import java.io.IOException;
-import java.net.URI;
-import java.net.URISyntaxException;
-import java.net.URL;
-import java.nio.file.Path;
-import java.nio.file.Paths;
 import java.util.List;
 
 import static org.junit.Assert.*;
@@ -19,13 +14,12 @@ import static org.junit.Assert.*;
 public class SaxParserMainUnitTest {
 
     @Test
-    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException, URISyntaxException {
+    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
         SAXParserFactory factory = SAXParserFactory.newInstance();
         SAXParser saxParser = factory.newSAXParser();
 
         SaxParserMain.BaeldungHandler baeldungHandler = new SaxParserMain.BaeldungHandler();
-        Path path = Paths.get(new URL("src\\test\\resources\\sax\\baeldung.xml").toURI());
-        saxParser.parse(path.toFile(), baeldungHandler);
+        saxParser.parse("src/test/resources/sax/baeldung.xml", baeldungHandler);
 
         SaxParserMain.Baeldung result = baeldungHandler.getWebsite();
 

From 59729b7e2339cd63072566db388691b22112f6bc Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sun, 1 Sep 2019 21:37:01 +0200
Subject: [PATCH 6/9] BAEL-3219 Parsing an XML File Using SAX Parser

---
 xml/pom.xml                                   |  7 ++
 .../java/com/baeldung/sax/SaxParserMain.java  | 93 +++----------------
 2 files changed, 18 insertions(+), 82 deletions(-)

diff --git a/xml/pom.xml b/xml/pom.xml
index bbd607f9b5..3b8b88ba73 100644
--- a/xml/pom.xml
+++ b/xml/pom.xml
@@ -111,6 +111,13 @@
             <version>${commons-lang.version}</version>
         </dependency>
 
+        <dependency>
+            <groupId>org.projectlombok</groupId>
+            <artifactId>lombok</artifactId>
+            <version>1.18.8</version>
+            <scope>provided</scope>
+        </dependency>
+
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>
diff --git a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
index fdf9b524d0..654d5162cd 100644
--- a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
+++ b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
@@ -1,5 +1,7 @@
 package com.baeldung.sax;
 
+import lombok.Data;
+import lombok.ToString;
 import org.xml.sax.Attributes;
 import org.xml.sax.SAXException;
 import org.xml.sax.helpers.DefaultHandler;
@@ -10,7 +12,6 @@ import javax.xml.parsers.SAXParserFactory;
 import java.io.IOException;
 import java.util.ArrayList;
 import java.util.List;
-import java.util.Objects;
 
 public class SaxParserMain {
     public static void main(String[] args) throws ParserConfigurationException, SAXException, IOException {
@@ -18,11 +19,17 @@ public class SaxParserMain {
         SAXParser saxParser = factory.newSAXParser();
 
         BaeldungHandler baeldungHandler = new BaeldungHandler();
-        saxParser.parse("xml\\src\\main\\resources\\sax\\baeldung.xml", baeldungHandler);
+        saxParser.parse("xml/src/main/resources/sax/baeldung.xml", baeldungHandler);
         System.out.println(baeldungHandler.getWebsite());
     }
 
+    @ToString
     public static class BaeldungHandler extends DefaultHandler {
+        private static final String ARTICLES = "articles";
+        private static final String ARTICLE = "article";
+        private static final String TITLE = "title";
+        private static final String CONTENT = "content";
+
         private Baeldung website;
         private String elementValue;
 
@@ -68,94 +75,16 @@ public class SaxParserMain {
         public Baeldung getWebsite() {
             return website;
         }
-
-        @Override
-        public String toString() {
-            return "BaeldungHandler{" +
-                    "website=" + website +
-                    ", elementValue='" + elementValue + '\'' +
-                    '}';
-        }
-
-        private static final String ARTICLES = "articles";
-        private static final String ARTICLE = "article";
-        private static final String TITLE = "title";
-        private static final String CONTENT = "content";
     }
 
-
+    @Data
     public static class Baeldung {
         private List<BaeldungArticle> articleList;
-
-        public List<BaeldungArticle> getArticleList() {
-            return articleList;
-        }
-
-        public void setArticleList(List<BaeldungArticle> articleList) {
-            this.articleList = articleList;
-        }
-
-        @Override
-        public boolean equals(Object o) {
-            if (this == o) return true;
-            if (o == null || getClass() != o.getClass()) return false;
-            Baeldung baeldung = (Baeldung) o;
-            return Objects.equals(articleList, baeldung.articleList);
-        }
-
-        @Override
-        public int hashCode() {
-            return Objects.hash(articleList);
-        }
-
-        @Override
-        public String toString() {
-            return "Baeldung{" +
-                    "articleList=" + articleList +
-                    '}';
-        }
     }
 
+    @Data
     public static class BaeldungArticle {
         private String title;
         private String content;
-
-        public String getTitle() {
-            return title;
-        }
-
-        public void setTitle(String title) {
-            this.title = title;
-        }
-
-        public String getContent() {
-            return content;
-        }
-
-        public void setContent(String content) {
-            this.content = content;
-        }
-
-        @Override
-        public boolean equals(Object o) {
-            if (this == o) return true;
-            if (o == null || getClass() != o.getClass()) return false;
-            BaeldungArticle that = (BaeldungArticle) o;
-            return Objects.equals(title, that.title) &&
-                    Objects.equals(content, that.content);
-        }
-
-        @Override
-        public int hashCode() {
-            return Objects.hash(title, content);
-        }
-
-        @Override
-        public String toString() {
-            return "BaeldungArticle{" +
-                    "title='" + title + '\'' +
-                    ", content='" + content + '\'' +
-                    '}';
-        }
     }
 }

From f82e007672f60178d1e3b4e5ba207536805f464b Mon Sep 17 00:00:00 2001
From: YassinHajaj <yassin.hajaj@hotmail.com>
Date: Fri, 6 Sep 2019 16:01:11 +0000
Subject: [PATCH 7/9] BAEL-3219

---
 xml/pom.xml                                   | 13 ++------
 .../java/com/baeldung/sax/SaxParserMain.java  | 32 ++++++++++++++++---
 xml/src/main/resources/sax/baeldung.xml       |  6 ++--
 .../baeldung/sax/SaxParserMainUnitTest.java   |  8 ++---
 xml/src/test/resources/sax/baeldung.xml       |  6 ++--
 5 files changed, 40 insertions(+), 25 deletions(-)

diff --git a/xml/pom.xml b/xml/pom.xml
index 3b8b88ba73..24969d1d6b 100644
--- a/xml/pom.xml
+++ b/xml/pom.xml
@@ -111,13 +111,6 @@
             <version>${commons-lang.version}</version>
         </dependency>
 
-        <dependency>
-            <groupId>org.projectlombok</groupId>
-            <artifactId>lombok</artifactId>
-            <version>1.18.8</version>
-            <scope>provided</scope>
-        </dependency>
-
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter</artifactId>
@@ -332,7 +325,7 @@
                         </configuration>
                     </plugin>
                     <plugin>
-                        <!-- NOTE: We don't need a groupId specification 
+                        <!-- NOTE: We don't need a groupId specification
                             because the group is org.apache.maven.plugins ...which is assumed by default. -->
                         <artifactId>maven-assembly-plugin</artifactId>
                         <configuration>
@@ -352,9 +345,9 @@
                         </configuration>
                         <executions>
                             <execution>
-                                <id>make-assembly</id><!-- this is used for 
+                                <id>make-assembly</id><!-- this is used for
                                     inheritance merges -->
-                                <phase>package</phase><!-- append to the 
+                                <phase>package</phase><!-- append to the
                                     packaging phase. -->
                                 <goals>
                                     <goal>attached</goal><!-- goals == mojos -->
diff --git a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
index 654d5162cd..5aba9409de 100644
--- a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
+++ b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
@@ -1,7 +1,5 @@
 package com.baeldung.sax;
 
-import lombok.Data;
-import lombok.ToString;
 import org.xml.sax.Attributes;
 import org.xml.sax.SAXException;
 import org.xml.sax.helpers.DefaultHandler;
@@ -23,7 +21,7 @@ public class SaxParserMain {
         System.out.println(baeldungHandler.getWebsite());
     }
 
-    @ToString
+    //@ToString
     public static class BaeldungHandler extends DefaultHandler {
         private static final String ARTICLES = "articles";
         private static final String ARTICLE = "article";
@@ -77,14 +75,38 @@ public class SaxParserMain {
         }
     }
 
-    @Data
+    //@Data
     public static class Baeldung {
         private List<BaeldungArticle> articleList;
+
+        public void setArticleList(List<BaeldungArticle> articleList) {
+            this.articleList = articleList;
+        }
+
+        public List<BaeldungArticle> getArticleList() {
+            return this.articleList;
+        }
     }
 
-    @Data
+    //@Data
     public static class BaeldungArticle {
         private String title;
         private String content;
+
+        public void setTitle(String title) {
+            this.title = title;
+        }
+
+        public String getTitle() {
+            return this.title;
+        }
+
+        public void setContent(String content) {
+            this.content = content;
+        }
+
+        public String getContent() {
+            return this.content;
+        }
     }
 }
diff --git a/xml/src/main/resources/sax/baeldung.xml b/xml/src/main/resources/sax/baeldung.xml
index 4b839f35c7..6736d5bdca 100644
--- a/xml/src/main/resources/sax/baeldung.xml
+++ b/xml/src/main/resources/sax/baeldung.xml
@@ -2,15 +2,15 @@
     <articles>
         <article>
             <title>Parsing an XML File Using SAX Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>SAX Parser's Lorem ipsum...</content>
         </article>
         <article>
             <title>Parsing an XML File Using DOM Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>DOM Parser's Lorem ipsum...</content>
         </article>
         <article>
             <title>Parsing an XML File Using StAX Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>StAX Parser's Lorem ipsum...</content>
         </article>
     </articles>
 </baeldung>
\ No newline at end of file
diff --git a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
index 8a8aa62dd2..333c5619c8 100644
--- a/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
+++ b/xml/src/test/java/com/baeldung/sax/SaxParserMainUnitTest.java
@@ -14,7 +14,7 @@ import static org.junit.Assert.*;
 public class SaxParserMainUnitTest {
 
     @Test
-    public void parse_baeldung_xml_document() throws IOException, SAXException, ParserConfigurationException {
+    public void givenAProperXMLFile_whenItIsParsed_ThenAnObjectContainsAllItsElements() throws IOException, SAXException, ParserConfigurationException {
         SAXParserFactory factory = SAXParserFactory.newInstance();
         SAXParser saxParser = factory.newSAXParser();
 
@@ -31,14 +31,14 @@ public class SaxParserMainUnitTest {
 
         SaxParserMain.BaeldungArticle articleOne = articles.get(0);
         assertEquals("Parsing an XML File Using SAX Parser", articleOne.getTitle());
-        assertEquals("Lorem ipsum...", articleOne.getContent());
+        assertEquals("SAX Parser's Lorem ipsum...", articleOne.getContent());
 
         SaxParserMain.BaeldungArticle articleTwo = articles.get(1);
         assertEquals("Parsing an XML File Using DOM Parser", articleTwo.getTitle());
-        assertEquals("Lorem ipsum...", articleTwo.getContent());
+        assertEquals("DOM Parser's Lorem ipsum...", articleTwo.getContent());
 
         SaxParserMain.BaeldungArticle articleThree = articles.get(2);
         assertEquals("Parsing an XML File Using StAX Parser", articleThree.getTitle());
-        assertEquals("Lorem ipsum...", articleThree.getContent());
+        assertEquals("StAX Parser's Lorem ipsum...", articleThree.getContent());
     }
 }
diff --git a/xml/src/test/resources/sax/baeldung.xml b/xml/src/test/resources/sax/baeldung.xml
index 4b839f35c7..6736d5bdca 100644
--- a/xml/src/test/resources/sax/baeldung.xml
+++ b/xml/src/test/resources/sax/baeldung.xml
@@ -2,15 +2,15 @@
     <articles>
         <article>
             <title>Parsing an XML File Using SAX Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>SAX Parser's Lorem ipsum...</content>
         </article>
         <article>
             <title>Parsing an XML File Using DOM Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>DOM Parser's Lorem ipsum...</content>
         </article>
         <article>
             <title>Parsing an XML File Using StAX Parser</title>
-            <content>Lorem ipsum...</content>
+            <content>StAX Parser's Lorem ipsum...</content>
         </article>
     </articles>
 </baeldung>
\ No newline at end of file

From d7107a5931a92570e7aa5be428e9e50ef0abf201 Mon Sep 17 00:00:00 2001
From: YassinHajaj <YassinHajaj@users.noreply.github.com>
Date: Sat, 28 Sep 2019 13:57:53 +0200
Subject: [PATCH 8/9] BAEL-3219

---
 xml/pom.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/xml/pom.xml b/xml/pom.xml
index 24969d1d6b..bbd607f9b5 100644
--- a/xml/pom.xml
+++ b/xml/pom.xml
@@ -325,7 +325,7 @@
                         </configuration>
                     </plugin>
                     <plugin>
-                        <!-- NOTE: We don't need a groupId specification
+                        <!-- NOTE: We don't need a groupId specification 
                             because the group is org.apache.maven.plugins ...which is assumed by default. -->
                         <artifactId>maven-assembly-plugin</artifactId>
                         <configuration>
@@ -345,9 +345,9 @@
                         </configuration>
                         <executions>
                             <execution>
-                                <id>make-assembly</id><!-- this is used for
+                                <id>make-assembly</id><!-- this is used for 
                                     inheritance merges -->
-                                <phase>package</phase><!-- append to the
+                                <phase>package</phase><!-- append to the 
                                     packaging phase. -->
                                 <goals>
                                     <goal>attached</goal><!-- goals == mojos -->

From 83535d53c8a94286ef1d023edc7d0936ddd32b64 Mon Sep 17 00:00:00 2001
From: "yassin.hajaj" <yassin.hajaj@hotmail.com>
Date: Sat, 28 Sep 2019 14:19:11 +0200
Subject: [PATCH 9/9] BAEL-3219

---
 xml/src/main/java/com/baeldung/sax/SaxParserMain.java | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
index 5aba9409de..4908c10386 100644
--- a/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
+++ b/xml/src/main/java/com/baeldung/sax/SaxParserMain.java
@@ -21,7 +21,6 @@ public class SaxParserMain {
         System.out.println(baeldungHandler.getWebsite());
     }
 
-    //@ToString
     public static class BaeldungHandler extends DefaultHandler {
         private static final String ARTICLES = "articles";
         private static final String ARTICLE = "article";
@@ -75,7 +74,6 @@ public class SaxParserMain {
         }
     }
 
-    //@Data
     public static class Baeldung {
         private List<BaeldungArticle> articleList;
 
@@ -88,7 +86,6 @@ public class SaxParserMain {
         }
     }
 
-    //@Data
     public static class BaeldungArticle {
         private String title;
         private String content;