Opened 17 years ago

Closed 17 years ago

#6810 closed (fixed)

[newforms-admin] Add tests for admin views

Reported by: Robert Myers Owned by: Robert Myers
Component: Testing framework Version: newforms-admin
Severity: Keywords:
Cc: Triage Stage: Accepted
Has patch: no Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no
Pull Requests:How to create a pull request

Description

There are no tests for the newforms admin views, the test client should allow us to test login and various GET and POST actions.

Change History (5)

by Robert Myers, 17 years ago

Attachment: 6810.patch​ added

first attempt tests permissions for login/add/change/delete views

by Robert Myers, 17 years ago

Attachment: 6810-2.patch​ added

Cleaned up the test a little to improve readability

comment:1 by Brian Rosner, 17 years ago

Owner: changed from Robert Myers to Brian Rosner
Status: new → assigned

comment:2 by Brian Rosner, 17 years ago

Owner: changed from Brian Rosner to Robert Myers
Status: assigned → new
Triage Stage: Unreviewed → Accepted

Opps, didn't mean to actually accept the ticket for myself, but to Accept it ;) I will look over these tests in more depth later.

comment:3 by Brian Rosner, 17 years ago

Resolution: → fixed
Status: new → closed

(In [7589]) newforms-admin: Fixed #6810 -- Added tests for admin views. Thanks rmyers!

Note: See TracTickets for help on using tickets.
Back to Top