selfie icon indicating copy to clipboard operation
selfie copied to clipboard

Rename duplicated test method for config parsing

Open emmanuel-ferdman opened this issue 8 months ago • 1 comments

PR Summary

This small PR fixes the renames the duplicated test method for config parsing.

emmanuel-ferdman avatar Jun 10 '25 10:06 emmanuel-ferdman

@emmanuel-ferdman what is the purpose of this change?

ckirsch avatar Jun 23 '25 13:06 ckirsch

@ckirsch The purpose of this change is to address an issue where there are two distinct test methods defined with the exact same name: test_invalid_command_placement. In Python, when you define multiple methods with the same name within a single class, the last definition overwrites any previous ones. As a result, only the second test (the one intended to check invalid_format_cfg) was actually being discovered and executed by the test runner. The first test (intended to check invalid_command_placement_cfg) was silent and never ran, potentially hiding a bug or a gap in the test coverage for that specific scenario.

emmanuel-ferdman avatar Jun 23 '25 14:06 emmanuel-ferdman