{"action":"opened","issue":{"url":"https://api.github.com/repos/desihub/desispec/issues/184","repository_url":"https://api.github.com/repos/desihub/desispec","labels_url":"https://api.github.com/repos/desihub/desispec/issues/184/labels{/name}","comments_url":"https://api.github.com/repos/desihub/desispec/issues/184/comments","events_url":"https://api.github.com/repos/desihub/desispec/issues/184/events","html_url":"https://github.com/desihub/desispec/issues/184","id":154312483,"number":184,"title":"Integration tests use DESIMODEL but don't insist that it be set.","user":{"login":"weaverba137","id":3780690,"avatar_url":"https://avatars.githubusercontent.com/u/3780690?v=3","gravatar_id":"","url":"https://api.github.com/users/weaverba137","html_url":"https://github.com/weaverba137","followers_url":"https://api.github.com/users/weaverba137/followers","following_url":"https://api.github.com/users/weaverba137/following{/other_user}","gists_url":"https://api.github.com/users/weaverba137/gists{/gist_id}","starred_url":"https://api.github.com/users/weaverba137/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/weaverba137/subscriptions","organizations_url":"https://api.github.com/users/weaverba137/orgs","repos_url":"https://api.github.com/users/weaverba137/repos","events_url":"https://api.github.com/users/weaverba137/events{/privacy}","received_events_url":"https://api.github.com/users/weaverba137/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/desihub/desispec/labels/bug","name":"bug","color":"fc2929"}],"state":"open","locked":false,"assignee":{"login":"sbailey","id":218471,"avatar_url":"https://avatars.githubusercontent.com/u/218471?v=3","gravatar_id":"","url":"https://api.github.com/users/sbailey","html_url":"https://github.com/sbailey","followers_url":"https://api.github.com/users/sbailey/followers","following_url":"https://api.github.com/users/sbailey/following{/other_user}","gists_url":"https://api.github.com/users/sbailey/gists{/gist_id}","starred_url":"https://api.github.com/users/sbailey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbailey/subscriptions","organizations_url":"https://api.github.com/users/sbailey/orgs","repos_url":"https://api.github.com/users/sbailey/repos","events_url":"https://api.github.com/users/sbailey/events{/privacy}","received_events_url":"https://api.github.com/users/sbailey/received_events","type":"User","site_admin":false},"milestone":null,"comments":0,"created_at":"2016-05-11T18:36:50Z","updated_at":"2016-05-11T18:36:50Z","closed_at":null,"body":"Integration tests use the value of `DESIMODEL`, but the `check_env()` function does not require that it be set.\r\n\r\nLine 120 of integration_test.py:\r\n\r\n```python\r\n psffile = '{}/data/specpsf/psf-{}.fits'.format(os.getenv('DESIMODEL'), channel)\r\n```\r\n\r\nThere are probably other environment variables in this situation.  For example the use of `STD_TEMPLATES` really is optional, but its default value is based on `DESI_ROOT` and that is also not checked by `check_env()`."}}