Unit tests for acl controls
Bug #1618966 reported by
Nicholas Skaggs
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical Juju |
Fix Released
|
High
|
Horacio Durán |
Bug Description
In response to bug 1613688 and other bugs like it, it would be useful to have complete coverage for all acl actions within the juju unit tests. ACL's are very important being a security feature, and will see more and more use. We need to ensure we've tested all possible scenarios and I believe this will be most efficiently done inside a unit test.
tags: | added: eda |
tags: | added: unit-tests |
Changed in juju: | |
status: | Incomplete → Triaged |
importance: | Undecided → High |
milestone: | none → 2.0-beta18 |
assignee: | nobody → Horacio Durán (hduran-8) |
Changed in juju: | |
milestone: | 2.0-beta18 → 2.0-beta19 |
Changed in juju: | |
milestone: | 2.0-beta19 → 2.0-rc1 |
Changed in juju: | |
milestone: | 2.0-rc1 → 2.0-rc2 |
Changed in juju: | |
status: | Triaged → In Progress |
Changed in juju: | |
milestone: | 2.0-rc2 → none |
Changed in juju: | |
milestone: | none → 2.0.0 |
Changed in juju: | |
milestone: | 2.0-rc3 → 2.0.0 |
Changed in juju: | |
milestone: | 2.0.0 → 2.0.1 |
Changed in juju: | |
milestone: | 2.0.1 → none |
Changed in juju: | |
status: | In Progress → Fix Committed |
milestone: | none → 2.1-rc1 |
Changed in juju: | |
milestone: | 2.1-rc1 → 2.1-beta3 |
Changed in juju: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
@Nicholas,
Here is current unit test coverage for ACL on master: com/juju/ juju/api/ modelmanager: 80.0% of statements com/juju/ juju/apiserver/ modelmanager: 78.6% of statements com/juju/ juju/cmd/ juju/model: 67.1% of statements
* github.
* github.
* github.
Are you asking for 100% coverage?
If not, please be more specific what unit tests you'd like to see :D