Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#5447 defect closed fixed (fixed)

twisted.test was removed from /documents/ but links still point at twisted.test.proto_helpers.html

Reported by: Julian Berman Owned by: Thijs Triemstra
Priority: normal Milestone:
Component: core Keywords: documentation
Cc: oubiwann, Thijs Triemstra, jesstess Branch: branches/test-apidocs-5447
branch-diff, diff-cov, branch-cov, buildbot
Author: thijs


Change History (11)

comment:1 Changed 5 years ago by DefaultCC Plugin

Cc: oubiwann added

comment:2 Changed 5 years ago by Thijs Triemstra

Cc: Thijs Triemstra added
Component: websitecore

There's also a typo in the TypeError used in the calculus/ trial howto example.

raise TypeError("Coudln't coerce arguments to integers: %s" % args)

comment:3 Changed 5 years ago by Thijs Triemstra

Owner: set to Thijs Triemstra
Status: newassigned

I suggest getting rid of those API links because the tests aren't included in the API documentation.

comment:4 Changed 5 years ago by Thijs Triemstra

Author: thijs
Branch: branches/test-apidocs-5447

(In [33735]) Branching to 'test-apidocs-5447'

comment:5 Changed 5 years ago by Thijs Triemstra

(In [33736]) Fix links and typo in example of trial howto, refs #5447

comment:6 Changed 5 years ago by Thijs Triemstra

Keywords: review added
Owner: Thijs Triemstra deleted
Status: assignednew

comment:7 Changed 5 years ago by jesstess

Owner: set to jesstess

comment:8 Changed 5 years ago by jesstess

Cc: jesstess added
Keywords: review removed
Owner: changed from jesstess to Thijs Triemstra

Looks great, please merge!

comment:9 Changed 5 years ago by Thijs Triemstra

Resolution: fixed
Status: newclosed

Fixed in r33776.

comment:10 Changed 5 years ago by ralphm

That should be r33766.

I also want to note that this module is excluded because pydoctor has Twisted-specific exclusions which happen to include all packages named 'test'. These can be found in pydoctor.twistedmodel.TwistedSystem as pointed to by our documentation build configuration.

Maybe we should augment/move that to the Twisted repository so that it does not exclude twisted.test.proto_helpers.

comment:11 Changed 5 years ago by Jean-Paul Calderone

We should move all of the helper code from proto_helpers out of twisted.test (note "move" is a complicated verb, it involves making sure the APIs make sense and writing documentation and unit tests).

Note: See TracTickets for help on using tickets.