From 556aaf487230a217055ea21f6edb8a912f0e10db Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Federico=20Dom=C3=ADnguez=20Molina?= <fd.molina@outlook.com>
Date: Tue, 16 Jul 2024 16:51:39 -0500
Subject: [PATCH 01/17] Editing dockerfile to ensure npm is installed correctly

---
 Dockerfile | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Dockerfile b/Dockerfile
index 4c464fdf..06d207ea 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -8,7 +8,11 @@ FROM python:3.8
 
 # Add the NodeSource PPA
 # (see: https://github.com/nodesource/distributions/blob/master/README.md)
-RUN curl -sL https://deb.nodesource.com/setup_12.x | bash -
+RUN curl -sL https://deb.nodesource.com/setup_12.x | bash - && \
+    apt-get update && \
+    apt-get install -y --no-install-recommends postgresql-client nodejs && \
+    apt-get install -y npm && \
+    node -v && npm -v
 
 # Install any additional OS-level packages you need via apt-get. RUN statements
 # add additional layers to your image, increasing its final size. Keep your

From 1c8b01706e6a58556f72196eceb0d9f98d19ca4f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Federico=20Dom=C3=ADnguez=20Molina?= <fd.molina@outlook.com>
Date: Tue, 16 Jul 2024 17:10:26 -0500
Subject: [PATCH 02/17] Editing dockerfile to ensure npm is installed correctly

---
 Dockerfile | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index 06d207ea..d250560a 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -8,11 +8,7 @@ FROM python:3.8
 
 # Add the NodeSource PPA
 # (see: https://github.com/nodesource/distributions/blob/master/README.md)
-RUN curl -sL https://deb.nodesource.com/setup_12.x | bash - && \
-    apt-get update && \
-    apt-get install -y --no-install-recommends postgresql-client nodejs && \
-    apt-get install -y npm && \
-    node -v && npm -v
+RUN curl -sL https://deb.nodesource.com/setup_12.x | bash -
 
 # Install any additional OS-level packages you need via apt-get. RUN statements
 # add additional layers to your image, increasing its final size. Keep your
@@ -23,7 +19,7 @@ RUN curl -sL https://deb.nodesource.com/setup_12.x | bash - && \
 #
 # Read more on Dockerfile best practices at the source:
 # https://docs.docker.com/develop/develop-images/dockerfile_best-practices
-RUN apt-get update && apt-get install -y --no-install-recommends postgresql-client nodejs
+RUN apt-get update && apt-get install -y --no-install-recommends postgresql-client nodejs npm
 
 # Inside the container, create an app directory and switch into it
 RUN mkdir /app

From c7af93f9e06a6f5b38d1b16a1e876db186948319 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 19:41:38 +0000
Subject: [PATCH 03/17] Completing parse function

---
 parserator_web/views.py | 35 ++++++++++++++++++++++++++++++++++-
 1 file changed, 34 insertions(+), 1 deletion(-)

diff --git a/parserator_web/views.py b/parserator_web/views.py
index 0be3f4a9..0e43dc21 100644
--- a/parserator_web/views.py
+++ b/parserator_web/views.py
@@ -6,6 +6,7 @@
 from rest_framework.exceptions import ParseError
 
 
+
 class Home(TemplateView):
     template_name = 'parserator_web/index.html'
 
@@ -14,11 +15,43 @@ class AddressParse(APIView):
     renderer_classes = [JSONRenderer]
 
     def get(self, request):
+        """
+        Extracts the address from the get request
+        """
         # TODO: Flesh out this method to parse an address string using the
         # parse() method and return the parsed components to the frontend.
-        return Response({})
+        
+        address = request.GET("address", "")
+        parsed_address, address_type = self.parse(address)
+        
+        return Response({"input_string": address,
+                         "address_components": parsed_address,
+                         "address_type": address_type })
 
     def parse(self, address):
+        """
+        Uses the usaddress package to parse the address
+        """
         # TODO: Implement this method to return the parsed components of a
         # given address using usaddress: https://github.com/datamade/usaddress
