unified prefix for unit test files
Issue #47
closed
"TestTestXYZ.py" is awful. Should we use a unified prefix like "test_" for test files? The underscore would clearly label them as not required to run the EnMAP-Box
Comments (5)
-
-
reporter
zweitens, da PEP8 https://www.python.org/dev/peps/pep-0008/#package-and-module-names. Um den code nicht aufzublähen würde ich bei kleinen unterkomplexen modulen die testklassen gleich im modul lassen.
-
- changed status to closed
Mir gefällt aktuell folgendes:
packages_all_lowercase\ModulesMayBeCamelCase.py with associated test module called TestModulesMayBeCamelCase.py
Vielleicht ändere ich nochmal meine Meinung dazu. Muss mich da noch mehr in gute Python Software Design Literatur einlesen.
-
reporter
ist okey für mich. "Test" prefix nur für "Tests", wenn ich was interaktives brauche mach ich eine "sandboxXYZ.py" auf
-
Ah ja, das ist auch gut. SandboxMyClass.py oder sandbox_my_module.py gefällt mir aich gut.
- Log in to comment
Ja, ich bin da auch noch nicht entschieden. Nehmen wir mal z.B. die FileSearcher Klasse. Aktuell liegt diese Klasse in FileSearcher.py und der UnitTest in TestFileSearcher.py.
Zwei Varianten machen für mich am meisten Sinn:
FileSearcher Klasse in FileSearcher.py und den UnitTest in TestFileSearcher.py
FileSearcher Klasse in filesearcher.py und den UnitTest in test_filesearcher.py ---> sieht etwas aus wie bei IDL, wo die Dateinamen alle LowerCase sein mussten :-)
Was ich nicht so schön fände wäre die Auflösung des Camel Cases nach file_searcher.py und den UnitTest in test_file_searcher.py.
Was denkst du? Ich denke (2.) wäre die pythonischste Variant.