Ticket #854 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

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:1 Changed 3 years ago by dread

  • Description modified (diff)

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.