+
+        if not address:
+            raise ParseError("Address can not be empty")
+        
+        try:
+            parsed_address = usaddress.parse(address)
+            address_type = usaddress.tag(address)
+
+        except Exception as error:
+            print(f"Error parsing address: {error}")
+            return {}, "ERROR"
+
+        address_components = {}
+        
+        for element in parsed_address:
+            address_components[element[1]] = element[0]
+
+        address_type = address_type[-1]
+        
+
         return address_components, address_type

From d88e72e26974a944fffe2d8c952ff102db7c0ae2 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 21:56:38 +0000
Subject: [PATCH 04/17] Adding javascript

---
 parserator_web/static/js/index.js | 60 +++++++++++++++++++++++++++++++
 1 file changed, 60 insertions(+)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 492674cc..06f22e66 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -1,2 +1,62 @@
 /* TODO: Flesh this out to connect the form to the API and render results
    in the #address-results div. */
+
+   document.addEventListener("DOMContentLoaded", function () {
+    const form = document.getElementById("address-form");
+  
+    form.addEventListener("submit", async function (e) {
+      e.preventDefault();
+  
+      const address = document.getElementById("address").value;
+      const url = "/api/parse/";
+      const queryParams = new URLSearchParams({ address: address });
+  
+      try {
+        const response = await fetch(`${url}?${queryParams}`, {
+          method: 'GET'
+        });
+  
+        const respData = await response.json();
+  
+        if (!response.ok) {
+          throw new Error(respData.detail || 'An error occurred');
+        }
+  
+        displayResults(respData);
+      } catch (error) {
+        displayError(error.message);
+      }
+    });
+  
+    function displayResults(data) {
+      document.getElementById("error-results").style.display = "none";
+      const resultsDiv = document.getElementById("address-results");
+      resultsDiv.style.display = "block";
+  
+      document.getElementById("parse-type").textContent = data.address_type;
+  
+      const tableBody = document.querySelector("#address-results-table tbody");
+      tableBody.innerHTML = '';
+  
+      for (const [key, value] of Object.entries(data.address_components)) {
+        const row = document.createElement("tr");
+        const partCell = document.createElement("td");
+        const tagCell = document.createElement("td");
+  
+        partCell.textContent = key;
+        tagCell.textContent = value;
+  
+        row.appendChild(partCell);
+        row.appendChild(tagCell);
+        tableBody.appendChild(row);
+      }
+    }
+  
+    function displayError(errorMessage) {
+      document.getElementById("address-results").style.display = "none";
+      const errorDiv = document.getElementById("error-results");
+      errorDiv.style.display = "block";
+      document.getElementById("parse-error").textContent = errorMessage;
+    }
+  });
+  
\ No newline at end of file

From fb167097f4211e5fa8b415152801c166f46adaab Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:05:28 +0000
Subject: [PATCH 05/17] Debugging code

---
 parserator_web/static/js/index.js | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 06f22e66..589a0280 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -1,12 +1,14 @@
 /* TODO: Flesh this out to connect the form to the API and render results
    in the #address-results div. */
 
