Visitar URL original
Test coverage · Issue #7 · python/pymanager · GitHub
Skip to content

Test coverage #7

Description

@zooba

Test coverage is somewhat spotty right now. Plenty of the core pieces are covered, along with actual bugs that were discovered by usage, but there are plenty of obvious gaps in our current verification.

Any contribution to increase coverage of the pytest tests is welcome.

Note that these tests do not have a usable py command, and so need to be structured to not rely on that - plenty of mocks, monkey-patching and directly importing commands is intended!

(For tests that can only be run after the full app is compiled/installed, create a new issue describing the scenario and why you think it wouldn't be valid as a unit test.)

Activity

  1. added a commit that references this issue on Aug 1, 2026
  2. breadonb commented on Aug 1, 2026

    @breadonb
    Contributor

    I added unit tests for config_bool, _expand_vars, config_append, config_dict_merge, config_split, and config_split_append. These tests increased coverage for config.py from 17% to 30%. I deliberately limited this PR to the dependency-free helpers, with resolve_config and merge_config as natural follow-up work. PR: (#392)

  3. added a commit that references this issue on Aug 18, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions