Details
-
Bug
-
Status: Open
-
Major
-
Resolution: Unresolved
-
None
-
None
-
None
Description
Now that kmarton cleaned up all the junk our tests were creating as part of OOZIE-3145 we should extend our pre-commit check to make sure no tmp files will be around after the tests are run.
We should extend our pre-commit checks with this logic. It should be something like:
- make sure git s returns nothing (I think this is already part of our pre-commit)
- apply the patch
- make a git add . or git commit -am "tmp" or something similar so we can decide whether a change is part of the patch or not.
- run tests
- call git status to check if any of the git tracked files are changed or any new ones are added during test runs
- add -1 to the pre-commit report if there are files laying around after the tests