Skip to content

Skip test_recipe_builds[unicode_all_over] for rattler solver - #5995

Merged
jezdez merged 2 commits into
conda:mainfrom
jsmolic:rattler-test-unicode
May 29, 2026
Merged

Skip test_recipe_builds[unicode_all_over] for rattler solver#5995
jezdez merged 2 commits into
conda:mainfrom
jsmolic:rattler-test-unicode

Conversation

@jsmolic

@jsmolic jsmolic commented May 28, 2026

Copy link
Copy Markdown
Contributor

The recipe contains characters that are not allowed by the rattler solver either; skip the test in that case too.

Originally discovered when running the test suite with rattler solver in #5989:

FAILED tests/test_api_build.py::test_recipe_builds[unicode_all_over] - conda.exceptions.InvalidMatchSpec: Invalid spec '♥=1': Invalid spec '♥=1': package name contains invalid characters (♥)

Description

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?

The recipe contains characters that are not allowed by the rattler
solver either; skip the test in that case too.

Originally discovered when running the test suite with rattler solver in
conda#5989:

```
FAILED tests/test_api_build.py::test_recipe_builds[unicode_all_over] - conda.exceptions.InvalidMatchSpec: Invalid spec '♥=1': Invalid spec '♥=1': package name contains invalid characters (♥)
```
@jsmolic
jsmolic requested a review from a team as a code owner May 28, 2026 16:24
@github-project-automation github-project-automation Bot moved this to 🆕 New in 🔎 Review May 28, 2026
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label May 28, 2026
@codspeed-hq

codspeed-hq Bot commented May 28, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 5 untouched benchmarks


Comparing jsmolic:rattler-test-unicode (49ab432) with main (8cd9d29)

Open in CodSpeed

@github-project-automation github-project-automation Bot moved this from 🆕 New to ✅ Approved in 🔎 Review May 29, 2026
@jezdez
jezdez enabled auto-merge (squash) May 29, 2026 07:14
@jezdez
jezdez merged commit 57ed884 into conda:main May 29, 2026
26 checks passed
@github-project-automation github-project-automation Bot moved this from ✅ Approved to 🏁 Done in 🔎 Review May 29, 2026
@jsmolic
jsmolic deleted the rattler-test-unicode branch May 29, 2026 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed [bot] added once the contributor has signed the CLA

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants