Ticket #854 (closed defect: fixed)
Tests for authorization_group
| Reported by: | dread | Owned by: | johnlawrenceaspden |
|---|---|---|---|
| Priority: | major | Milestone: | ckan-v1.4-sprint-6 |
| Component: | ckan | Keywords: | tests |
| Cc: | Repository: | ckan | |
| Theme: | none |
Description (last modified by johnlawrenceaspden) (diff)
The coverage tool (run by buildbot in the ckan build) reports that only 24% of lines of ckan.controllers.authorization_group are run in tests and 38% of ckan.forms.authorization_group. This suggests a need for more tests.
Change History
comment:2 Changed 3 years ago by rgrp
- Keywords tests added
- Priority changed from awaiting triage to major
- Milestone set to ckan-v1.4-sprint-4
comment:3 Changed 3 years ago by johnlawrenceaspden
- Owner changed from pudo to johnlawrenceaspden
- Repository set to ckan
- Theme set to none
- Status changed from new to assigned
comment:4 Changed 3 years ago by rgrp
- Milestone changed from ckan-v1.4-sprint-4 to ckan-v1.4-sprint-5
comment:5 Changed 3 years ago by thejimmyg
- Milestone changed from ckan-v1.4-sprint-5 to ckan-v1.4-sprint-6
comment:6 Changed 3 years ago by johnlawrenceaspden
- Status changed from assigned to closed
- Resolution set to fixed
- Description modified (diff)
Coverage now up to 84% and 81%. Remaining untested code is error conditions, which we decided weren't worth the effort of locking down.
fixed on feature-854-tests-for-authz-groups, now merged into default code.
Note: See
TracTickets for help on using
tickets.
