Ticket #5906 enhancement closed fixed

Opened 20 months ago

Last modified 20 months ago

port accumulateMethods and prefixedMethods to Python 3

Reported by: exarkun Owned by: exarkun
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/some-reflect3-5906
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

The unit tests for SynchronousTestCase depend on these two methods.

Change History

1

Changed 20 months ago by exarkun

  • branch set to branches/some-reflect3-5906
  • branch_author set to exarkun

(In [35364]) Branching to 'some-reflect3-5906'

2

Changed 20 months ago by exarkun

  • keywords review added
  • owner changed from exarkun to itamar

I filed #5909 for improving the docstrings here.  Build results

3

Changed 20 months ago by itamar

  • keywords review removed
  • owner changed from itamar to exarkun

Hooray for more test coverage.

  1. You should add a note to the test module referencing #5885, for easy grepping of things that need fixing.
  2. I named the equivalent util module _utilpy3; either naming scheme is fine, we should just try to be consistent, so either rename yours or if you prefer this scheme I can rename mine.
  3. The X Y Z naming scheme is a tiny bit confusing; maybe "Base" and "Sub" and "method"?
  4. Needs news file.

Fix the above, then feel free to merge.

4

Changed 20 months ago by exarkun

  • status changed from new to closed
  • resolution set to fixed

(In [35374]) Merge some-reflect3-5906

Author: exarkun Reviewer: itamar Fixes: #5906

Port a couple twisted.python.reflect methods to Python 3, in support of porting a minimal subset of trial. Also, add some unit tests for these previously untested methods.

Note: See TracTickets for help on using tickets.