Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wrapper for isPossibleNumber & isValidNumber #33

Merged
merged 141 commits into from
Apr 29, 2024

Conversation

Anrufliste
Copy link
Collaborator

This PR is work in progress. Currently the Testcases are prepared, to check the existing two method (and the upcoming wrapper methods) against invalid German NDC.

Those are generated from the list of valid NDCs where a specific range does not use all possible variants.

@Anrufliste Anrufliste marked this pull request as draft October 11, 2023 15:23
Anrufliste and others added 26 commits October 12, 2023 21:21
…C up to 0999 - city are codes finished ... only need todo special codes
Update branch to catch up with current main branch
- Moving "check if original lib fixed non check of NAC..." from NormalizationTest to IsPossibleNumberWithReasonTest
- improve logging in NormalizationTest & PhoneNumberOfflineGeocoderTest
- Moving "check if original lib fixed isValidNumber for invalid German NDC" from IsPossibleNumberWithReasonTest into new IsValidNumberTest
Use PhoneLib 8.13.27 and prepare release
Start Snapshot 1.1.17 after release 1.1.16
@Anrufliste
Copy link
Collaborator Author

Made a rebase on current master

…Test: German invalid traffic routing 01981xx of mobile Emergency calls

Enabled slf4j in testing
…est: German traffic routing 01982 of Emergency calls

Fix slf4j warn(ing) in testing
…est: German traffic routing 01986 of public service calls
…est: German traffic routing 01987 of EU public service calls
…est: German traffic routing 01988 for international free calls
…est: German traffic routing 01989 for Call Assistant
…est: German traffic routing 0199 for internal traffic routing
@Anrufliste Anrufliste dismissed kaikreuzer’s stale review April 28, 2024 09:51

All Feedback is adapted into the branch

…est: German personal numbers 032 range - low level reserve
…est: German personal numbers 032 range - mid level reserve
…est: German personal numbers 032 range - high level reserve
…est: invalid German NDC

This is split up into 4 test, because of the 64kB Size limit when adapting them: groovyjarjarasm.asm.MethodTooLargeException

The ranges are 010 - 02999; 030 - 039999; 040 - 069; 0700 - 0999
@Anrufliste Anrufliste changed the title WIP: Wrapper for isPossibleNumber & isValidNumber Wrapper for isPossibleNumber & isValidNumber Apr 28, 2024
@Anrufliste Anrufliste marked this pull request as ready for review April 28, 2024 14:34
@Anrufliste Anrufliste merged commit 79e9647 into main Apr 29, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants