some Python cleanups based on PyCharm inspections - #2999

test_addon_relative_path now fails since it was never run - somebody needs to have a look at that.

test_addon_relative_path now fails since it was never run - somebody needs to have a look at that.

ok great.. I can look at it in a few days.

ok great.. I can look at it in a few days.

Thanks. Happy new year!

test_addon_relative_path now fails since it was never run - somebody needs to have a look at that.

I think that the stdout output is ok. Could you please tweak the assertion so it ignores/checks that there is an extra Checking helloworld/main.c: SOME_CONFIG...

Turns out there was a missing newline in the assert I adjusted.

There is a new failure now ...

    def test_compile_commands_unused_function():
        create_unused_function_compile_commands()
        ret, stdout, stderr = cppcheck(['--enable=all', '--error-exitcode=1', '--project=./proj-inline-suppress-unusedFunction/compile_commands.json'])
>       assert ret == 1
E       AssertionError

@danmar Turns out this is also broken on main - please have a look.

If Appveyor and Travis are green this can be merged.