ALE Spice Directory Fix (#382)

* Set mission directories to an empty list if ALESPICEROOT is not set * Removed ALESPICEROOT environment variable not set warning * Add and update tests for when no ALESPICEROOT is set
DOI-USGS · May 30, 2022 · 8acadb5 · 8acadb5
1 parent 93c0743
commit 8acadb5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/ale/__init__.py b/ale/__init__.py
@@ -19,7 +19,6 @@
 try:
     spice_root = os.environ['ALESPICEROOT']
 except:
-    warnings.warn('ALESPICEROOT environment variable not set, Spice Drivers will not function correctly')
     spice_root = None
 
 # bring ale stuff into main ale module

diff --git a/tests/pytests/test_env.py b/tests/pytests/test_env.py
@@ -6,9 +6,8 @@
 
 def test_env_not_set(monkeypatch):
     monkeypatch.delenv('ALESPICEROOT', raising=False)
-    with pytest.warns(UserWarning, match='ALESPICEROOT environment variable not set'):
-        reload(ale)
-        assert not ale.spice_root
+    reload(ale)
+    assert not ale.spice_root
 
 def test_env_set(monkeypatch):
     monkeypatch.setenv('ALESPICEROOT', '/foo/bar')