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.

Sign up for GitHub

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

Jump to bottom

Use WireMock in weather extension integration tests #6047

Closed
jamesnetherton opened this issue Apr 26, 2024 · 1 comment
Closed

Use WireMock in weather extension integration tests #6047

jamesnetherton opened this issue Apr 26, 2024 · 1 comment
Assignees
@shravaniPshetty
Labels
area/testing area/weather test
Milestone
3.12.0

Comments

@jamesnetherton
Copy link
Contributor

Seems the test will call out to a real API, which I've seen fail a few times on CI.

Caused by: org.apache.hc.client5.http.HttpHostConnectException: Connect to http://api.openweathermap.org:80 [api.openweathermap.org/38.89.70.158] failed: Connection timed out: getsockopt

We should be able to record & replay the API calls with WireMock.
@shravaniPshetty
Copy link
Contributor

I'd want to work on it @jamesnetherton

shravaniPshetty pushed a commit to shravaniPshetty/camel-quarkus that referenced this issue Jun 18, 2024
Using WireMock in weather extension integration tests apache#6047
67fae84
jamesnetherton pushed a commit that referenced this issue Jun 18, 2024
@jamesnetherton
Using WireMock in weather extension integration tests #6047
ebb852f
@jamesnetherton jamesnetherton added this to the 3.12.0 milestone Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@shravaniPshetty shravaniPshetty

Labels
area/testing area/weather test
Projects
None yet
Milestone
3.12.0
Development

No branches or pull requests
2 participants
@jamesnetherton @shravaniPshetty