corso-python-prato / share-system-team2

Apache License 2.0
2 stars 9 forks source link

test_utils.py module renamed in tstutils.py #65

Closed iacopy closed 10 years ago

iacopy commented 10 years ago

Per evitare un conflitto "semantico" con i moduli che testano altri moduli, ho rinominato _testutils.py in tstutils.py (per un po' ho pensato che testasse un fantomatico modulo 'utils.py'). In questo modo si capisce che serve per i test ma si differenzia nettamente dai moduli tipo test_qualcosa.py. Sono ancora indeciso sulle varie alternative ma non sul fatto che _testutils.py sia da rinominare.

Alcune alternative (accanto ho scritto il perché ho preferito evitarle):

Riferimento: http://legacy.python.org/dev/peps/pep-0008/#package-and-module-names

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.44%) when pulling 1aad11f857134d8ee8e6357473ecba957f7c8c6e on iacopy:test-utils-renaming-refactoring into 3194b40b5d01509dba50dd9febc98e6168bbd647 on corso-python-prato:master.

coveralls commented 10 years ago

Coverage Status

Coverage increased (+0.32%) when pulling a57cd8586c09b22c4a8c03ee6f0facb8ec80407d on iacopy:test-utils-renaming-refactoring into ab3a311c9bd3e166b9352c7290198ae07703d2c4 on corso-python-prato:master.