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

Added support for Norwegian test numbers #10

Merged
merged 1 commit into from
Aug 26, 2024
Merged

Conversation

niklasmattsson
Copy link
Contributor

@niklasmattsson niklasmattsson commented Aug 23, 2024

No description provided.

Copy link

Code Coverage

Package Line Rate Branch Rate Complexity Health
VatValidation 100% 95% 274
VatValidation 100% 95% 274
Summary 100% (458 / 458) 95% (342 / 360) 548

@caspChristian
Copy link
Member

Det här riskerar att breaka på andra ställen där den används (på grund av andra länder som inte mappar upp)
Tycker inte dessa ska vara tillåtna som standard.

@caspChristian
Copy link
Member

Låt mig iaf först göra ett race och konvertera testfallen till XML innan vi tittar på att merga denna.

@caspChristian
Copy link
Member

Jag försökte hitta den faktiska sidan på skatteetaten.no för dessa, men hittar inget alls. skulle vilja att det är tydligt

Testerna är inlinade, samt att även sad path testas och inte bara happy

Copy link
Member

@caspChristian caspChristian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok om vi uppdaterar (amendar commiten) med full URL till vart testdata finns

@niklasmattsson
Copy link
Contributor Author

Ok om vi uppdaterar (amendar commiten) med full URL till vart testdata finns

https://testdata.skatteetaten.no (Kräver dock norsk e-identitet)

image

@niklasmattsson niklasmattsson merged commit 81276c9 into main Aug 26, 2024
2 checks passed
@niklasmattsson niklasmattsson deleted the no-test-numbers branch August 26, 2024 14:49
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