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

Add user journey test to check that logs can be found for failed builds #797

Merged

Conversation

peytondmurray
Copy link
Contributor

Fixes #672.

Description

This PR implements a user journey test that checks that a user can access logs for a failed build.

This pull request:

  • Adds a simple broken environment spec
  • Implements a get_logs function for the user journey API utils
  • Adds a test for the user journey mentioned above

Pull request checklist

  • Did you test this change locally?
  • Did you update the documentation (if required)?
  • Did you add/update relevant tests for this change (if required)?

How to test

Run the conda-store server:

docker compose up

Then run the test:

cd conda-store-server
pytest tests/user_journeys/test_user_journeys.py::test_failed_build_logs

Copy link

netlify bot commented Mar 25, 2024

Deploy Preview for kaleidoscopic-dango-0cf31d ready!

Name Link
🔨 Latest commit df79cc6
🔍 Latest deploy log https://app.netlify.com/sites/kaleidoscopic-dango-0cf31d/deploys/6605c63fe95c930008748988
😎 Deploy Preview https://deploy-preview-797--kaleidoscopic-dango-0cf31d.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 configuration.

@peytondmurray
Copy link
Contributor Author

Test failures appear to be unrelated.

@peytondmurray peytondmurray merged commit 9a9496f into conda-incubator:main Mar 28, 2024
17 of 21 checks passed
@peytondmurray peytondmurray deleted the test-failed-build-logs-672 branch March 28, 2024 23:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done 💪🏾
Development

Successfully merging this pull request may close these issues.

Integration Test: access logs from failed buld
2 participants