documentation lacks a TDD example.

Bug #18885 reported by Robert Collins
6
Affects Status Importance Assigned to Milestone
Twisted
Fix Released
Unknown
twisted (Ubuntu)
Confirmed
Low
Unassigned

Bug Description

The documentation (specifically the finger-server evolution) gives a nice intro
to the code, but without demonstrating how to unit test the code being written.
Given twisteds event driven nature this makes it hard to learn to write good
unit-tested twisted code. It would really help to have a TDD version of the
finger-server evolution.

http://twistedmatrix.com/bugs/issue1148: http://twistedmatrix.com/bugs/issue1148

Revision history for this message
Matthias Klose (doko) wrote :

which twisted version?

Revision history for this message
Robert Collins (lifeless) wrote :

all of them ;0. There is now an upstream bug for this
..http://twistedmatrix.com/bugs/issue1148

Matthias Klose (doko)
Changed in twisted:
status: Unconfirmed → Confirmed
Matthias Klose (doko)
Changed in twisted:
assignee: doko → nobody
Changed in twisted:
status: Unknown → New
Revision history for this message
rusivi2 (rusivi2-deactivatedaccount) wrote :

Thank you for posting this bug.

Does this occur in Lucid?

Changed in twisted (Ubuntu):
status: Confirmed → Incomplete
Revision history for this message
Robert Collins (lifeless) wrote :

Thats very easy for you to check you know, and given the upstream bug watch you don't even need to look at the code!

Changed in twisted (Ubuntu):
status: Incomplete → Confirmed
Revision history for this message
rusivi2 (rusivi2-deactivatedaccount) wrote :

Robert Collins, thank you for responding so quickly and contributing to this bug. For me it is not that easy as I am learning about this. However, I will refrain from posting further unless I have something more constructive to add.

Revision history for this message
Duncan McGreggor (oubiwann) wrote :

I believe that this has been addressed with therve's addition to the last release:
  http://twistedmatrix.com/documents/current/core/howto/trial.html

description: updated
Changed in twisted:
importance: Unknown → High
Revision history for this message
Glyph Lefkowitz (glyph) wrote :

The "remote watch" thingy appears to be targeting the wrong ticket. It's pointed at #1148 but based on its description it should arguably be pointed at http://twistedmatrix.com/trac/ticket/2443. I can't figure out how to re-target it, but someone else should feel free to. Perhaps a Launchpad usability bug, too?

Changed in twisted:
importance: High → Unknown
Matthias Klose (doko)
Changed in twisted:
status: New → Unknown
Changed in twisted:
status: Unknown → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.