#6051 defect closed fixed (fixed)

twisted.trial.test.test_tests doesn't follow the coding standard

Reported by: itamar Owned by: wsanchez
Priority: normal Milestone:
Component: trial Keywords: review
Cc: jml Branch:
Author: Launchpad Bug:

Description

twisted.trial.test.test_tests should follow the coding standard. All tests should have docstrings, their name should start with test_, etc..

Attachments (1)

ticket-6051-docs.patch (29.2 KB) - added by wirehead 17 months ago.
patch to make twisted/trial/test/test_tests.py comply to standard.

Download all attachments as: .zip

Change History (10)

comment:1 Changed 22 months ago by DefaultCC Plugin

  • Cc jml added

comment:2 Changed 17 months ago by weiteng

  • Owner set to weiteng

comment:3 Changed 17 months ago by weiteng

  • Owner weiteng deleted

comment:4 Changed 17 months ago by wirehead

  • Owner set to wirehead
  • Status changed from new to assigned

comment:5 Changed 17 months ago by wirehead

Present style dings:

  • Docstrings:
    • I am assuming that all those "See Module Docstrings" are not OK.
    • Some functions are missing docstrings altogether.
  • Naming
    • Some functions named testBlah instead of test_blah

Changed 17 months ago by wirehead

patch to make twisted/trial/test/test_tests.py comply to standard.

comment:6 Changed 17 months ago by wirehead

  • Keywords review added; easy removed
  • Owner wirehead deleted
  • Status changed from assigned to new

the ClassTest in GCMixin seems to be unnecessary and doesn't break anything. Am I right about this?

Should be nothing but formatting and comments otherwise..

comment:7 Changed 17 months ago by wsanchez

Very nice. +1.

comment:8 Changed 17 months ago by wsanchez

  • Owner set to wsanchez
  • Status changed from new to assigned

comment:9 Changed 17 months ago by wsanchez

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [37638]) Apply ticket-6051-docs.patch: Add docs to test_tests.

Author: wirehead
Reviewer: wsanchez
Fixes: #6051

Make twisted/trial/test/test_tests.py comply to standard.

Note: See TracTickets for help on using tickets.