Edgewall Software
Modify ↓

Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#4429 closed defect (fixed)

Permission changes broke the tests

Reported by: Manuzhai <manuzhai@…> Owned by: Matthew Good
Priority: low Milestone:
Component: admin/console Version: devel
Severity: normal Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

The changes in r4463 broke the unit tests for trac-admin permission commands. This should be fixed (ideally, the tests should have been checked before committing…).

Attachments (1)

trac-admin-new-permissions-tests.diff​ (3.2 KB ) - added by Manuzhai <manuzhai@…> 17 years ago.
Update the permissions lists.

Download all attachments as: .zip

Change History (3)

by Manuzhai <manuzhai@…>, 17 years ago

Attachment: trac-admin-new-permissions-tests.diff​ added

Update the permissions lists.

comment:1 by Manuzhai <manuzhai@…>, 17 years ago

Resolution: → fixed
Status: new → closed

Looks like mgood fixed this in r4477 (although it seems he didn't notice this ticket?).

comment:2 by Matthew Good, 17 years ago

Oops, you're right I didn't see this ticket, I just noticed the test breakage later. Thanks for the patch anyway.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Matthew Good.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Matthew Good to the specified user.

Add Comment
E-mail address and name can be saved in the Preferences .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.