nrpe lacks tests
Bug #1374089 reported by
Robert C Jennings
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| nrpe (Juju Charms Collection) |
Fix Released
|
Undecided
|
Robert C Jennings | ||
Bug Description
Created amulet tests for the nrpe charm while developing the same for a trusty version of the charm.
Related branches
lp:~rcj/charms/precise/nrpe/trunk
- Marco Ceppi (community): Approve
- Adam Israel (community): Approve
- Robert C Jennings (community): Needs Resubmitting
- Review Queue (community): Needs Fixing (automated testing)
-
Diff: 427 lines (+203/-80)8 files modifiedREADME.md (+30/-1)
hooks/config-changed (+19/-14)
hooks/local-monitors-relation-changed (+7/-3)
hooks/recursive_dictionary.py (+7/-6)
tests/00-setup (+1/-1)
tests/10-monitors (+64/-0)
tests/20-local_monitors (+75/-0)
tests/99-autogen (+0/-55)
Revision history for this message
| Robert C Jennings (rcj) wrote | #1 |
| Changed in nrpe (Juju Charms Collection): | |
| status: | New → Incomplete |
| status: | Incomplete → New |
Revision history for this message
| Robert C Jennings (rcj) wrote | #2 |
| Changed in nrpe (Juju Charms Collection): | |
| status: | New → Fix Committed |
| status: | Fix Committed → In Progress |
| Changed in nrpe (Juju Charms Collection): | |
| status: | In Progress → Fix Committed |
| assignee: | nobody → Robert C Jennings (rcj) |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
The tests are passing and the code in the merge proposal is ready for review.