Add Static Code Checks for Test Files

[tcm-marcel]

Add code to the validator script ./script/validator.py that checks the test cases and test utilities for the naming conventions listed in https://github.com/cmu-db/peloton/blob/master/test/README.md.

(follow-up from #770)

[tharun323]

please assign this to me !

[tcm-marcel]

@tharun323 Sure, give it a try. Please make sure, that the code is proper Python 3 and formatted correctly. (we just did this in #1127)

Some more details about what we want to do

• extend the validator script ./script/validator.py to apply some checks on our test source files
• check all ./test/**/*_test.cpp files for
  1. correct class naming
  2. correct method naming (test case)
     My first idea is to parse the lines with the gtest Macro TEST_F(..) and extract the two identifiers to check them.
• check all ./test/**/*_util.h files for correct class naming
  We have some exceptions there, but I haven't decided yet how to treat them. (e.g. PelotonCodeGenTest)

With this check we might find some files with wrong naming.

An issue can only be assigned to members, so I assigned it to me to show someone is working on it. If you have any questions, fell free to ask right here.

[schedutron]

@tharun323 Are you still working on this? If not, can I try my hand at this issue, @tcm-marcel?

[tcm-marcel]

I emailed @tharun323 and didn't get any answer, he doesn't seem reliable.

@schedutron Sure, you can work on it!

[schedutron]

@tcm-marcel Cool! I'm busy with a conference till Wednesday, but will surely start working on static code checks after that!