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

test: Adapt test_env_update_pypi_with_conda_forge #3455

Conversation

jjerphan
Copy link
Member

Resolve CI failures observed on #3450 and #3445.

Signed-off-by: Julien Jerphanion <git@jjerphan.xyz>
@jjerphan jjerphan added the release::bug_fixes For PRs fixing bugs label Sep 17, 2024
@@ -362,4 +362,4 @@ def test_env_update_pypi_with_conda_forge(tmp_home, tmp_root_prefix, tmp_path):
pip_list_output = helpers.umamba_run("-p", env_prefix, "pip", "list", "--format=json")
pip_packages_list = yaml.safe_load(pip_list_output)

assert any(pkg["name"] == "numpy" and pkg["version"] == "1.26.4" for pkg in pip_packages_list)
assert any(pkg["name"] == "numpy" for pkg in pip_packages_list)
Copy link
Member

Choose a reason for hiding this comment

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

What happens exactly? Is it updated with another version?

Copy link
Member Author

Choose a reason for hiding this comment

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

I cannot reproduce the error locally, and I needed to have a first relaxation of the test on the CI while working #3456 (hence the minimalist state of this PR).

The test suite does not fail anymore on main, so I am clueless for now. 🤷

@jjerphan
Copy link
Member Author

Closing since the error is not observed anymore.

@jjerphan jjerphan closed this Sep 18, 2024
@jjerphan jjerphan deleted the test/adapt-test_env_update_pypi_with_conda_forge branch September 18, 2024 09:48
@jjerphan jjerphan restored the test/adapt-test_env_update_pypi_with_conda_forge branch September 18, 2024 13:12
@jjerphan jjerphan reopened this Sep 18, 2024
@jjerphan
Copy link
Member Author

Reopening after the issue was observed again.

@Hind-M
Copy link
Member

Hind-M commented Sep 19, 2024

Superseded by #3459

@Hind-M Hind-M closed this Sep 19, 2024
@jjerphan jjerphan deleted the test/adapt-test_env_update_pypi_with_conda_forge branch September 19, 2024 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release::bug_fixes For PRs fixing bugs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants