Opened 4 years ago

Closed 4 years ago

#5906 enhancement closed fixed (fixed)

port accumulateMethods and prefixedMethods to Python 3

Reported by: Jean-Paul Calderone Owned by: Jean-Paul Calderone
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/some-reflect3-5906
branch-diff, diff-cov, branch-cov, buildbot
Author: exarkun

Description

The unit tests for SynchronousTestCase depend on these two methods.

Change History (4)

comment:1 Changed 4 years ago by Jean-Paul Calderone

Author: exarkun
Branch: branches/some-reflect3-5906

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

comment:2 Changed 4 years ago by Jean-Paul Calderone

Keywords: review added
Owner: changed from Jean-Paul Calderone to Itamar Turner-Trauring

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

comment:3 Changed 4 years ago by Itamar Turner-Trauring

Keywords: review removed
Owner: changed from Itamar Turner-Trauring to Jean-Paul Calderone

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.

comment:4 Changed 4 years ago by Jean-Paul Calderone

Resolution: fixed
Status: newclosed

(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.