Add test cases for the file metric calculation for C++ #89
Conversation
| @@ -58,7 +58,7 @@ This is quite slow and can take up to one or two hours but can provide good resu | |||
| Counts expressions that branch the control flow (if-statements, loops, catch-blocks, etc. - but no else/default/finally statements) and other expressions that are considered to increase the complexity of the code inside a file: | |||
|
|
|||
| - function declarations (including lambda expressions) | |||
| - logical operations (like AND and OR) | |||
| - binary logical operations (like AND and OR) | |||
There was a problem hiding this comment.
Clearification, as we do not count unary logical operations, like the use of the ! / negation operator.
| ["hpp", Languages.CPlusPlus], | ||
| ["hxx", Languages.CPlusPlus], | ||
| ["hh", Languages.CPlusPlus], |
There was a problem hiding this comment.
Added more file extensions for as supported by the GNU compiler for C++, but not the .C (capital C) extension, as we currently convert all file extension to lower case, and lower case c would indicate a file for the C programming language.
Note that we will have some trouble here anyway if we introduce a parser for the C programming language, as they also use the .h header files. We probably have to introduce a command line option to make the user decide whether his .h files should be interpreted as C or C++ code. Just leaving .h for C is no viable option here, as the use of the .h file extension is also very common for C++ headers.
| @@ -56,7 +56,7 @@ export function sortCouplingResults(couplingResult: CouplingResult) { | |||
| * @param metric Name of the metric. | |||
| * @param expected Expected test result. | |||
| * */ | |||
| export async function testFileMetrics(inputPath: string, metric: FileMetric, expected: number) { | |||
| export async function testFileMetric(inputPath: string, metric: FileMetric, expected: number) { | |||
There was a problem hiding this comment.
renamed testFileMetrics to testFileMetric, as it only checks one value.
ResistantBear
force-pushed
the
test/84/add-c++-tests
branch
from
318f981 to
36e12ad
Compare
ResistantBear
force-pushed
the
test/84/add-c++-tests
branch
from
36e12ad to
3b9ca10
Compare
Adds test source and header files in the C++ programming languages. These files are tested to compile correctly using the LLVM C++ compiler clang that comes with MacOS (command
clang -c -xc++ -std=c++20 <file>)Adds test cases that check if the file metrics are calculated correctly if these C++ files are used as input.
Additional notes:
for_range_loop) were not counted when calculating the complexity metricResolves #84