Opened 5 years ago

Closed 5 years ago

#16198 closed Cleanup/optimization (fixed)

model_forms tests are still doctests.

Reported by: Gregor Müllegger Owned by: nobody
Component: Testing framework Version: 1.3
Severity: Normal Keywords: model_forms, doctests
Cc: Peter van Kampen Triage Stage: Ready for checkin
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

Refactor model_forms tests to be unittests instead of doctests.

Attachments (2)

ticket16198_model_forms_test_refactoring.1.diff (121.4 KB) - added by Gregor Müllegger 5 years ago.
Sorry previous patch version was just plain wrong.
ticket16198_model_forms_test_refactoring.2.diff (121.7 KB) - added by Gregor Müllegger 5 years ago.
Added Peter van Kampen to AUTHORS file, he did most of the work. Thank you!

Download all attachments as: .zip

Change History (6)

Changed 5 years ago by Gregor Müllegger

Sorry previous patch version was just plain wrong.

Changed 5 years ago by Gregor Müllegger

Added Peter van Kampen to AUTHORS file, he did most of the work. Thank you!

comment:1 Changed 5 years ago by Peter van Kampen

Cc: Peter van Kampen added
Needs documentation: unset
Needs tests: unset
Patch needs improvement: unset

comment:2 Changed 5 years ago by Aymeric Augustin

Triage Stage: UnreviewedAccepted

comment:3 Changed 5 years ago by melinath

Triage Stage: AcceptedReady for checkin

Applies cleanly & tests pass. Yay!

comment:4 Changed 5 years ago by Russell Keith-Magee

Resolution: fixed
Status: newclosed

In [16358]:

Fixed #16198 -- Ported the model_forms doctests. Our long national nightmare is over. Oh, and we have always been at war with doctests. Thanks to Peter van Kampen and Gregor Müllegger for the patch.

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