fix: address linting issues and add temp output to gitignore

- Fix line too long (123 > 120) in test_cli.py
- Run ruff format on test files
- Add tests/smoke/.temp-output/ to .gitignore

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

Author: mldangelo

.gitignore

@@ -42,6 +42,7 @@ htmlcov/
 .tox/
 .mypy_cache/
 .ruff_cache/
+tests/smoke/.temp-output/
 
 # Distribution
 dist/

tests/smoke/test_smoke.py

@@ -150,9 +150,7 @@ class TestEvalCommand:
     def test_basic_eval(self):
         """Test basic eval with echo provider."""
         config_path = CONFIGS_DIR / "basic.yaml"
-        stdout, stderr, exit_code = run_promptfoo(
-            ["eval", "-c", str(config_path), "--no-cache"]
-        )
+        stdout, stderr, exit_code = run_promptfoo(["eval", "-c", str(config_path), "--no-cache"])
 
         assert exit_code == 0, f"Eval failed:\nSTDOUT: {stdout}\nSTDERR: {stderr}"
         # Should show evaluation results
@@ -267,9 +265,7 @@ def test_verbose_flag(self):
         """Test --verbose flag."""
         config_path = CONFIGS_DIR / "basic.yaml"
 
-        stdout, stderr, exit_code = run_promptfoo(
-            ["eval", "-c", str(config_path), "--verbose", "--no-cache"]
-        )
+        stdout, stderr, exit_code = run_promptfoo(["eval", "-c", str(config_path), "--verbose", "--no-cache"])
 
         assert exit_code == 0
         # Verbose mode should produce output
@@ -283,9 +279,7 @@ def test_success_exit_code(self):
         """Test exit code 0 when all assertions pass."""
         config_path = CONFIGS_DIR / "basic.yaml"
 
-        stdout, stderr, exit_code = run_promptfoo(
-            ["eval", "-c", str(config_path), "--no-cache"]
-        )
+        stdout, stderr, exit_code = run_promptfoo(["eval", "-c", str(config_path), "--no-cache"])
 
         assert exit_code == 0
 
@@ -365,9 +359,7 @@ def test_contains_assertion(self):
         """Test contains assertion."""
         config_path = CONFIGS_DIR / "basic.yaml"
 
-        stdout, stderr, exit_code = run_promptfoo(
-            ["eval", "-c", str(config_path), "--no-cache"]
-        )
+        stdout, stderr, exit_code = run_promptfoo(["eval", "-c", str(config_path), "--no-cache"])
 
         assert exit_code == 0
         # All assertions should pass
@@ -377,9 +369,7 @@ def test_multiple_assertions(self):
         """Test multiple assertions in single test."""
         config_path = CONFIGS_DIR / "assertions.yaml"
 
-        stdout, stderr, exit_code = run_promptfoo(
-            ["eval", "-c", str(config_path), "--no-cache"]
-        )
+        stdout, stderr, exit_code = run_promptfoo(["eval", "-c", str(config_path), "--no-cache"])
 
         assert exit_code == 0
 

tests/test_cli.py

@@ -268,7 +268,8 @@ def test_find_external_promptfoo_prevents_recursion_windows(self, monkeypatch: p
         real_promptfoo = "C:\\npm\\prefix\\promptfoo.cmd"
 
         monkeypatch.setattr(sys, "argv", [wrapper_path])
-        monkeypatch.setenv("PATH", "C:\\Users\\user\\AppData\\Local\\Programs\\Python\\Python312\\Scripts;C:\\npm\\prefix")
+        test_path = "C:\\Users\\user\\AppData\\Local\\Programs\\Python\\Python312\\Scripts;C:\\npm\\prefix"
+        monkeypatch.setenv("PATH", test_path)
 
         def mock_which(cmd: str, path: Optional[str] = None) -> Optional[str]:
             if cmd != "promptfoo":
@@ -471,9 +472,7 @@ def test_main_exits_when_neither_external_nor_npx_available(
         node_path = "C:\\Program Files\\nodejs\\node.exe" if sys.platform == "win32" else "/usr/bin/node"
 
         monkeypatch.setattr(sys, "argv", ["promptfoo", "eval"])
-        monkeypatch.setattr("shutil.which", lambda cmd, path=None: {
-            "node": node_path
-        }.get(cmd))
+        monkeypatch.setattr("shutil.which", lambda cmd, path=None: {"node": node_path}.get(cmd))
 
         with pytest.raises(SystemExit) as exc_info:
             main()