DAFT-4: Fix double-parsing HTML (TheJokersThief#4)

Author: TheJokersThief

daft_scraper/search/__init__.py

@@ -69,7 +69,6 @@ def _extract_json(self, page_html: BeautifulSoup):
     def _get_page_data(self, path, params):
         """Request a page and parse its JSON"""
         page = self.site.get(path, params=params)
-        page = BeautifulSoup(page)
         return self._extract_json(page)
 
     def _get_listings(self, listings: dict):

tests/test_search.py

@@ -2,14 +2,15 @@
 # -*- coding: utf-8 -*-
 import mock
 import unittest
+from bs4 import BeautifulSoup
 
 from daft_scraper.search import DaftSearch, SearchType
 from daft_scraper.search.options import PropertyType, PropertyTypesOption, Facility, FacilitiesOption, PriceOption, BedOption
 from daft_scraper.search.options_location import LocationsOption, Location
 
 
 with open('tests/fixtures/sample_page.html') as file:
-    SEARCH_PAGE_HTML = file.read()
+    SEARCH_PAGE_HTML = BeautifulSoup(file.read(), features="html.parser")
 
 
 class TestDaftScraper(unittest.TestCase):