{"action":"created","issue":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/843","labels_url":"https://api.github.com/repos/avocado-framework/avocado/issues/843/labels{/name}","comments_url":"https://api.github.com/repos/avocado-framework/avocado/issues/843/comments","events_url":"https://api.github.com/repos/avocado-framework/avocado/issues/843/events","html_url":"https://github.com/avocado-framework/avocado/pull/843","id":111646049,"number":843,"title":"Loader make unittests run","user":{"login":"lmr","id":296807,"avatar_url":"https://avatars.githubusercontent.com/u/296807?v=3","gravatar_id":"","url":"https://api.github.com/users/lmr","html_url":"https://github.com/lmr","followers_url":"https://api.github.com/users/lmr/followers","following_url":"https://api.github.com/users/lmr/following{/other_user}","gists_url":"https://api.github.com/users/lmr/gists{/gist_id}","starred_url":"https://api.github.com/users/lmr/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lmr/subscriptions","organizations_url":"https://api.github.com/users/lmr/orgs","repos_url":"https://api.github.com/users/lmr/repos","events_url":"https://api.github.com/users/lmr/events{/privacy}","received_events_url":"https://api.github.com/users/lmr/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/avocado-framework/avocado/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":7,"created_at":"2015-10-15T15:43:50Z","updated_at":"2015-10-23T05:29:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/avocado-framework/avocado/pulls/843","html_url":"https://github.com/avocado-framework/avocado/pull/843","diff_url":"https://github.com/avocado-framework/avocado/pull/843.diff","patch_url":"https://github.com/avocado-framework/avocado/pull/843.patch"},"body":"Proposed solution for the card:\r\n\r\nhttps://trello.com/c/GwjMV3l7/455-bug-avocado-can-t-run-python-unittests-that-have-no-exec-permission"},"comment":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/comments/150476605","html_url":"https://github.com/avocado-framework/avocado/pull/843#issuecomment-150476605","issue_url":"https://api.github.com/repos/avocado-framework/avocado/issues/843","id":150476605,"user":{"login":"ldoktor","id":1074021,"avatar_url":"https://avatars.githubusercontent.com/u/1074021?v=3","gravatar_id":"","url":"https://api.github.com/users/ldoktor","html_url":"https://github.com/ldoktor","followers_url":"https://api.github.com/users/ldoktor/followers","following_url":"https://api.github.com/users/ldoktor/following{/other_user}","gists_url":"https://api.github.com/users/ldoktor/gists{/gist_id}","starred_url":"https://api.github.com/users/ldoktor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldoktor/subscriptions","organizations_url":"https://api.github.com/users/ldoktor/orgs","repos_url":"https://api.github.com/users/ldoktor/repos","events_url":"https://api.github.com/users/ldoktor/events{/privacy}","received_events_url":"https://api.github.com/users/ldoktor/received_events","type":"User","site_admin":false},"created_at":"2015-10-23T05:29:33Z","updated_at":"2015-10-23T05:29:33Z","body":"The more I think about it I considering having only the unsafe version. Because when the user is idiot and wants to probe nasty code, than he deserves the screwed-up system. Normal people probe only tests they intend to run and tests they intend to run will eventually be executed.\r\n\r\nSo ast-based discover is pretty cool and surely can be extended of something similar to `ast.literal_eval` to probe for even more code. But even that would not cover 100% and would be eventually prone to some malicious code.\r\n\r\nAll in all I'd focus on real users who want to run they tests rather than on avocado-qa who want to find the corner-case which breaks it. We should probably even remove the stdout/stderr hacks to notify users their tests are written badly (or replace them with iostring and raise exception when they were used during discovery) and we might add test-discover-timeout to timeout in case the discovery takes longer than xxx."}}