update test and revert

Author: xzrderek

eval_protocol/cli_commands/upload.py

@@ -186,30 +186,14 @@ def pytest_collection_modifyitems(self, items):
     ]
 
     try:
-        # Run pytest collection with output captured so we can surface errors in CI
+        # Suppress pytest output
         import io
         import contextlib
 
-        stdout = io.StringIO()
-        stderr = io.StringIO()
-        with contextlib.redirect_stdout(stdout), contextlib.redirect_stderr(stderr):
-            exit_code = pytest.main(args, plugins=[plugin])
-
-        if exit_code != 0:
-            # In CI this helps diagnose why discovery returned an empty list
-            sys.stderr.write(f"[ep upload] pytest collection failed with exit code {exit_code} for root {abs_root}\n")
-            out = stdout.getvalue()
-            err = stderr.getvalue()
-            if out:
-                sys.stderr.write("[ep upload] pytest collection stdout:\n")
-                sys.stderr.write(out + "\n")
-            if err:
-                sys.stderr.write("[ep upload] pytest collection stderr:\n")
-                sys.stderr.write(err + "\n")
-            return []
-    except Exception as e:
-        # Log the exception instead of silently swallowing it
-        sys.stderr.write(f"[ep upload] pytest collection raised an exception for root {abs_root}: {e!r}\n")
+        with contextlib.redirect_stdout(io.StringIO()), contextlib.redirect_stderr(io.StringIO()):
+            pytest.main(args, plugins=[plugin])
+    except Exception:
+        # If pytest collection fails, fall back to empty list
         return []
 
     # Process collected items

tests/test_ep_upload_e2e.py

@@ -410,7 +410,7 @@ async def test_quickstart_eval(row: EvaluationRow) -> EvaluationRow:
 
     test_project_dir, test_file_path = create_test_project_with_evaluation_test(
         test_content,
-        "quickstart.py",  # Non test_* filename
+        "ep_upload_non_test_prefixed_eval.py",  # Non test_* filename
     )
 
     original_cwd = os.getcwd()
@@ -423,7 +423,8 @@ async def test_quickstart_eval(row: EvaluationRow) -> EvaluationRow:
 
         assert len(discovered_tests) == 1
         assert "test_quickstart_eval" in discovered_tests[0].qualname
-        assert "quickstart.py" in discovered_tests[0].file_path
+        # Verify we discovered a non-test-prefixed file (our unique filename)
+        assert "ep_upload_non_test_prefixed_eval.py" in discovered_tests[0].file_path
 
     finally:
         os.chdir(original_cwd)