Edgewall Software
Modify

Opened 3 years ago

Closed 2 years ago

Last modified 2 years ago

#12564 closed enhancement (fixed)

Add a test for Request.__repr__

Reported by: Tim Graham <timograham@…> Owned by: Tim Graham <timograham@…>
Priority: normal Milestone: 1.3.2
Component: web frontend Version:
Severity: normal Keywords:
Cc: Branch:
Release Notes:

Added test coverage for Request.__repr__.

API Changes:

Description

Currently you can comment out some querystring lines in Request.__repr__ and the tests will still pass.

Attachments (1)

12564.diff (532 bytes ) - added by Tim Graham <timograham@…> 3 years ago.

Download all attachments as: .zip

Change History (6)

by Tim Graham <timograham@…>, 3 years ago

Attachment: 12564.diff added

comment:1 by Jun Omae, 3 years ago

Milestone: next-stable-1.0.x
Type: defectenhancement

Thanks. However we should add more test cases if we'd like to test it.

comment:2 by Ryan J Ollos, 3 years ago

Milestone: next-stable-1.0.xnext-stable-1.2.x

Moved ticket assigned to next-stable-1.0.x since maintenance of 1.0.x is coming to a close. Please move the ticket back if it's critical to fix on 1.0.x.

comment:3 by Ryan J Ollos, 3 years ago

Milestone: next-stable-1.2.x1.3.2
Owner: set to Ryan J Ollos
Release Notes: modified (diff)
Status: newassigned

comment:4 by Ryan J Ollos, 2 years ago

Resolution: fixed
Status: assignedclosed

Two test cases committed to trunk in r15592. I didn't see any other paths that needed coverage, but feel free to extend the test coverage.

comment:5 by Ryan J Ollos, 2 years ago

Owner: changed from Ryan J Ollos to Tim Graham <timograham@…>

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Tim Graham <timograham@…>.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


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