Skip to content

Fillapix Test Suite#955

Open
jadeandtea wants to merge 20 commits intoBram-Hub:devfrom
chrism45621:main
Open

Fillapix Test Suite#955
jadeandtea wants to merge 20 commits intoBram-Hub:devfrom
chrism45621:main

Conversation

@jadeandtea
Copy link
Collaborator

Description

This commit contains a number of test puzzles for Fillapix.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Enhancement (improvement to an already existing feature)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

The tests pass.

Checklist:

  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (if applicable)
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Added 7 new puzzles for Nurikabe 5x5 easy
Implemented the toomanyblackcellscontradictionruletest. I wrote this code using 2 different tests which pass all tests. For those tests, I added 2 .txt files to represent the board. I also changed the name of one of the other Fillapix puzzles to be .txt.
Implemented TooFewBlackCellsContradictionRuleTest. This tests the toofewblackcells contradiction rule. It uses 2 different test boards as the test cases.
Implemented these files to fully represent the tiles of the fillapix puzzle.
Implemented BlackOrWhiteCaseRuleTest and a test grid along with it. This test case verifies that the blackorwhite case rule works, meaning that you can place black or white at a certain cell.
Implemented SatisfyClueCaseRuleTest, tests the case rule for fillapix by setting up a clue of value 2 with 3 unknown cells around it.
@jadeandtea jadeandtea changed the base branch from master to dev January 30, 2026 22:01
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