Skip to content

test: regression tests for bugs in 5645 and 5644#5648

Merged
isuruf merged 17 commits into25.3.xfrom
regr-tests
Mar 24, 2025
Merged

test: regression tests for bugs in 5645 and 5644#5648
isuruf merged 17 commits into25.3.xfrom
regr-tests

Conversation

@beckermr
Copy link
Copy Markdown
Contributor

@beckermr beckermr commented Mar 21, 2025

Description

This PR adds a test for the regressions reported in #5645 and #5644.

Checklist - did you ...

  • Add a file to the news directory (using the template) for the next release's release notes?
  • Add / update necessary tests?
  • Add / update outdated documentation?

Extra items

The test failures in #5649 are:

FAILED tests/test_api_build.py::test_api_build_grpc_issue5632 - conda_libmamba_solver.conda_build_exceptions.ExplainedDependencyNeedsBuildingError: Unsatisfiable dependencies for platform linux-64: {MatchSpec("python==3.11[build=*_cpython]")}
Encountered problems while solving:
  - unsupported request

Could not solve for environment specs
The following package could not be installed
└─ python =3.11,(>=3.10,<3.11.0a0) *_cpython does not exist (perhaps a typo or a missing channel).

This is the correct failure due to the conflicting python specs.

The other test failure for #5644 that I get locally looks correct too:

FAILED

================================================================================ FAILURES ================================================================================
__________________________________________________________________ test_api_build_pytorch_cpu_issue5644 __________________________________________________________________

Traceback (most recent call last):
  File "/Users/beckermr/mambaforge/envs/cf-dev/lib/python3.11/site-packages/conda/models/version.py", line 44, in __call__
    return cls._cache_[arg]
           ~~~~~~~~~~~^^^^^
KeyError: '>=3.10,<3.11.0a0'
...
conda.exceptions.InvalidVersionSpec: Invalid version '>=3.10,<3.11.0a0': invalid character(s)

This looks correct relative to the original issue, but is for a different python version which is fine. I will check back on the CI version in a bit to ensure that works as well.

Yep the test failure is correct:

FAILED tests/test_api_build.py::test_api_build_pytorch_cpu_issue5644 - conda.exceptions.InvalidVersionSpec: Invalid version '>=3.10,<3.11.0a0': invalid character(s)

pre-commit-ci bot and others added 7 commits March 17, 2025 11:52
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.9.10 → v0.11.0](astral-sh/ruff-pre-commit@v0.9.10...v0.11.0)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Merge `25.3.x` back into `main`
@github-project-automation github-project-automation bot moved this to 🆕 New in 🔎 Review Mar 21, 2025
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label Mar 21, 2025
@beckermr beckermr changed the title Regr tests test: regression tests for bugs in 5645 and 5644 Mar 21, 2025
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 21, 2025

CodSpeed Performance Report

Merging #5648 will not alter performance

Comparing regr-tests (8c88aed) with 25.3.x (e77c4c6)

Summary

✅ 5 untouched benchmarks

@beckermr beckermr marked this pull request as ready for review March 22, 2025 11:59
@beckermr beckermr requested a review from a team as a code owner March 22, 2025 11:59
@beckermr
Copy link
Copy Markdown
Contributor Author

@conda/builds-tools This PR is ready for review!

@isuruf isuruf enabled auto-merge (squash) March 24, 2025 12:31
@isuruf
Copy link
Copy Markdown
Contributor

isuruf commented Mar 24, 2025

Thanks

@beckermr
Copy link
Copy Markdown
Contributor Author

If you approve the pr @isuruf, we can merge and then check your fix against the new tests.

@isuruf isuruf merged commit 70e79f2 into 25.3.x Mar 24, 2025
28 checks passed
@isuruf isuruf deleted the regr-tests branch March 24, 2025 12:55
@github-project-automation github-project-automation bot moved this from 🆕 New to 🏁 Done in 🔎 Review Mar 24, 2025
@isuruf
Copy link
Copy Markdown
Contributor

isuruf commented Mar 24, 2025

Ah, that's what github was waiting for

@github-actions github-actions bot added the locked [bot] locked due to inactivity label Mar 25, 2026
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 25, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed [bot] added once the contributor has signed the CLA locked [bot] locked due to inactivity

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

5 participants