test: remove underscore prefix from shared test helper functions

These functions are imported across modules, so they are part of the
test package's public API and shouldn't use the private convention.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: vdusek

tests/e2e/test_crawlee_crawlers/conftest.py

@@ -15,12 +15,12 @@
 _ACTOR_SOURCE_DIR = Path(__file__).parent / 'actor_source'
 
 
-def _read_actor_source(filename: str) -> str:
+def read_actor_source(filename: str) -> str:
     return (_ACTOR_SOURCE_DIR / filename).read_text()
 
 
-def _get_playwright_dockerfile() -> str:
-    return _read_actor_source('Dockerfile.playwright').replace(
+def get_playwright_dockerfile() -> str:
+    return read_actor_source('Dockerfile.playwright').replace(
         'PYTHON_VERSION_PLACEHOLDER',
         _PYTHON_VERSION,
     )
@@ -33,7 +33,7 @@ def _get_playwright_dockerfile() -> str:
 }
 
 
-async def _verify_crawler_results(
+async def verify_crawler_results(
     actor: ActorClientAsync,
     run_result: ActorRun,
     expected_crawler_type: str,

tests/e2e/test_crawlee_crawlers/test_adaptive_playwright_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _get_playwright_dockerfile, _read_actor_source, _verify_crawler_results
+from .conftest import get_playwright_dockerfile, read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,12 +12,12 @@ async def test_adaptive_playwright_crawler(make_actor: MakeActorFunction, run_ac
     actor = await make_actor(
         label='crawl-adaptive',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_adaptive_playwright_crawler.py'),
-            'Dockerfile': _get_playwright_dockerfile(),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_adaptive_playwright_crawler.py'),
+            'Dockerfile': get_playwright_dockerfile(),
         },
         additional_requirements=['crawlee[all]>=1.0.0,<2.0.0'],
         memory_mbytes=1024,
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'AdaptivePlaywrightCrawler')
+    await verify_crawler_results(actor, run_result, 'AdaptivePlaywrightCrawler')

tests/e2e/test_crawlee_crawlers/test_basic_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _read_actor_source, _verify_crawler_results
+from .conftest import read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,9 +12,9 @@ async def test_basic_crawler(make_actor: MakeActorFunction, run_actor: RunActorF
     actor = await make_actor(
         label='crawl-basic',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_basic_crawler.py'),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_basic_crawler.py'),
         },
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'BasicCrawler')
+    await verify_crawler_results(actor, run_result, 'BasicCrawler')

tests/e2e/test_crawlee_crawlers/test_beautifulsoup_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _read_actor_source, _verify_crawler_results
+from .conftest import read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,10 +12,10 @@ async def test_beautifulsoup_crawler(make_actor: MakeActorFunction, run_actor: R
     actor = await make_actor(
         label='crawl-bsoup',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_beautifulsoup_crawler.py'),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_beautifulsoup_crawler.py'),
         },
         additional_requirements=['crawlee[beautifulsoup]>=1.0.0,<2.0.0'],
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'BeautifulSoupCrawler')
+    await verify_crawler_results(actor, run_result, 'BeautifulSoupCrawler')

tests/e2e/test_crawlee_crawlers/test_http_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _read_actor_source, _verify_crawler_results
+from .conftest import read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,9 +12,9 @@ async def test_http_crawler(make_actor: MakeActorFunction, run_actor: RunActorFu
     actor = await make_actor(
         label='crawl-http',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_http_crawler.py'),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_http_crawler.py'),
         },
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'HttpCrawler')
+    await verify_crawler_results(actor, run_result, 'HttpCrawler')

tests/e2e/test_crawlee_crawlers/test_parsel_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _read_actor_source, _verify_crawler_results
+from .conftest import read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,9 +12,9 @@ async def test_parsel_crawler(make_actor: MakeActorFunction, run_actor: RunActor
     actor = await make_actor(
         label='crawl-parsel',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_parsel_crawler.py'),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_parsel_crawler.py'),
         },
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'ParselCrawler')
+    await verify_crawler_results(actor, run_result, 'ParselCrawler')

tests/e2e/test_crawlee_crawlers/test_playwright_crawler.py

@@ -2,7 +2,7 @@
 
 from typing import TYPE_CHECKING
 
-from .conftest import _get_playwright_dockerfile, _read_actor_source, _verify_crawler_results
+from .conftest import get_playwright_dockerfile, read_actor_source, verify_crawler_results
 
 if TYPE_CHECKING:
     from ..conftest import MakeActorFunction, RunActorFunction
@@ -12,12 +12,12 @@ async def test_playwright_crawler(make_actor: MakeActorFunction, run_actor: RunA
     actor = await make_actor(
         label='crawl-playwright',
         source_files={
-            'server.py': _read_actor_source('server.py'),
-            'src/main.py': _read_actor_source('main_playwright_crawler.py'),
-            'Dockerfile': _get_playwright_dockerfile(),
+            'server.py': read_actor_source('server.py'),
+            'src/main.py': read_actor_source('main_playwright_crawler.py'),
+            'Dockerfile': get_playwright_dockerfile(),
         },
         additional_requirements=['crawlee[playwright]>=1.0.0,<2.0.0'],
         memory_mbytes=1024,
     )
     run_result = await run_actor(actor)
-    await _verify_crawler_results(actor, run_result, 'PlaywrightCrawler')
+    await verify_crawler_results(actor, run_result, 'PlaywrightCrawler')