{"action":"created","issue":{"url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/43","labels_url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/43/labels{/name}","comments_url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/43/comments","events_url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/43/events","html_url":"https://github.com/lillian-lemmer/hypatia/pull/43","id":93004346,"number":43,"title":"[Test] Add a shell script to install `py.test`","user":{"login":"ejmr","id":780512,"avatar_url":"https://avatars.githubusercontent.com/u/780512?v=3","gravatar_id":"","url":"https://api.github.com/users/ejmr","html_url":"https://github.com/ejmr","followers_url":"https://api.github.com/users/ejmr/followers","following_url":"https://api.github.com/users/ejmr/following{/other_user}","gists_url":"https://api.github.com/users/ejmr/gists{/gist_id}","starred_url":"https://api.github.com/users/ejmr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ejmr/subscriptions","organizations_url":"https://api.github.com/users/ejmr/orgs","repos_url":"https://api.github.com/users/ejmr/repos","events_url":"https://api.github.com/users/ejmr/events{/privacy}","received_events_url":"https://api.github.com/users/ejmr/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-07-04T11:33:58Z","updated_at":"2015-07-04T15:45:23Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lillian-lemmer/hypatia/pulls/43","html_url":"https://github.com/lillian-lemmer/hypatia/pull/43","diff_url":"https://github.com/lillian-lemmer/hypatia/pull/43.diff","patch_url":"https://github.com/lillian-lemmer/hypatia/pull/43.patch"},"body":"This patch adds a shell script to the `tests/` directory which will\r\ninstall `py.test` on the developer's system, since that is the tool\r\nused for Hypatia's test suite.  The rationale for putting this into a\r\nseparate script and not the existing installation scripts is that\r\npytest is *not* required for running Hypatia, only for developing it.\r\nTherefore we do not want to install unnecessary dependencies for users\r\nwho are installing Hypatia with no intention of doing any serious\r\namount of development.\r\n\r\nThe installation script tries to install the Python 3 version of\r\npytest by testing for the existence of `pip3`.  The justification here\r\nis that if `pip3` exists then it is very likely the developer prefers\r\nto work on Hypatia using Python 3.  The script will fallback to the\r\nPython 2.7 version of `pip` if `pip3` is not available.\r\n\r\nThe script relies on the use of redirecting output to `/dev/null` as\r\npart of testing whether or not pytest installed successfully, which is\r\nan aspect of the script that may break on Windows systems.  I do not\r\nremember off the top of my head if Windows ports of shells like GNU\r\nBash handle `/dev/null` appropriately for that operating system.\r\n\r\nSigned-off-by: Eric James Michael Ritz <ejmr@plutono.com>\r\n\r\nGitHub-Issue: #42\r\nSee-Also: http://pytest.org/latest/"},"comment":{"url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/comments/118525214","html_url":"https://github.com/lillian-lemmer/hypatia/pull/43#issuecomment-118525214","issue_url":"https://api.github.com/repos/lillian-lemmer/hypatia/issues/43","id":118525214,"user":{"login":"lillian-lemmer","id":7135753,"avatar_url":"https://avatars.githubusercontent.com/u/7135753?v=3","gravatar_id":"","url":"https://api.github.com/users/lillian-lemmer","html_url":"https://github.com/lillian-lemmer","followers_url":"https://api.github.com/users/lillian-lemmer/followers","following_url":"https://api.github.com/users/lillian-lemmer/following{/other_user}","gists_url":"https://api.github.com/users/lillian-lemmer/gists{/gist_id}","starred_url":"https://api.github.com/users/lillian-lemmer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lillian-lemmer/subscriptions","organizations_url":"https://api.github.com/users/lillian-lemmer/orgs","repos_url":"https://api.github.com/users/lillian-lemmer/repos","events_url":"https://api.github.com/users/lillian-lemmer/events{/privacy}","received_events_url":"https://api.github.com/users/lillian-lemmer/received_events","type":"User","site_admin":false},"created_at":"2015-07-04T15:45:23Z","updated_at":"2015-07-04T15:45:23Z","body":"Thank you very much, ejmr! I really appreciate your continued support!\r\n\r\nThe tests I run (see: test.sh) require more than py.test. Thus, I think it'd be good to have a general `requirements/testing.txt` for `pip install --user requirements/testing.txt`, especially since all the requirements can be installed via pip.\r\n\r\nNonetheless, I'll make sure to actually *look* at your commits later. :) I pretty much just woke up--what a nice thing to wake up to, by the by. :) Thanks, again!"}}