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

Issue 64: Added VIES test service #109

Merged
merged 1 commit into from
May 27, 2020
Merged

Conversation

DragonBe
Copy link
Owner

I have taken a look at this request to access the integration test service of VIES which were made in issue #64 and issue #83. I can conclude that yes, it is possible to access this test service of VIES for integration tests (resolves #64), but I was not able to verify if a backend service of VIES was not accessible (issue #83).

In my test case I used Germany as it was supposed to be "DE-Germany Unavailable daily from 11:00 PM to 01:30 AM" as was stated on https://ec.europa.eu/taxation_customs/vies/help.html. But on https://ec.europa.eu/taxation_customs/vies/monitoring.html it shows green. Not sure if the information in the help page is still accurate since my normal and test SOAP calls all succeeded.

I have taken a look at this request to access the integration test service of VIES which were made in issue #64 and issue #83. I can conclude that yes, it is possible to access this test service of VIES for integration tests (issue #64), but I was not able to verify if a backend service of VIES was not accessible.

In my test case I used Germany as it was supposed to be "DE-Germany	Unavailable daily from 11:00 PM to 01:30 AM" as was stated on https://ec.europa.eu/taxation_customs/vies/help.html. But on https://ec.europa.eu/taxation_customs/vies/monitoring.html it shows green. Not sure if the information in the help page is still accurate since my normal and test SOAP calls all succeeded.
@DragonBe DragonBe requested a review from krzaczek April 25, 2020 22:06
@DragonBe DragonBe merged commit 81102b3 into master May 27, 2020
DragonBe added a commit that referenced this pull request May 27, 2020
Fixes #64 and closes PR #109, with many thanks to @MPablo for reporting this issue. 👏
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cant use VIES test service
1 participant