Closed emontnemery closed 1 month ago
@coderabbitai review
tests/hacsbase/test_configuration.py (2)
`46-46`: Renaming of the test function to `test_ignore_experimental` accurately reflects its purpose. --- Line range hint `46-52`: Ensure the test covers all relevant scenarios for the `experimental` setting.Verification successful
--- The `experimental` setting is indeed referenced in multiple test files, indicating that it is being tested in various contexts. Here are the relevant locations: - `tests/test_config_flow.py` - `tests/test_data_client.py` - `tests/hacsbase/test_configuration.py` - `tests/integration/test_integration_setup.py` - `tests/conftest.py` This suggests that the `experimental` setting is covered in different scenarios across the test suite.Scripts Executed
The following scripts were executed for the analysis: --- Script: ```shell #!/bin/bash # Description: Verify that the `experimental` setting is tested comprehensively. # Test: Search for other occurrences of the `experimental` setting in tests. rg --type python $'experimental' tests/ ``` Length of output: 69 --- --- Script: ```shell #!/bin/bash # Description: Verify that the `experimental` setting is tested comprehensively. # Test: Search for other occurrences of the `experimental` setting in tests. rg --type py 'experimental' tests/ ``` Length of output: 1147
Correct name of configuration test which asserts experimental setting is ignored