Opened 9 years ago
Closed 9 years ago
#5506 closed (fixed)
Add some tests for Generic views
| Reported by: | robh | Owned by: | Russell Keith-Magee |
|---|---|---|---|
| Component: | Generic views | Version: | master |
| Severity: | Keywords: | sprintsept14 | |
| Cc: | Triage Stage: | Ready for checkin | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | UI/UX: |
Description
This sprint has seen a few tests being written for generic views, where previously there were none.
I've worked on #391 and #3031 and after testing found that they are both "worksforme".
There's also #1282 and #5494, which changed actual code.
All of these have used the TestClient to test the views, which appears to be the most practical option.
Attachments (1)
Change History (5)
Changed 9 years ago by
comment:1 Changed 9 years ago by
| Needs documentation: | unset |
|---|---|
| Needs tests: | unset |
| Patch needs improvement: | unset |
| Summary: | [patch] Generic views have no testing → Add some tests for Generic views |
| Triage Stage: | Unreviewed → Ready for checkin |
comment:3 Changed 9 years ago by
| Owner: | changed from nobody to Russell Keith-Magee |
|---|
I added the start of a view-testing regression test in [6370]; this one is next on my list.
comment:4 Changed 9 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
Note: See
TracTickets for help on using
tickets.
Tests for #391 (object_detail can't find 'today') and #3031 (archive_month includes too much)