Skip to content

fix(pet-conda): shorten Windows test fixture path#377

Open
benogeorge wants to merge 1 commit intomicrosoft:mainfrom
benogeorge:fix/pet-conda-long-path
Open

fix(pet-conda): shorten Windows test fixture path#377
benogeorge wants to merge 1 commit intomicrosoft:mainfrom
benogeorge:fix/pet-conda-long-path

Conversation

@benogeorge
Copy link

Rename the long conda fixture root so Windows GitHub Actions checkouts stay under the path limit.

This preserves the history-based manager detection coverage while avoiding the long-path failure reported in #171.

Fixes #171
Co-Authored-By: Claude noreply@anthropic.com

Rename the long conda fixture root so Windows GitHub Actions checkouts stay under the path limit.

Keep the history-based manager detection test coverage intact while avoiding the cargo fetch failure reported in microsoft#171.

Fixes microsoft#171
Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings March 23, 2026 16:35
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR shortens the Conda test fixture directory name under crates/pet-conda/tests/unix/ to avoid Windows path-length checkout failures (as reported in #171), while keeping the history-based “manager detection from history” test coverage intact.

Changes:

  • Rename the long fixture root to conda_hist and update the Rust test to reference the new fixture path.
  • Update the embedded paths in the fixture history file to reflect the new location.
  • Add/move the associated Conda fixture files (conda-meta JSONs, history files, and bin stubs) under the new shorter directory.

Reviewed changes

Copilot reviewed 2 out of 26 changed files in this pull request and generated no comments.

Show a summary per file
File Description
crates/pet-conda/tests/lib_test.rs Updates fixture root reference from the old long name to conda_hist.
crates/pet-conda/tests/unix/conda_hist/env_python_3/conda-meta/history Updates embedded # cmd: paths to the new conda_hist fixture location.
crates/pet-conda/tests/unix/conda_hist/env_python_3/conda-meta/history_template Template used by the test to generate a machine-local history with <CONDA_INSTALL> substituted.
crates/pet-conda/tests/unix/conda_hist/env_python_3/conda-meta/python_abi-3.12-4_cp312.json Conda meta fixture file used for env identification/versioning coverage.
crates/pet-conda/tests/unix/conda_hist/env_python_3/conda-meta/python-dateutil-2.8.2-pyhd8ed1ab_0.json Conda meta fixture file under the shortened path.
crates/pet-conda/tests/unix/conda_hist/env_python_3/bin/python3.12 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/env_python_3/bin/python3.1 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/env_python_3/bin/python3 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/env_python_3/bin/python Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/env_python_3/bin/conda Fixture conda stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/envs/.conda_envs_dir_test Marker file for envs dir fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/condabin/conda Conda executable stub fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/bin/python3.10 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/bin/python3.1 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/bin/python3 Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/bin/python Fixture python stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/bin/conda Fixture conda stub under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/history Conda install history fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/conda-23.1.0-py310hca03da5_0.json Conda package metadata fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/conda-build-3.23.3-py310hca03da5_0.json Conda-build package metadata fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/pytest-7.1.2-py310hca03da5_0.json Pytest package metadata fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/python-dateutil-2.8.2-pyhd3eb1b0_0.json python-dateutil package metadata fixture under shortened path.
crates/pet-conda/tests/unix/conda_hist/some_other_location/conda_install/conda-meta/python-fastjsonschema-2.16.2-py310hca03da5_0.json fastjsonschema package metadata fixture under shortened path (previously hit by Windows path limit).

@benogeorge
Copy link
Author

benogeorge commented Mar 23, 2026 via email

@benogeorge
Copy link
Author

benogeorge commented Mar 23, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pet crate can't be fetched by cargo in GitHub Actions machine with Windows

2 participants