Fix conftest for e2e

tests/e2e/conftest.py

@@ -1,3 +1,4 @@
+import logging
 import os
 import uuid
 from datetime import (
@@ -21,6 +22,23 @@
 # around names and values of the metadata.
 
 
+@fixture(scope="session", autouse=True)
+def cleanup_logging_handlers():
+    """Remove all logging handlers after each test session, to avoid
+    messy errors from the `logging` module.
+
+    The errors happen because `Run` installs `Run.close` as an `atexit` handler.
+    The method logs some messages. The output is captured by pytest, which closes
+    its logging handler early, causing "ValueError: I/O operation on closed file."
+    """
+
+    try:
+        yield
+    finally:
+        logger = logging.getLogger("neptune")
+        logger.handlers.clear()
+
+
 @fixture(scope="module")
 def project(request):
     # Assume the project name and API token are set in the environment using the standard
@@ -114,10 +132,10 @@ def run(project, run_init_kwargs):
 @fixture(scope="module")
 def sync_run(project, run_init_kwargs):
     """Blocking run for logging data"""
-    return SyncRun(**run_init_kwargs)
+    return SyncRun(project=run_init_kwargs["project"], run_id=run_init_kwargs["run_id"], resume=True)
 
 
 @fixture
-def ro_run(run, project):
+def ro_run(project, run, run_init_kwargs):
     """ReadOnlyRun pointing to the same run as the neptune_scale.Run"""
-    return ReadOnlyRun(read_only_project=project, custom_id=run._run_id)
+    return ReadOnlyRun(read_only_project=project, custom_id=run_init_kwargs["run_id"])