Ticket #6010 enhancement closed fixed

Opened 9 months ago

Last modified 9 months ago

Port twisted.test.test_defgen and twisted.internet.test.test_inlinecb to Python 3

Reported by: itamar Owned by: itamar
Priority: normal Milestone: Python 3.3 Minimal
Component: core Keywords:
Cc: Branch: branches/defgen-py3-6010
(diff, github, buildbot, log)
Author: itamarst Launchpad Bug:

Description (last modified by itamar) (diff)

twisted.test.test_defgen and twisted.internet.test.test_inlinecb test inlineCallbacks and deferredGenerator. It should be ported to Python 3.

Change History

1

Changed 9 months ago by itamar

  • description modified (diff)
  • summary changed from Port twisted.test.test_defgen to Python 3 to Port twisted.test.test_defgen and twisted.internet.test.test_inlinecb to Python 3

2

Changed 9 months ago by exarkun

How about not bothering with test_defgen or deferredGenerator? I just filed #6044.

3

Changed 9 months ago by itamar

There are a bunch of ported tests that rely on deferredGenerator, so probably best to port it (unless it turns out to be easier to fix those tests). In particular, I suspect it will be trivial to do.

4

Changed 9 months ago by itamar

  • owner changed from itamar to exarkun
  • keywords review added

 http://buildbot.twistedmatrix.com/boxes-supported?branch=/branches/defgen-py3-6010 started.

I renamed a hidden module, removed an exec (both unnecessary now that we use modern Pythons), and fixed a couple of syntax errors in exception catching; the tests are otherwise unmodified.

5

Changed 9 months ago by itamarst

  • branch set to branches/defgen-py3-6010
  • branch_author set to itamarst

(In [35945]) Branching to 'defgen-py3-6010'

6

Changed 9 months ago by exarkun

  • owner changed from exarkun to itamar
  • keywords review removed

Thanks.

The ticket reference at the top of test_inlinecb.py doesn't seem too useful anymore? At least, these tests are no longer skipped in any way. Actually, the tests in test_defgen.py aren't skipped anymore either and there is no generator_failure_tests.py file in twisted/test/ at all. Does that mean #4182 is resolved?

No review comments about the branch itself. Please merge!

7

Changed 9 months ago by itamarst

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

(In [35976]) Merge defgen-py3-6010.

Author: itamar Review: exarkun Fixes: #6010

Port deferredGenerator and inlineCallbacks tests to Python 3.

Note: See TracTickets for help on using tickets.