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 retrieve a lockfile for a build #807

Merged

Conversation

peytondmurray
Copy link
Contributor

Fixes #673.

Description

This PR adds a user journey test to add a lockfile for a build. Depends on #804, so I'll mark this as a draft until that one is merged.

This pull request:

  • Adds a user journey test for getting a lockfile for a successful build as an admin user, and checking that it contains the expected dependencies.
  • Adds a function to api_utils.py to retrieve the lockfile from the conda-store server

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_get_lockfile

Copy link

netlify bot commented Apr 4, 2024

Deploy Preview for conda-store canceled.

Name Link
🔨 Latest commit 64899ee
🔍 Latest deploy log https://app.netlify.com/sites/conda-store/deploys/6626ea51b1ab3600085f080d

@peytondmurray peytondmurray marked this pull request as ready for review April 4, 2024 23:15
Copy link
Contributor

@dcmcand dcmcand left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@peytondmurray peytondmurray merged commit 524a1cc into conda-incubator:main Apr 22, 2024
21 of 25 checks passed
@peytondmurray peytondmurray deleted the test-get-lockfile-673 branch April 22, 2024 23:08
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: Developer get lockfile for environment
2 participants