Simplify check_compliance.py a bit to make it easier to follow, and to
make it easier to add new tests:
Store the path printed for each test in a ComplianceTest.path_hint
class variable instead of passing it to ComplianceTest.prepare().
Print the "Running in ..." message in main()
instead of in prepare().
Remove ComplianceTest.prepare() and all the calls to it, as it's no
longer needed.
Store the commit range in a global COMMIT_RANGE variable, set in
main(), instead of passing it to ComplianceTest.prepare()
This simplifies ComplianceTest.__init() to take no args. Use
__init() to initialize ComplianceTest.case, instead of doing it in
ComplianceTest.prepare().
Remove some unused stuff too:
The class variables on the ComplianceTest class itself. Instead,
document what class variables subclasses need to set.
The empty ComplianceTest.run(). Document that subclasses need it
instead.
The unused ComplianceTest._title variable
The unused 'suite' argument to ComplianceTest.__init__(), which was
used to set an unused ComplianceTest.suite variable
Remove the _'s in front of class variables. No other things are
marked private, and they're used outside the class.
Simplify check_compliance.py a bit to make it easier to follow, and to make it easier to add new tests:
Store the path printed for each test in a ComplianceTest.path_hint class variable instead of passing it to ComplianceTest.prepare(). Print the "Running in ..." message in main()
instead of in prepare().
Remove ComplianceTest.prepare() and all the calls to it, as it's no longer needed.
Store the commit range in a global COMMIT_RANGE variable, set in main(), instead of passing it to ComplianceTest.prepare()
This simplifies ComplianceTest.__init() to take no args. Use __init() to initialize ComplianceTest.case, instead of doing it in ComplianceTest.prepare().
Remove some unused stuff too:
The class variables on the ComplianceTest class itself. Instead, document what class variables subclasses need to set.
The empty ComplianceTest.run(). Document that subclasses need it instead.
The unused ComplianceTest._title variable
The unused 'suite' argument to ComplianceTest.__init__(), which was used to set an unused ComplianceTest.suite variable
Remove the _'s in front of class variables. No other things are marked private, and they're used outside the class.