Alan-Jowett / bdd_test / 19196520224

88%

push

github

feat: Comprehensive code coverage improvements (#15)

* feat: Comprehensive code coverage improvements

- Add 12 new BDD test expressions covering edge cases and error paths
- Add 2 error handling tests (nonexistent files, empty files)
- Expand from 13 to 25 total tests (92% increase in test coverage)
- Generate comprehensive PNG visualizations for all test cases
- Clean up incorrectly placed output files in source directory
- Improve branch coverage with complex expressions, unusual variable names, and operator combinations

New test expressions:
- single_variable.txt: Minimal BDD with one variable
- deeply_nested.txt: Complex nested expressions (13 variables)
- multiple_not.txt: Chain of NOT operators
- all_operators.txt: Comprehensive AND/OR/XOR/NOT test
- many_variables.txt: Large OR chain (10 variables)
- same_variable.txt: Repeated variable expressions
- xor_chain.txt: XOR operator chains
- underscore_vars.txt: Variable naming edge cases
- single_variable_unusual_name.txt: Special characters in variable names
- lenient_parsing.txt: Parser flexibility demonstration

Error handling improvements:
- test_nonexistent_file: File not found scenarios
- test_empty_expression_file: Empty file handling

Expected coverage improvement from 90.7% to 95%+ line coverage and 87.66% to 95%+ branch coverage.

Signed-off-by: Alan Jowett <alan.jowett@microsoft.com>

* fix: Move edge case expressions to subfolder to fix CI documentation workflow

- Move problematic expression files (invalid_operator.txt, lenient_parsing.txt,
  comments_only.txt, empty_file.txt) to test_expressions/edge_cases/
- Move corresponding expected result files to test_expressions/default_ordering/edge_cases/
- Update CMakeLists.txt to reference moved files correctly
- CI documentation workflow now only processes valid expressions from main directory
- Maintains test coverage while fixing documentation generation failures

This resolves the issue where the documentation workflow was trying to ... (continued)

494 of 543 branches covered (90.98%)

Branch coverage included in aggregate %.

383 of 392 relevant lines covered (97.7%)

122.16 hits per line