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

Fix testing of features at the workspace level #37

Merged
merged 2 commits into from
Dec 3, 2023
Merged

Fix testing of features at the workspace level #37

merged 2 commits into from
Dec 3, 2023

Conversation

jmmv
Copy link
Owner

@jmmv jmmv commented Dec 3, 2023

The test.sh script contains logic to avoid testing features that don't exist in the subcrates, but this logic breaks apart when run from the top level. Fix this by always running with the requested features at the top level.

The test.sh script contains logic to avoid testing features that don't
exist in the subcrates, but this logic breaks apart when run from the
top level.  Fix this by always running with the requested features at
the top level.
Sometimes, Azure geolocation tests in pull requests fail with a 401,
saying that the Maps key is wrong.  However, this only happens to a
test within the same PR run and only randomly, so the configuration
is correct.

Try to workaround this with retries, which are also a good idea to
have because we are making the tests depend on an external service.
@jmmv jmmv merged commit 535e7dc into main Dec 3, 2023
6 checks passed
@jmmv jmmv deleted the tests branch December 3, 2023 16:21
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.

1 participant