#14454 closed (fixed)
Convert admin_widgets doctest to unit tests
| Reported by: | Preston Timmons | Owned by: | nobody |
|---|---|---|---|
| Component: | Testing framework | Version: | dev |
| Severity: | Keywords: | admin test | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | yes |
| Easy pickings: | no | UI/UX: | no |
Description
Attached is a patch that converts the admin_widgets doctests to unit tests. These changes were tested against trunk 14196 using the sqlite test settings.
Attachments (2)
Change History (7)
by , 14 years ago
| Attachment: | admin_widgets.diff added |
|---|
comment:1 by , 14 years ago
| Patch needs improvement: | set |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
by , 14 years ago
| Attachment: | admin_widgets_corrected.diff added |
|---|
comment:2 by , 14 years ago
Thanks, Alex. I updated the patch against 14217. One of the lines in the first patch had trailing whitespace which git didn't approve of. It should apply cleanly now.
comment:3 by , 14 years ago
| Resolution: | → fixed |
|---|---|
| Status: | new → closed |
comment:4 by , 14 years ago
Note:
See TracTickets
for help on using tickets.
The patch doesn't apply cleanly for me at HEAD. Looks like it hasn't been properly updated since unittest2 landed. If you could update it I'll look at merging it pronto.