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

Issue 337: Added pkg_key to ensure 'packages' == 'dependencies' #359

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Prev Previous commit
Next Next commit
TST: Updated test_prepare.py::test_default_to_system_environ for COND…
…A_STACK_2.

If the user has `auto_stack: 1` or higher and has a stacked environment
as indicated by `CONDA_STACK_2 == True`, then the PATH environmental
variable might be modified bt prepare.  This change ignores the error if
`CONDA_STACK_2 == True`.  A better resolution would be to control the
configuration of the conda environment before running tests.
  • Loading branch information
Michael McNeil Forbes committed Jan 30, 2022
commit 69a9dc9cd2c60d3b3be0329c2d87f7e08b5203e5
11 changes: 8 additions & 3 deletions anaconda_project/test/test_prepare.py
Original file line number Diff line number Diff line change
@@ -138,9 +138,14 @@ def prepare_system_environ(dirname):
if original in ('root', 'base') and updated in ('root', 'base'):
print("we have a root/base environment name issue here")
continue
if key == 'PATH' and platform.system() == 'Windows':
print("prepare changed PATH on Windows and ideally it would not.")
continue
if key == 'PATH':
if platform.system() == 'Windows':
print("prepare changed PATH on Windows and ideally it would not.")
continue
if result.environ.get("CONDA_STACKED_2", False):
print("prepare changed PATH with stacked environments.")
continue

updated = updated.split(os.pathsep)
original = original.split(os.pathsep)
print("ORIGINAL {}: {}".format(key, repr(original)))