+// Adding event listeners
    document.addEventListener("DOMContentLoaded", function () {
     const form = document.getElementById("address-form");
   
     form.addEventListener("submit", async function (e) {
       e.preventDefault();
-  
+    
+    //   Declaring variables
       const address = document.getElementById("address").value;
       const url = "/api/parse/";
       const queryParams = new URLSearchParams({ address: address });
@@ -16,10 +18,10 @@
           method: 'GET'
         });
   
-        const respData = await response.json();
+        const content = await response.json();
   
         if (!response.ok) {
-          throw new Error(respData.detail || 'An error occurred');
+          throw new Error(content.detail || 'An error occured during execution');
         }
   
         displayResults(respData);

From b4d1957ab5cc5017da2d380e35935cf14a1aa427 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:20:00 +0000
Subject: [PATCH 06/17] Added necessary ids

---
 parserator_web/templates/parserator_web/index.html | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/parserator_web/templates/parserator_web/index.html b/parserator_web/templates/parserator_web/index.html
index a72d9c80..43bf31ad 100644
--- a/parserator_web/templates/parserator_web/index.html
+++ b/parserator_web/templates/parserator_web/index.html
@@ -11,7 +11,7 @@ <h3 id="usaddress-parser"><i class="fa fa-fw fa-map-marker-alt"></i> U.S. addres
       <p>Dealing with some messy or unstructured addresses? We can parse them for you.</p>
       <div class="card card-body bg-light">
         <p><strong>Try it out!</strong> Parse an address in the United States into fields like <code>AddressNumber</code>, <code>StreetName</code> and <code>ZipCode</code>.</p>
-        <form class="form" role="form">
+        <form id="address-form" class="form" role="form">
           {% csrf_token %}
           <input name="address" type="text" class="form-control" id="address" placeholder="123 Main St. Suite 100 Chicago, IL">
           <button id="submit" type="submit" class="btn btn-success mt-3">Parse!</button>
@@ -21,7 +21,7 @@ <h3 id="usaddress-parser"><i class="fa fa-fw fa-map-marker-alt"></i> U.S. addres
       <div id="address-results" style="display:none">
         <h4>Parsing results</h4>
         <p>Address type: <strong><span id="parse-type"></span></strong></p>
-        <table class="table table-bordered">
+        <table id="address-results-table" class="table table-bordered">
           <thead>
             <tr>
               <th>Address part</th>
@@ -36,7 +36,6 @@ <h4>Parsing results</h4>
   </div>
 </div>
 {% endblock %}
-
 {% block extra_js %}
   <script src="{% static 'js/index.js' %}"></script>
 {% endblock %}

From b530bbb379e9608228e0f7900baabe8a0dac1a8c Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:22:15 +0000
Subject: [PATCH 07/17] Debugging GET.get

---
 parserator_web/views.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/parserator_web/views.py b/parserator_web/views.py
index 0e43dc21..f619ef43 100644
--- a/parserator_web/views.py
+++ b/parserator_web/views.py
@@ -21,7 +21,7 @@ def get(self, request):
         # TODO: Flesh out this method to parse an address string using the
         # parse() method and return the parsed components to the frontend.
         
-        address = request.GET("address", "")
+        address = request.GET.get("address")
         parsed_address, address_type = self.parse(address)
         
         return Response({"input_string": address,

From 63d1dfefc4dd87a095cac4e343177f3960309450 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:28:05 +0000
Subject: [PATCH 08/17] minor changes

---
 parserator_web/static/js/index.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 589a0280..330fa540 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -24,7 +24,7 @@
           throw new Error(content.detail || 'An error occured during execution');
         }
   
-        displayResults(respData);
+        displayResults(content);
       } catch (error) {
         displayError(error.message);
       }

From 09b9afa87b2744b71dc447d713e384482eeec2e1 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:34:45 +0000
Subject: [PATCH 09/17] Debug: added missing id

---
 parserator_web/static/js/index.js                  | 14 ++++++++++----
 parserator_web/templates/parserator_web/index.html |  4 ++++
 2 files changed, 14 insertions(+), 4 deletions(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 330fa540..3216316e 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -18,19 +18,21 @@
           method: 'GET'
         });
   
-        const content = await response.json();
+        const contentData = await response.json();
   
         if (!response.ok) {
-          throw new Error(content.detail || 'An error occured during execution');
+          throw new Error(contentData.detail || 'An error occured during execution');
         }
   
-        displayResults(content);
+        displayResults(contentData);
       } catch (error) {
+        console.error('Error:', error.message); // Log error for debugging
         displayError(error.message);
       }
     });
   
     function displayResults(data) {
+      console.log('Displaying results'); // Debug log
       document.getElementById("error-results").style.display = "none";
       const resultsDiv = document.getElementById("address-results");
       resultsDiv.style.display = "block";
@@ -39,7 +41,8 @@
   
       const tableBody = document.querySelector("#address-results-table tbody");
       tableBody.innerHTML = '';
-  
+    
+    // Creating table
       for (const [key, value] of Object.entries(data.address_components)) {
         const row = document.createElement("tr");
         const partCell = document.createElement("td");
@@ -50,14 +53,17 @@
   
         row.appendChild(partCell);
         row.appendChild(tagCell);
+
         tableBody.appendChild(row);
       }
     }
   
     function displayError(errorMessage) {
       document.getElementById("address-results").style.display = "none";
+
       const errorDiv = document.getElementById("error-results");
       errorDiv.style.display = "block";
+      
       document.getElementById("parse-error").textContent = errorMessage;
     }
   });
