Review comments

Author: Pijukatel

src/crawlee/browsers/_playwright_browser_plugin.py

@@ -90,12 +90,11 @@ def __init__(
             # Chromium parameter 'channel' set to 'chrome' enables using installed Google Chrome.
             default_launch_browser_options['channel'] = 'chrome'
 
-            if explicit_browser_launch_options.get(
+            if executable_path := explicit_browser_launch_options.get(
                 'executable_path', default_launch_browser_options.get('executable_path')
             ):
                 logger.debug(
-                    f'Using browser executable from {default_launch_browser_options["executable_path"]},'
-                    f" which takes precedence over 'chrome' channel."
+                    f"Using browser executable from {executable_path}, which takes precedence over 'chrome' channel."
                 )
 
         self._browser_type: BrowserType = browser_type

src/crawlee/project_template/templates/main_playwright_chrome.py

@@ -10,5 +10,6 @@
     headless=True,
     max_requests_per_crawl=10,
     browser_type="chrome",
-    {{ self.http_client_instantiation() }})
+    {{ self.http_client_instantiation() }}
+)
 # % endblock

src/crawlee/project_template/templates/main_playwright_firefox.py

@@ -10,5 +10,6 @@
     headless=True,
     max_requests_per_crawl=10,
     browser_type="firefox",
-    {{ self.http_client_instantiation() }})
+    {{ self.http_client_instantiation() }}
+)
 # % endblock

src/crawlee/project_template/templates/main_playwright_webkit.py

@@ -10,5 +10,6 @@
     headless=True,
     max_requests_per_crawl=10,
     browser_type="webkit",
-    {{ self.http_client_instantiation() }})
+    {{ self.http_client_instantiation() }}
+)
 # % endblock

src/crawlee/project_template/templates/routes_camoufox.py

@@ -1,4 +1,4 @@
-from crawlee.crawlers import PlaywrightCrawlingContext
+rom crawlee.crawlers import PlaywrightCrawlingContext
 from crawlee.router import Router
 
 router = Router[PlaywrightCrawlingContext]()

tests/e2e/conftest.py

@@ -15,6 +15,9 @@ def pytest_configure(config: Config) -> None:
         'impit',
         'playwright',
         'playwright_camoufox',
+        'playwright_chrome',
+        'playwright_firefox',
+        'playwright_webkit',
         'parsel',
         'beautifulsoup',
         'uv',