Skip to content

Fix width/height parameter order in test cases#196

Open
CJay-Cipher wants to merge 1 commit into01-edu:masterfrom
CJay-Cipher:patch-1
Open

Fix width/height parameter order in test cases#196
CJay-Cipher wants to merge 1 commit into01-edu:masterfrom
CJay-Cipher:patch-1

Conversation

@CJay-Cipher
Copy link

Corrected dimension parameters in quadchecker test cases to follow the proper [quadName] [width] [height] format:

  • Fixed "A\nA" test: [2] [1] → [1] [2]
  • Fixed "AA" test: [1] [2] → [2] [1]
  • Fixed "A\nC" test: quadD → quadC, dimensions [2] [1] → [1] [2]
  • Fixed "AC" test: Added quadD match, dimensions [1] [2] → [2] [1]
  • Fixed multi-line quadA test: [3] [5] → [5] [3]

All expected outputs now correctly represent width (columns) before height (rows).

Corrected dimension parameters in quadchecker test cases to follow the proper [quadName] [width] [height] format:

- Fixed "A\nA" test: [2] [1] → [1] [2]
- Fixed "AA" test: [1] [2] → [2] [1]
- Fixed "A\nC" test: quadD → quadC, dimensions [2] [1] → [1] [2]
- Fixed "AC" test: Added quadD match, dimensions [1] [2] → [2] [1]
- Fixed multi-line quadA test: [3] [5] → [5] [3]

All expected outputs now correctly represent width (columns) before height (rows).
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.

1 participant