-
Notifications
You must be signed in to change notification settings - Fork 0
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
VCR::Errors due to mismatch in API_BASE_URL in test setup #31
Comments
Setting the env variable in the spec helper was the simplest: ENV['IOKI_PLATFORM_API_BASE_URL'] ||= "https://demo.io.ki/api/"
ENV['IOKI_PASSENGER_API_BASE_URL'] ||= "https://demo.io.ki/api/"
ENV['IOKI_DRIVER_API_BASE_URL'] ||= "https://demo.io.ki/api/" |
Sounds good @eikes. Can you prepare a PR for this? |
A5308Y
added a commit
that referenced
this issue
Nov 22, 2021
Fixes #31 VCR::Errors due to mismatch in API_BASE_URL in test setup when run locally
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I just cloned this repository and noticed that the
spec/ioki/vcr
tests all failed with similar messages:The reason seems to be, that in the
spec/fixtures/vcr_cassettes
files thehttps://demo.io.ki/api/
API_BASE_URL is used, but this is not configured by default in the test setup wherehttps://app.io.ki/api/
is used.I was able to fix this by exporting the appropriate ENV variables as it is done in the
.github/ci_workflow.env
file:This should be documented somewhere or maybe simply use the
https://app.io.ki/api/
API_BASE_URL in the VCR files as well and remove these ENV vars from the CI setup? Or set the ENV variable in thespec_helper.rb
file?The text was updated successfully, but these errors were encountered: