Skip to content

[WIP] Complete test cases for src/cuddCof.c to target 90% coverage#90

Merged
doganulus merged 3 commits into4.0.0from
copilot/add-test-cases-for-cuddcofc
Dec 1, 2025
Merged

[WIP] Complete test cases for src/cuddCof.c to target 90% coverage#90
doganulus merged 3 commits into4.0.0from
copilot/add-test-cases-for-cuddcofc

Conversation

Copy link
Copy Markdown

Copilot AI commented Nov 30, 2025

  • Analyze source file src/cuddCof.c and understand functions to test
  • Run initial coverage build and identify coverage gaps (10.4% initially)
  • Write comprehensive tests for Cudd_Cofactor function including error paths
  • Write comprehensive tests for Cudd_CheckCube function
  • Write comprehensive tests for Cudd_VarsAreSymmetric function
  • Write comprehensive tests for cuddGetBranches function
  • Write comprehensive tests for cuddCofactorRecur function paths
  • Write comprehensive tests for ddVarsAreSymmetricBefore function
  • Write comprehensive tests for ddVarsAreSymmetricBetween function
  • Verify 90%+ coverage is achieved (94.6% - 159/168 lines covered)
  • Fix code review issues (proper memory management for intermediate BDD nodes)
  • Pass CodeQL security check (0 alerts)
Original prompt

Complete test cases for src/cuddCof.c and target 90% coverage for the source file.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI and others added 2 commits November 30, 2025 17:45
Co-authored-by: doganulus <1174212+doganulus@users.noreply.github.com>
Co-authored-by: doganulus <1174212+doganulus@users.noreply.github.com>
Copilot AI requested a review from doganulus November 30, 2025 18:02
@doganulus doganulus marked this pull request as ready for review December 1, 2025 06:06
@doganulus doganulus merged commit deda337 into 4.0.0 Dec 1, 2025
38 of 71 checks passed
@doganulus doganulus deleted the copilot/add-test-cases-for-cuddcofc branch December 5, 2025 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants