Skip to content

Improve unit test coverage for juju.model.Model.deploy #1162

@james-garner-canonical

Description

@james-garner-canonical

Description

deploy does not appear to be well covered by unit tests, if at all. There are a bunch of tricky arguments, like storage constraints, that could perhaps be checked in unit tests, though the appropriate level to mock or fake at is not immediately clear to me.

See: #1105 (comment)

It would be nice to run coverage and get an idea of what is and isn't covered by unit tests. We could add a tox environment for it.

Urgency

Casually wishlisting

Code I'd Like to Run

...

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/testsUpdates the tests or test infrastructurehint/3.xgoing on main branchkind/wishlistrequested featurepriority/lowlow priority

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions