Fixed names in ErrorHandler and enumeration of errors and removed explicit declaration of true or false in a CHECK

- changed variable names
- added breaks
- corrected a comment
- deleted explicit declaration of true/false in ifs
- fixed a bug in tests caused by passing the message by reference

Added comments in tests and changed a uint8_t to a uint16_t variable that counts the event action definitions in the eventActionStatusReport

Changed the way add action-definition function works after rechecking the standard and updated the corresponding tests as well as added documentation in those tests

# Conflicts:
#	CMakeLists.txt
#	src/Services/EventReportService.cpp

Remove unnecessary #define CATCH_CONFIG_MAIN to comply with the Catch2 tutorial, credits to @thodkatz for the catch (no pun intended :P)

Replace the unacceptable disgrace called parsing based on spaces with new, readString() based method. Also write some tests for insertion and calling, and general code cleanup and reformatting.

Be quiet Vera!

I said be quiet Vera! And you, clang-tidy, do not even think about raising your voice!

Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work

Revert "Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work"

This reverts commit db9e73ad

Revert "Got an idea to pacify clang-tidy without commenting anything out, let's see if it's going to work"

This reverts commit db9e73ad9d8508e59764d7359408aac962a20afe

Fix a blunder with the test version of call()

Attempt to calm down clang-tidy and fix some stupid warnings in my CRC code (nothing much, just turned the counter int in a for-loop into a uint32_t to shut gcc up)

Comment out the empty constructor to mute clang-tidy's warning about trivial constructors

Wrote better tests, fixed code format, added tests in main for testing purposes and added FailedStartOfExecutionError in ErrorHandler

Changed passing by value messages with pass by reference messages

- A potential bug