Closed Bug 875039 Opened 11 years ago Closed 9 years ago

Move zamboni's assert3xx into its own module and use it in webpay

Categories

(Marketplace Graveyard :: Code Quality, defect, P4)

Product:
Marketplace Graveyard
Component:
Code Quality
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED WONTFIX

People

(Reporter: wraithan, Unassigned)

Details

(Whiteboard: p=1) 

This will clean up a bunch of tests and test more thoroughly. Also we can stop saying 'we should port assert3xx to this codebase' in code reviews.
Assignee: wraithan → nobody
There is no mention of assertRedirects or assert3xx in webpay. In Django 1.7 you can just specify: fetch_redirect_response=False, so there's no need to worry about this.

https://docs.djangoproject.com/en/1.7/topics/testing/tools/#django.test.SimpleTestCase.assertRedirects
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.