diff --git a/parserator_web/templates/parserator_web/index.html b/parserator_web/templates/parserator_web/index.html
index 43bf31ad..f6e28a7d 100644
--- a/parserator_web/templates/parserator_web/index.html
+++ b/parserator_web/templates/parserator_web/index.html
@@ -18,6 +18,10 @@ <h3 id="usaddress-parser"><i class="fa fa-fw fa-map-marker-alt"></i> U.S. addres
         </form>
       </div>
       <!-- TODO: Display parsed address components here. -->
+      
+    <div id="error-results", style="display:none;">
+        <p id="parse-error"></p>
+     </div>
       <div id="address-results" style="display:none">
         <h4>Parsing results</h4>
         <p>Address type: <strong><span id="parse-type"></span></strong></p>

From 63b383067514021a5fa940e6c5c550b9541ce4d7 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:45:21 +0000
Subject: [PATCH 10/17] Catching and raising exception when parsing is
 unsuccessful

---
 parserator_web/views.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/parserator_web/views.py b/parserator_web/views.py
index f619ef43..143c813a 100644
--- a/parserator_web/views.py
+++ b/parserator_web/views.py
@@ -38,13 +38,14 @@ def parse(self, address):
         if not address:
             raise ParseError("Address can not be empty")
         
+        # Catch error and raise it as ParseError
         try:
             parsed_address = usaddress.parse(address)
             address_type = usaddress.tag(address)
 
         except Exception as error:
-            print(f"Error parsing address: {error}")
-            return {}, "ERROR"
+            raise ParseError(error)
+
 
         address_components = {}
         

From 863497376adb2667acbfb6ab2f05e2bdf800c4f1 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 22:57:26 +0000
Subject: [PATCH 11/17] Adding 1st test

---
 tests/test_views.py | 29 +++++++++++++++++++++++++++++
 1 file changed, 29 insertions(+)

diff --git a/tests/test_views.py b/tests/test_views.py
index bfd5d0b7..15442c5a 100644
--- a/tests/test_views.py
+++ b/tests/test_views.py
@@ -1,10 +1,39 @@
 import pytest
+from http import HTTPStatus
 
+PARSE_API = "/api/parse/"
 
 def test_api_parse_succeeds(client):
+    """
+    Tests the API by sending an address
+    """
     # TODO: Finish this test. Send a request to the API and confirm that the
     # data comes back in the appropriate format.
     address_string = '123 main st chicago il'
+    correct_answer = {
+        "AddressNumber": "123",
+        "StreetName": "main",
+        "StreetNamePostType": "st",
+        "PlaceName": "chicago",
+        "StateName": "il"
+
+    }
+
+    correct_address_type = "Street Address"
+
+    payload = {
+        "address": address_string
+    }
+
+    response = client.get(PARSE_API, payload)
+    address_type = response.json()["address_type"]
+    components = response.json()["address_components"]
+
+    assert response.status == HTTPStatus.OK
+    assert address_type == correct_address_type
+    assert components == correct_answer
+
+
     pytest.fail()
 
 

From 6eecae925b57e73a1f153c0cf26992b1a3f53c8d Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 23:05:55 +0000
Subject: [PATCH 12/17] Restructuring code

---
 parserator_web/views.py | 68 ++++++++++++++++++++++++++++++-----------
 tests/test_views.py     |  9 +++---
 2 files changed, 55 insertions(+), 22 deletions(-)

diff --git a/parserator_web/views.py b/parserator_web/views.py
index 143c813a..022d3f02 100644
--- a/parserator_web/views.py
+++ b/parserator_web/views.py
@@ -1,14 +1,14 @@
 import usaddress
 from django.views.generic import TemplateView
+from http import HTTPStatus
 from rest_framework.views import APIView
 from rest_framework.response import Response
 from rest_framework.renderers import JSONRenderer
 from rest_framework.exceptions import ParseError
 
 
-
 class Home(TemplateView):
