fix: resolve test file paths in discover_tests_pytest to fix path com…

[KRRT7]

…parison

[codeflash-ai]

⚡️ Codeflash found optimizations for this PR

📄 138% (1.38x) speedup for add_runtime_comments in codeflash/languages/javascript/edit_tests.py

⏱️ Runtime : 11.3 milliseconds → 4.74 milliseconds (best of 250 runs)

A dependent PR with the suggested changes has been created. Please review:

• ⚡️ Speed up function add_runtime_comments by 138% in PR #1605 (fix-tracer-replay-discovery) #1607

If you approve, it will be merged into this PR (branch fix-tracer-replay-discovery).

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[claude]

PR Review Summary

Prek Checks

✅ All checks passed — no formatting or linting issues found.

Mypy

✅ No new type errors introduced by this PR. Pre-existing errors in unchanged code are unrelated.

Code Review

✅ No critical bugs, security vulnerabilities, or breaking API changes found.

Changes reviewed:

• discover_unit_tests.py: Path comparison fix — resolves both discover_only_these_tests and test["test_file"] paths before comparison. This correctly fixes the path mismatch issue.
• time_utils.py: format_runtime_comment extracted from JS-specific edit_tests.py into shared utility with configurable comment_prefix parameter (# default for Python, // for JS).
• edit_tests.py: Clean removal of duplicated function, now imports from time_utils.
• code_replacer.py: Removal of OptimFunctionCollector and OptimFunctionReplacer classes — confirmed no remaining references in codebase.
• cli.py: _handle_show_config now reads actual config file when available, falls back to auto-detection otherwise.
• test_code_replacement.py: Removed tests for deleted OptimFunctionCollector class.

Test Coverage

File	Main	PR	Change
codeflash/cli_cmds/cli.py	39%	43%	+4% ✅
codeflash/code_utils/time_utils.py	99%	95%	-4% ⚠️
codeflash/discovery/discover_unit_tests.py	74%	75%	+1% ✅
codeflash/languages/javascript/edit_tests.py	N/A	N/A	—
codeflash/languages/python/static_analysis/code_replacer.py	76%	84%	+8% ✅
tests/test_code_replacement.py	96%	95%	-1%
Overall	78%	79%	+1% ✅

Notes:

• time_utils.py coverage dropped from 99% → 95% because the new format_runtime_comment function (5 statements) is not exercised by any Python test. It is only called from edit_tests.py (JavaScript support), which has no test coverage. Consider adding a unit test for format_runtime_comment.
• code_replacer.py coverage improved (+8%) due to removal of 143 lines of dead code.
• edit_tests.py (JavaScript) has no test coverage on either branch.
• 8 test failures in test_tracer.py are pre-existing on main — not related to this PR.

---

Last updated: 2026-02-20