Skip to content
Snippets Groups Projects
Commit a8be1460 authored by acpaquette's avatar acpaquette Committed by GitHub
Browse files

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
parent 6c2da895
No related branches found
No related tags found
No related merge requests found
...@@ -19,7 +19,6 @@ else: ...@@ -19,7 +19,6 @@ else:
try: try:
spice_root = os.environ['ALESPICEROOT'] spice_root = os.environ['ALESPICEROOT']
except: except:
warnings.warn('ALESPICEROOT environment variable not set, Spice Drivers will not function correctly')
spice_root = None spice_root = None
# bring ale stuff into main ale module # bring ale stuff into main ale module
......
...@@ -6,7 +6,6 @@ import ale ...@@ -6,7 +6,6 @@ import ale
def test_env_not_set(monkeypatch): def test_env_not_set(monkeypatch):
monkeypatch.delenv('ALESPICEROOT', raising=False) monkeypatch.delenv('ALESPICEROOT', raising=False)
with pytest.warns(UserWarning, match='ALESPICEROOT environment variable not set'):
reload(ale) reload(ale)
assert not ale.spice_root assert not ale.spice_root
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment