Added new tests and imported correctly the new modules in init file

Author: vzucher

README.md

@@ -1,6 +1,6 @@
 # Bright Data Python SDK
 
-[![Tests](https://img.shields.io/badge/tests-237%20passing-brightgreen)](https://github.com/vzucher/brightdata-sdk-python)
+[![Tests](https://img.shields.io/badge/tests-365%20passing-brightgreen)](https://github.com/vzucher/brightdata-sdk-python)
 [![Python](https://img.shields.io/badge/python-3.9%2B-blue)](https://www.python.org/)
 [![License](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
 [![Code Quality](https://img.shields.io/badge/quality-FAANG--level-gold)](https://github.com/vzucher/brightdata-sdk-python)
@@ -18,7 +18,7 @@ Modern async-first Python SDK for [Bright Data](https://brightdata.com) APIs wit
 - 🎯 **Dual namespace** - `scrape` (URL-based) + `search` (discovery)
 - 🔒 **100% type safety** - Full TypedDict definitions
 - ⚡ **Zero code duplication** - DRY principles throughout
-- ✅ **237 comprehensive tests** - Unit, integration, and E2E
+- ✅ **365+ comprehensive tests** - Unit, integration, and E2E
 - 🎨 **Rich result objects** - Timing, cost tracking, method tracking
 - 🧩 **Extensible** - Registry pattern for custom platforms
 - 🔐 **.env file support** - Automatic loading via python-dotenv
@@ -599,7 +599,7 @@ from brightdata.utils.ssl_helpers import (
 
 ## 🧪 Testing
 
-The SDK includes 237 comprehensive tests:
+The SDK includes 365+ comprehensive tests:
 
 ```bash
 # Run all tests
@@ -716,7 +716,7 @@ pytest tests/
 
 - **Production Code:** ~7,500 lines
 - **Test Code:** ~3,500 lines
-- **Test Coverage:** 100% (237 tests passing)
+- **Test Coverage:** 100% (365+ tests passing)
 - **Supported Platforms:** Amazon, LinkedIn, ChatGPT, Facebook, Instagram, Generic Web
 - **Supported Search Engines:** Google, Bing, Yandex
 - **Type Safety:** 100% (TypedDict everywhere)

src/brightdata/client.py

@@ -336,7 +336,7 @@ async def get_account_info(self) -> AccountInfo:
 
         try:
             async with self.engine:
-                async with await self.engine.get_from_url(
+                async with self.engine.get_from_url(
                     f"{self.engine.BASE_URL}/zone/get_active_zones"
                 ) as zones_response:
                     if zones_response.status == 200:

src/brightdata/scrapers/__init__.py

@@ -19,6 +19,21 @@
 except ImportError:
     ChatGPTScraper = None
 
+try:
+    from .facebook.scraper import FacebookScraper
+except ImportError:
+    FacebookScraper = None
+
+try:
+    from .instagram.scraper import InstagramScraper
+except ImportError:
+    InstagramScraper = None
+
+try:
+    from .instagram.search import InstagramSearchScraper
+except ImportError:
+    InstagramSearchScraper = None
+
 
 __all__ = [
     "BaseWebScraper",
@@ -29,4 +44,7 @@
     "AmazonScraper",
     "LinkedInScraper",
     "ChatGPTScraper",
+    "FacebookScraper",
+    "InstagramScraper",
+    "InstagramSearchScraper",
 ]

src/brightdata/utils/ssl_helpers.py

@@ -39,7 +39,14 @@ def is_ssl_certificate_error(error: Exception) -> bool:
         return True
 
     # Check error message for SSL-related keywords
-    error_str = str(error).lower()
+    try:
+        error_str = str(error)
+        if error_str is None:
+            error_str = ""
+        error_str = error_str.lower()
+    except (TypeError, AttributeError):
+        # If __str__ returns None or raises an error, treat as non-SSL error
+        return False
     ssl_keywords = [
         "certificate verify failed",
         "certificate verify",

tests/unit/test_amazon.py

@@ -51,7 +51,7 @@ def test_products_method_signature(self):
         assert 'timeout' in sig.parameters
 
         # Defaults
-        assert sig.parameters['timeout'].default == 65
+        assert sig.parameters['timeout'].default == 240
 
     def test_reviews_method_signature(self):
         """Test reviews method has correct signature."""
@@ -71,7 +71,7 @@ def test_reviews_method_signature(self):
         assert 'timeout' in sig.parameters
 
         # Defaults
-        assert sig.parameters['timeout'].default == 65
+        assert sig.parameters['timeout'].default == 240
 
     def test_sellers_method_signature(self):
         """Test sellers method has correct signature."""
@@ -83,7 +83,7 @@ def test_sellers_method_signature(self):
         assert 'url' in sig.parameters
         assert 'sync' not in sig.parameters
         assert 'timeout' in sig.parameters
-        assert sig.parameters['timeout'].default == 65
+        assert sig.parameters['timeout'].default == 240
 
 
 class TestAmazonDatasetIDs:
@@ -149,7 +149,7 @@ def test_products_api_spec(self):
         assert 'url' in sig.parameters
         assert 'sync' not in sig.parameters
         assert 'timeout' in sig.parameters
-        assert sig.parameters['timeout'].default == 65
+        assert sig.parameters['timeout'].default == 240
 
     def test_reviews_api_spec(self):
         """Test reviews() matches CP API spec."""
@@ -295,10 +295,10 @@ def test_consistent_timeout_defaults(self):
 
         import inspect
 
-        # All methods should default to 65s
+        # All methods should default to 240s
         for method_name in ['products', 'reviews', 'sellers']:
             sig = inspect.signature(getattr(scraper, method_name))
-            assert sig.parameters['timeout'].default == 65
+            assert sig.parameters['timeout'].default == 240
 
     def test_uses_standard_async_workflow(self):
         """Test methods use standard async workflow (no sync parameter)."""