-    template_name = 'parserator_web/index.html'
+    template_name = "parserator_web/index.html"
 
 
 class AddressParse(APIView):
@@ -20,13 +20,51 @@ def get(self, request):
         """
         # TODO: Flesh out this method to parse an address string using the
         # parse() method and return the parsed components to the frontend.
-        
+
         address = request.GET.get("address")
-        parsed_address, address_type = self.parse(address)
-        
-        return Response({"input_string": address,
-                         "address_components": parsed_address,
-                         "address_type": address_type })
+        try:
+            parsed_address, address_type = self.parse(address)
+            response = Response(
+                {
+                    "input_string": address,
+                    "address_components": parsed_address,
+                    "address_type": address_type,
+                },
+                status=HTTPStatus.OK,
+            )
+
+        except usaddress.RepeatedLabelError:
+            response = Response(
+                {
+                    "input_string": address,
+                    "address_components": {},
+                    "address_type": "",
+                    "error": "Remove repeated labels before parsing",
+                },
+                status=HTTPStatus.BAD_REQUEST,
+            )
+
+        except ParseError as error:
+            response = Response(
+                {
+                    "input_string": address,
+                    "address_components": {},
+                    "address_type": "",
+                    "error": str(error),
+                },
+                status=HTTPStatus.BAD_REQUEST,
+            )
+
+        except Exception as error:
+            response = Response(
+                {
+                    "input_string": address,
+                    "address_components": {},
+                    "address_type": "",
+                    "error": f"Unknown error: {error}",
+                },
+                status=HTTPStatus.INTERNAL_SERVER_ERROR,
+            )
 
     def parse(self, address):
         """
@@ -37,22 +75,16 @@ def parse(self, address):
 
         if not address:
             raise ParseError("Address can not be empty")
-        
-        # Catch error and raise it as ParseError
-        try:
-            parsed_address = usaddress.parse(address)
-            address_type = usaddress.tag(address)
-
-        except Exception as error:
-            raise ParseError(error)
 
+        # Catch error and raise it as ParseError
+        parsed_address = usaddress.parse(address)
+        address_type = usaddress.tag(address)
 
         address_components = {}
-        
+
         for element in parsed_address:
             address_components[element[1]] = element[0]
 
         address_type = address_type[-1]
-        
 
         return address_components, address_type
diff --git a/tests/test_views.py b/tests/test_views.py
index 15442c5a..b311a1bf 100644
--- a/tests/test_views.py
+++ b/tests/test_views.py
@@ -7,8 +7,6 @@ def test_api_parse_succeeds(client):
     """
     Tests the API by sending an address
     """
-    # TODO: Finish this test. Send a request to the API and confirm that the
-    # data comes back in the appropriate format.
     address_string = '123 main st chicago il'
     correct_answer = {
         "AddressNumber": "123",
@@ -25,6 +23,7 @@ def test_api_parse_succeeds(client):
         "address": address_string
     }
 
+    # Invoking API and getting response contents
     response = client.get(PARSE_API, payload)
     address_type = response.json()["address_type"]
     components = response.json()["address_components"]
@@ -38,7 +37,9 @@ def test_api_parse_succeeds(client):
 
 
 def test_api_parse_raises_error(client):
-    # TODO: Finish this test. The address_string below will raise a
-    # RepeatedLabelError, so ParseAddress.parse() will not be able to parse it.
+    """
+    Verify API raises an error
+    """
+
     address_string = '123 main st chicago il 123 main st'
     pytest.fail()

From c67bcf573b6f6fea646afe7f71a94bf84ac3ff8f Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 23:11:47 +0000
Subject: [PATCH 13/17] Return properly, ensure error is displayed

---
 parserator_web/static/js/index.js | 2 +-
 parserator_web/views.py           | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 3216316e..de8ca9a0 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -21,7 +21,7 @@
         const contentData = await response.json();
   
         if (!response.ok) {
-          throw new Error(contentData.detail || 'An error occured during execution');
+          throw new Error(contentData.detail || contentData.error);
         }
   
         displayResults(contentData);
