chore: order dunder methods better

src/crawling/ids.py

@@ -81,9 +81,3 @@ def get_bestsellers_ids(self, scrolls: int = 1) -> list[Product]:
     def get_query_ids(self, query: str, scrolls: int = 1) -> list[Product]:
         """Fetch product IDs from a Amazon search query"""
         return self._get_ids(f"https://amazon.com/s?k={quote_plus(query)}", scrolls)
-
-    def __enter__(self) -> Self:
-        return self
-
-    def __exit__(self, *_: Any) -> None:
-        self.browser.quit()

src/crawling/sync_reviews.py

@@ -26,6 +26,12 @@ def __init__(self, fake_display: bool = True) -> None:
 
         self.browser = Firefox(options=opts)
 
+    def __enter__(self) -> AmazonScraper:
+        return self
+
+    def __exit__(self, *_: Any) -> None:
+        self.close()
+
     def get_bestselling(self) -> Generator[str, None, None]:
         """Fetch product IDs from Amazon's Bestsellers page"""
         self.browser.get("https://www.amazon.com/gp/bestsellers/")
@@ -109,9 +115,3 @@ def select_reviews(content: Any) -> Generator[Review, None, None]:
     def close(self) -> None:
         """Close the browser"""
         self.browser.quit()
-
-    def __enter__(self) -> AmazonScraper:
-        return self
-
-    def __exit__(self, *_: Any) -> None:
-        self.close()

src/crawling/threaded_reviews.py

@@ -41,6 +41,12 @@ def __init__(self, fake_display: bool = True) -> None:
 
         self.captcha_handler = self.handle_captcha
 
+    def __enter__(self) -> Self:
+        return self
+
+    def __exit__(self, *_: Any) -> None:
+        self.close()
+
     def handle_captcha(self, future: Future, browsers: list) -> None:
         """Default CAPTCHA handler"""
         try:
@@ -192,10 +198,3 @@ def close(self) -> None:
         with ThreadPoolExecutor() as executor:
             for browser in self.browsers:
                 executor.submit(browser.quit)
-
-    # these methods are practically useless at this point
-    def __enter__(self) -> Self:
-        return self
-
-    def __exit__(self, exc_type: Any, exc_val: Any, exc_tb: Any) -> None:
-        self.close()