Skip to content

Add pytest-mock to meta.yaml test req#1233

Merged
lrandersson merged 1 commit into
conda:mainfrom
lrandersson:dev-ra-canary-issue
May 12, 2026
Merged

Add pytest-mock to meta.yaml test req#1233
lrandersson merged 1 commit into
conda:mainfrom
lrandersson:dev-ra-canary-issue

Conversation

@lrandersson
Copy link
Copy Markdown
Contributor

@lrandersson lrandersson commented May 12, 2026

Description

This PR intends to resolve the test failure on the canary builds:
32f91ad
Seems like I forgot to also update the recipe test requirements to include the new package: #1228. On main the canary build is failing with the new tests:

 ERROR tests/test_fcp.py::test_check_duplicates_files_returns_max_path_length
 ERROR tests/test_fcp.py::test_check_duplicates_files_empty_packages
 ERROR tests/test_fcp.py::test_check_duplicates_files_with_env_prefixes
 ERROR tests/test_fcp.py::test_check_duplicates_files_env_prefix_normalizes_trailing_slash

In particular:

E       fixture 'mocker' not found

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?

@lrandersson lrandersson self-assigned this May 12, 2026
@lrandersson lrandersson requested a review from a team as a code owner May 12, 2026 14:29
@conda-bot conda-bot added the cla-signed [bot] added once the contributor has signed the CLA label May 12, 2026
@github-project-automation github-project-automation Bot moved this to 🆕 New in 🔎 Review May 12, 2026
@github-project-automation github-project-automation Bot moved this from 🆕 New to ✅ Approved in 🔎 Review May 12, 2026
@lrandersson lrandersson merged commit aa15ee4 into conda:main May 12, 2026
4 checks passed
@github-project-automation github-project-automation Bot moved this from ✅ Approved to 🏁 Done in 🔎 Review May 12, 2026
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

Status: 🏁 Done

Development

Successfully merging this pull request may close these issues.

3 participants