diff --git a/parserator_web/views.py b/parserator_web/views.py
index 022d3f02..9ebf3022 100644
--- a/parserator_web/views.py
+++ b/parserator_web/views.py
@@ -30,7 +30,6 @@ def get(self, request):
                     "address_components": parsed_address,
                     "address_type": address_type,
                 },
-                status=HTTPStatus.OK,
             )
 
         except usaddress.RepeatedLabelError:
@@ -66,6 +65,8 @@ def get(self, request):
                 status=HTTPStatus.INTERNAL_SERVER_ERROR,
             )
 
+        return response
+
     def parse(self, address):
         """
         Uses the usaddress package to parse the address

From b3a3204f951914b24d845db011c9300985e4d145 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 23:13:27 +0000
Subject: [PATCH 14/17] Adding second test

---
 tests/test_views.py | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/tests/test_views.py b/tests/test_views.py
index b311a1bf..90e47e8d 100644
--- a/tests/test_views.py
+++ b/tests/test_views.py
@@ -42,4 +42,20 @@ def test_api_parse_raises_error(client):
     """
 
     address_string = '123 main st chicago il 123 main st'
+
+    correct_address_type = "Street Address"
+
+    payload = {
+        "address": address_string
+    }
+
+    # Invoking API and getting response contents
+    response = client.get(PARSE_API, payload)
+    address_type = response.json()["address_type"]
+    error = response.json()["error"]
+
+    assert response.status == HTTPStatus.BAD_REQUEST
+    assert address_type is None
+    assert error is not None
+
     pytest.fail()

From 82714ce0f59952390029cd2796b43a7df013daea Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Wed, 17 Jul 2024 23:18:12 +0000
Subject: [PATCH 15/17] Tests work

---
 tests/test_views.py | 13 +++----------
 1 file changed, 3 insertions(+), 10 deletions(-)

diff --git a/tests/test_views.py b/tests/test_views.py
index 90e47e8d..cb2368dc 100644
--- a/tests/test_views.py
+++ b/tests/test_views.py
@@ -28,14 +28,11 @@ def test_api_parse_succeeds(client):
     address_type = response.json()["address_type"]
     components = response.json()["address_components"]
 
-    assert response.status == HTTPStatus.OK
+    assert response.status_code == HTTPStatus.OK
     assert address_type == correct_address_type
     assert components == correct_answer
 
 
