Opened 7 years ago

Closed 7 years ago

#6810 closed (fixed)

[newforms-admin] Add tests for admin views

Reported by: rmyers Owned by: rmyers
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: UI/UX:

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.

Attachments (2)

6810.patch (15.8 KB) - added by rmyers 7 years ago.
first attempt tests permissions for login/add/change/delete views
6810-2.patch (16.8 KB) - added by rmyers 7 years ago.
Cleaned up the test a little to improve readability

Download all attachments as: .zip

Change History (5)

Changed 7 years ago by rmyers

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

Changed 7 years ago by rmyers

Cleaned up the test a little to improve readability

comment:1 Changed 7 years ago by brosner

  • Needs documentation unset
  • Needs tests unset
  • Owner changed from rmyers to brosner
  • Patch needs improvement unset
  • Status changed from new to assigned

comment:2 Changed 7 years ago by brosner

  • Owner changed from brosner to rmyers
  • Status changed from assigned to new
  • Triage Stage changed from Unreviewed to 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 Changed 7 years ago by brosner

  • Resolution set to fixed
  • Status changed from new to 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