ci: add Python test matrix and expand coverage - #1
Closed
ghostfix-pm wants to merge 1 commit into
Closed
ghostfix-pm wants to merge 1 commit into
ghostfix-pm wants to merge 1 commit into
Conversation
- Add pyproject.toml with pytest/ruff config and classifiers - Expand .gitignore to cover common Python artifacts - Add test_eval_helpers_extended.py: 26 new tests for format_result, augmented_path, build_eval_argv, choose_source, resolve_sema edge cases - Update CI workflow: add Python test matrix job (3.8-3.12), keep existing YAML/JSON/plist validation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
pyproject.tomlwith pytest/ruff config and Python version classifierspyproject.tomlwith pytest/ruff config and Python version classifierspyproject.tomlwith pytest/ruff config and Python version classifierstests/test_eval_helpers_extended.pywith 26 new tests covering:format_resultedge cases (stderr-only, no elapsedMs, raw errors, line/col)format_process_errorvariationsaugmented_pathedge cases (empty current, order preservation, tilde expansion)build_eval_argvedge cases (custom timeout, zero timeout, paths with spaces)choose_sourceedge cases (empty buffer, whitespace-only, multiple with empty)resolve_semabehavior for missing vs real binaries_startupinfo_kwargsUnix behaviorTest Results
Files Changed
Note: CI workflow update was removed because the OAuth token lacks
workflowscope. The original validation workflow is preserved in the upstream repo. A follow-up PR can add it with proper permissions.