tests: update crypto-square exercise (#775)#991
tests: update crypto-square exercise (#775)#991alienx5499 wants to merge 1 commit intoexercism:mainfrom
Conversation
… 54 character plaintext
|
Hello. Thanks for opening a PR on Exercism 🙂 We ask that all changes to Exercism are discussed on our Community Forum before being opened on GitHub. To enforce this, we automatically close all PRs that are submitted. That doesn't mean your PR is rejected but that we want the initial discussion about it to happen on our forum where a wide range of key contributors across the Exercism ecosystem can weigh in. You can use this link to copy this into a new topic on the forum. If we decide the PR is appropriate, we'll reopen it and continue with it, so please don't delete your local branch. If you're interested in learning more about this auto-responder, please read this blog post. Note: If this PR has been pre-approved, please link back to this PR on the forum thread and a maintainer or staff member will reopen it. |
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
Description
This PR updates the crypto-square exercise test cases to sync with the latest problem-specifications, contributing to issue #775 for updating test cases across exercises.
Changes Made
fbcb0c6d-4c39-4a31-83f6-c473baa6af80to33fd914e-fa44-445b-8f38-ff8fbc9fe6e6Files Modified
exercises/practice/crypto-square/.meta/tests.tomlexercises/practice/crypto-square/crypto_square_test.cppScreenshots
Before Changes
After Changes
Test Execution
Code Changes
Testing Instructions
To verify these changes work correctly:
Expected Test Output
Verification
✅ All 8 test cases pass
✅ New test case properly validates 54-character plaintext
✅ Configlet sync confirms no missing test cases for crypto-square
✅ Exercise maintains backward compatibility
Configlet Verification
Technical Details
configlet sync --tests include --update --yes -e crypto-squareto automatically sync test casesinclude = falsein tests.tomlreimplementsreference to show it replaces the old oneFile Diffs
Progress on Issue #775
This completes crypto-square exercise updates. Remaining exercises with missing test cases:
Fixes #775 (partial - crypto-square exercise)