Opened 4 years ago

Closed 4 years ago

Last modified 10 days ago

#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:
Cc: jml Branch:
Author:

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 4 years ago.
patch to make twisted/trial/test/test_tests.py comply to standard.

Download all attachments as: .zip

Change History (11)

comment:1 Changed 4 years ago by DefaultCC Plugin

  • Cc jml added

comment:2 Changed 4 years ago by weiteng

  • Owner set to weiteng

comment:3 Changed 4 years ago by weiteng

  • Owner weiteng deleted

comment:4 Changed 4 years ago by wirehead

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

comment:5 Changed 4 years 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 4 years ago by wirehead

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

comment:6 Changed 4 years 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 4 years ago by wsanchez

Very nice. +1.

comment:8 Changed 4 years ago by wsanchez

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

comment:9 Changed 4 years 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.

comment:10 Changed 10 days ago by hawkowl

  • Keywords review removed

[mass edit] Removing review from closed tickets.

Note: See TracTickets for help on using tickets.