Consolidate paths management
Bug #1449459 reported by
Free Ekanayaka
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Landscape Charm |
Fix Released
|
Medium
|
Free Ekanayaka | ||
Bug Description
We have a number of places where we need to inject paths in order to be able to unit-test them, it would be good consolidate that by passing around an object that knows about those paths.
Related branches
lp:~free.ekanayaka/landscape-charm/consolidate-paths
- Chris Glass (community): Approve
- 🤖 Landscape Builder: Approve (test results)
- Alberto Donato (community): Approve
-
Diff: 617 lines (+180/-97)9 files modifiedhooks/lib/callbacks/filesystem.py (+11/-16)
hooks/lib/callbacks/tests/test_filesystem.py (+9/-10)
hooks/lib/paths.py (+65/-0)
hooks/lib/relations/haproxy.py (+7/-6)
hooks/lib/relations/tests/test_haproxy.py (+13/-14)
hooks/lib/services.py (+12/-19)
hooks/lib/tests/rootdir.py (+18/-15)
hooks/lib/tests/test_paths.py (+31/-0)
hooks/lib/tests/test_services.py (+14/-17)
| tags: | removed: kanban |
| Changed in landscape-charm: | |
| status: | In Progress → Fix Committed |
| Changed in landscape-charm: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
