Ticket #3647 defect closed fixed

Opened 5 years ago

Last modified 4 years ago

There are many "StringIOWithoutClosing" classes in the test suite

Reported by: exarkun Owned by:
Priority: normal Milestone:
Component: core Keywords:
Cc: Branch: branches/remove-duplicate-stringios-3647
(diff, github, buildbot, log)
Author: exarkun Launchpad Bug:

Description

There shouldn't be this kind of rampant duplication of testing helpers.

Change History

1

Changed 5 years ago by exarkun

  • branch set to branches/remove-duplicate-stringios-3647
  • branch_author set to exarkun

(In [26200]) Branching to 'remove-duplicate-stringios-3647'

2

Changed 5 years ago by exarkun

  • keywords review added
  • owner exarkun deleted

 Build Results

Please review

3

Changed 5 years ago by therve

  • owner set to exarkun
  • keywords review removed

I just noticed some pyflakes:

twisted/words/test/test_irc.py:8: 'StringIO' imported but unused
twisted/words/test/test_toc.py:11: 'StringIO' imported but unused
twisted/test/test_postfix.py:9: 'protocols' imported but unused
twisted/test/test_postfix.py:10: 'internet' imported but unused
twisted/test/test_postfix.py:11: 'loopback' imported but unused
twisted/test/test_postfix.py:13: 'defer' imported but unused
twisted/test/test_postfix.py:13: 'protocol' imported but unused
twisted/test/test_finger.py:10: 'protocol' imported but unused
twisted/test/test_finger.py:10: 'reactor' imported but unused
twisted/test/test_protocols.py:9: 'StringIO' imported but unused

Once fixed, please merge. Thanks!

4

Changed 5 years ago by exarkun

(In [26219]) Remove unused imports

refs #3647

5

Changed 5 years ago by exarkun

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

(In [26220]) Merge remove-duplicate-stringios-3647

Author: exarkun Reviewer: therve Fixes: #3647

Replace numerous duplicate definitions of the StringIOWithoutClosing utility class with use of a single centralized definition. Also remove and replace other similar test helper classes in favor of use of the new centralized StringIOWithoutClosing.

6

Changed 4 years ago by exarkun

This was a duplicate of #3015.

7

Changed 3 years ago by <automation>

  • owner exarkun deleted
Note: See TracTickets for help on using tickets.