-    pytest.fail()
-
-
 def test_api_parse_raises_error(client):
     """
     Verify API raises an error
@@ -43,8 +40,6 @@ def test_api_parse_raises_error(client):
 
     address_string = '123 main st chicago il 123 main st'
 
-    correct_address_type = "Street Address"
-
     payload = {
         "address": address_string
     }
@@ -54,8 +49,6 @@ def test_api_parse_raises_error(client):
     address_type = response.json()["address_type"]
     error = response.json()["error"]
 
-    assert response.status == HTTPStatus.BAD_REQUEST
-    assert address_type is None
+    assert response.status_code == HTTPStatus.BAD_REQUEST
+    assert address_type == ''
     assert error is not None
-
-    pytest.fail()

From ef5e343df5bc581f2e79be29d44b92edc7bc6290 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Thu, 18 Jul 2024 00:38:15 +0000
Subject: [PATCH 16/17] Linting and formatting

---
 parserator_web/static/js/index.js | 26 +++++++++++++-------------
 tests/test_views.py               | 21 +++++++++------------
 2 files changed, 22 insertions(+), 25 deletions(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index de8ca9a0..205ae7b8 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -3,22 +3,22 @@
 
 // Adding event listeners
    document.addEventListener("DOMContentLoaded", function () {
-    const form = document.getElementById("address-form");
+    var form = document.getElementById("address-form");
   
     form.addEventListener("submit", async function (e) {
       e.preventDefault();
     
     //   Declaring variables
-      const address = document.getElementById("address").value;
-      const url = "/api/parse/";
-      const queryParams = new URLSearchParams({ address: address });
+      var address = document.getElementById("address").value;
+      var url = "/api/parse/";
+      var queryParams = new URLSearchParams({ address: address });
   
       try {
-        const response = await fetch(`${url}?${queryParams}`, {
+        var response = await fetch(`${url}?${queryParams}`, {
           method: 'GET'
         });
   
-        const contentData = await response.json();
+        var contentData = await response.json();
   
         if (!response.ok) {
           throw new Error(contentData.detail || contentData.error);
@@ -34,19 +34,19 @@
     function displayResults(data) {
       console.log('Displaying results'); // Debug log
       document.getElementById("error-results").style.display = "none";
-      const resultsDiv = document.getElementById("address-results");
+      var resultsDiv = document.getElementById("address-results");
       resultsDiv.style.display = "block";
   
       document.getElementById("parse-type").textContent = data.address_type;
   
-      const tableBody = document.querySelector("#address-results-table tbody");
+      var tableBody = document.querySelector("#address-results-table tbody");
       tableBody.innerHTML = '';
     
     // Creating table
-      for (const [key, value] of Object.entries(data.address_components)) {
-        const row = document.createElement("tr");
-        const partCell = document.createElement("td");
-        const tagCell = document.createElement("td");
+      for (var [key, value] of Object.entries(data.address_components)) {
+        var row = document.createElement("tr");
+        var partCell = document.createElement("td");
+        var tagCell = document.createElement("td");
   
         partCell.textContent = key;
         tagCell.textContent = value;
@@ -61,7 +61,7 @@
     function displayError(errorMessage) {
       document.getElementById("address-results").style.display = "none";
 
-      const errorDiv = document.getElementById("error-results");
+      var errorDiv = document.getElementById("error-results");
       errorDiv.style.display = "block";
       
       document.getElementById("parse-error").textContent = errorMessage;
diff --git a/tests/test_views.py b/tests/test_views.py
index cb2368dc..152e1bcc 100644
--- a/tests/test_views.py
+++ b/tests/test_views.py
@@ -1,27 +1,26 @@
-import pytest
+""""""
+
 from http import HTTPStatus
 
 PARSE_API = "/api/parse/"
 
+
 def test_api_parse_succeeds(client):
     """
     Tests the API by sending an address
     """
-    address_string = '123 main st chicago il'
+    address_string = "123 main st chicago il"
     correct_answer = {
         "AddressNumber": "123",
         "StreetName": "main",
         "StreetNamePostType": "st",
         "PlaceName": "chicago",
-        "StateName": "il"
-
+        "StateName": "il",
     }
 
     correct_address_type = "Street Address"
 
-    payload = {
-        "address": address_string
-    }
+    payload = {"address": address_string}
 
     # Invoking API and getting response contents
     response = client.get(PARSE_API, payload)
@@ -38,11 +37,9 @@ def test_api_parse_raises_error(client):
     Verify API raises an error
     """
 
-    address_string = '123 main st chicago il 123 main st'
+    address_string = "123 main st chicago il 123 main st"
 
-    payload = {
-        "address": address_string
-    }
+    payload = {"address": address_string}
 
     # Invoking API and getting response contents
     response = client.get(PARSE_API, payload)
@@ -50,5 +47,5 @@ def test_api_parse_raises_error(client):
     error = response.json()["error"]
 
     assert response.status_code == HTTPStatus.BAD_REQUEST
-    assert address_type == ''
+    assert address_type == ""
     assert error is not None

From 27344d3078bbc49f0e6a385a01763fcb6bab2da9 Mon Sep 17 00:00:00 2001
From: FedericoDM <fd.molina@outlook.com>
Date: Thu, 18 Jul 2024 00:39:37 +0000
Subject: [PATCH 17/17] Remove prints

---
 parserator_web/static/js/index.js | 2 --
 1 file changed, 2 deletions(-)

diff --git a/parserator_web/static/js/index.js b/parserator_web/static/js/index.js
index 205ae7b8..d54c6322 100644
--- a/parserator_web/static/js/index.js
+++ b/parserator_web/static/js/index.js
@@ -26,13 +26,11 @@
   
         displayResults(contentData);
       } catch (error) {
-        console.error('Error:', error.message); // Log error for debugging
         displayError(error.message);
       }
     });
   
     function displayResults(data) {
-      console.log('Displaying results'); // Debug log
       document.getElementById("error-results").style.display = "none";
       var resultsDiv = document.getElementById("address-results");
       resultsDiv.style.display = "block";