Repository navigation
test(registry): boundary fixture corpus for decimal exponents, scripts and slug limits - #22
Open
jdjioe5-cpu wants to merge 1 commit into
Open
jdjioe5-cpu wants to merge 1 commit into
jdjioe5-cpu wants to merge 1 commit into
Conversation
…s and slug limits - Closes Kinlock-Org#15 (Roadmap row M1-40, Priority P1) - Create comprehensive boundary fixture corpus in fixtures/boundary/ covering 0-decimal (JPY), 3-decimal (BHD), non-Latin Unicode (Japanese), RTL (Arabic), diacritics (French), and exactly 80-char slug - Cover explicit rejection boundaries: 81-char slug, short payout address (55 chars), lowercase address character, swapped prefix (S), directory-country mismatch, unauthorized attester, and cross-country duplicate slug - Add 13 dedicated test assertions in scripts/boundary.test.ts asserting exact rejection error messages and verifying canonical UTF-8 hashing integrity (29 tests total green) - Author fixtures/boundary/README.md explaining the corpus - Live payees/, attesters/, and schemas/ completely untouched - Update ROADMAP.md marking M1-40 as DONE with Changelog entry
10 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #15. Fulfills Roadmap row
M1-40(Phase 1, M0 Validate, Priority P1).This PR introduces a comprehensive boundary-fixture corpus for the registry validator in
fixtures/boundary/:valid/):0-decimal-jpy.json: 0-decimal currency (JPY, Japan)3-decimal-bhd.json: 3-decimal currency (BHD, Bahrain)non-latin-japanese.json: Non-Latin Unicode script indisplay_name(Japanese)rtl-arabic.json: Right-to-Left script indisplay_name(Arabic)diacritics-french.json: Multi-word French name with diacriticsslug-exact-80-chars.json: Slug at exactly the 80-character maximum limitinvalid/):slug-81-chars.json: Rejection on exceeding 80 characters (81 chars)payout-address-short-55.json: Rejection on wrong address length (55 chars)payout-address-lowercase.json: Rejection on lowercase address characterpayout-address-swapped-prefix.json: Rejection on invalid prefix (Sinstead ofG/C)directory-country-mismatch.json: Rejection on file placed in wrong country folderattester-unauthorized-country.json: Rejection on attester not authorized for target countryduplicate-slug-cross-country.json: Rejection on duplicate slug across different countriesscripts/boundary.test.tsasserting exact error message patterns and confirming canonical UTF-8 hashing stability. Total 29/29 tests green.fixtures/boundary/README.md.M1-40asDONEinROADMAP.mdwith Changelog entry.Non-Personal Data & Multi-Country Guarantee
payees/,attesters/, andsupported-countries.jsonremain completely untouched.Real Command Outputs Executed