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 stage tests #1756

Merged
merged 5 commits into from
Jun 5, 2023
Merged

Fix stage tests #1756

merged 5 commits into from
Jun 5, 2023

Conversation

chr-stian
Copy link
Contributor

No description provided.

chr-stian and others added 5 commits June 1, 2023 18:50
* rh-certified not in stage
No-Issue
* Add a document about keyword searching.
* Add doc to index.

No-Issue

Signed-off-by: James Tanner <[email protected]>
* Handle unexpected docker errors
No-Issue

* forgot the else bit
No-Issue
* rh-certified not in stage
No-Issue
@netlify
Copy link

netlify bot commented Jun 2, 2023

Deploy Preview for galaxyng ready!

Name Link
🔨 Latest commit 4dcba3e
🔍 Latest deploy log https://app.netlify.com/sites/galaxyng/deploys/6479efef99a3d10007cdfd50
😎 Deploy Preview https://deploy-preview-1756--galaxyng.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@chr-stian
Copy link
Contributor Author

chr-stian commented Jun 2, 2023

this PR is to fix stage tests but CI failed so I needed to update these tests:
galaxy_ng/tests/integration/api/test_cross_repository_search.py
please, let me know if this is wrong. I don't understand why test_cross_repository_search failed here but not here
I locally reproduced the failure and saw that the test was expecting an artifact to be in staging, but it was in published. But the same passed in a different PR that didn't touch anything related to this.
@awcrosby @jctanner @rochacbruno

@awcrosby awcrosby merged commit accc2eb into ansible:master Jun 5, 2023
EamonTracey pushed a commit to EamonTracey/galaxy_ng that referenced this pull request Jun 9, 2023
No-Issue

---------

Signed-off-by: James Tanner <[email protected]>
Co-authored-by: jctanner <[email protected]>
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.

3 participants