aglinxinyuan opened a new issue, #8034: URL: https://github.com/apache/texera/issues/8034
### Task Summary `PveManager` sits at 198 of 228 fully-covered lines. Two areas carry the gap, and both are error handling that nothing currently reaches: - **`resolveSystemPackages` has three give-up arms** — the throwaway venv cannot be created, the requirements install fails, or `pip freeze` exits non-zero — and none of the three early returns is executed by any test. The method also sanitises `pip freeze` output before that output becomes the `--constraint` file every user install is pinned against, and that sanitising step is unexercised. - **`deletePackages`' failure arm** — the branch taken when `pip uninstall` exits non-zero — is likewise never reached. Worth noting for whoever picks this up: `PveResourceSpec` already owns this class's tests (15 `"PveManager" should` blocks), so this is an extension of that file rather than a new spec. ### Task Type - [x] Testing / QA ### Was this issue authored using generative AI tooling? Generated-by: Claude Code (Opus 5) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
