WatchControllerStatusChanges needs unit tests

Bug #1543408 reported by Cheryl Jennings
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Expired
Low
Unassigned

Bug Description

The fix for bug #1541473 introduced the WatchControllerStatusChanges watcher, but did not include unit tests. These tests should be added.

Changed in juju-core:
status: New → Triaged
importance: Undecided → Medium
assignee: nobody → Cheryl Jennings (cherylj)
milestone: none → 2.0-beta2
Revision history for this message
Cheryl Jennings (cherylj) wrote :

The new code in peergrouper also needs tests. As part of that change, I should look at separating the tests into a peergrouper_tests package.

Curtis Hovey (sinzui)
Changed in juju-core:
milestone: 2.0-beta2 → 2.0-beta3
Changed in juju-core:
assignee: Cheryl Jennings (cherylj) → nobody
milestone: 2.0-beta3 → none
Curtis Hovey (sinzui)
tags: added: tech-debt
affects: juju-core → juju
Revision history for this message
Anastasia (anastasia-macmood) wrote :

As this is a tech-debt item, I am lowering its Importance.

Changed in juju:
importance: Medium → Low
Revision history for this message
Canonical Juju QA Bot (juju-qa-bot) wrote :

This bug has not been updated in 5 years, so we're marking it Expired. If you believe this is incorrect, please update the status.

Changed in juju:
status: Triaged → Expired
tags: added: expirebugs-bot
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.