{"comment":{"url":"https://api.github.com/repos/ruda/avocado/comments/10742092","html_url":"https://github.com/ruda/avocado/commit/b8a1e231456348bbd4864d74a12ddaf20a673b44#commitcomment-10742092","id":10742092,"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},"position":44,"line":164,"path":"avocado/plugins/runner.py","commit_id":"b8a1e231456348bbd4864d74a12ddaf20a673b44","created_at":"2015-04-15T18:16:53Z","updated_at":"2015-04-15T18:16:53Z","body":"Here's a quick patch I made to show my point\r\n\r\n\tdiff --git a/avocado/plugins/runner.py b/avocado/plugins/runner.py\r\n\tindex 0ecae4e..19f68bb 100644\r\n\t--- a/avocado/plugins/runner.py\r\n\t+++ b/avocado/plugins/runner.py\r\n\t@@ -35,6 +35,7 @@ class TestRunner(plugin.Plugin):\r\n\t     name = 'test_runner'\r\n\t     enabled = True\r\n\t     priority = 0\r\n\t+    view = None\r\n\t \r\n\t     def configure(self, parser):\r\n\t\t \"\"\"\r\n\t@@ -141,7 +142,9 @@ class TestRunner(plugin.Plugin):\r\n\t\t         else:\r\n\t\t             timeout = int(raw_timeout)\r\n\t\t     except (ValueError, TypeError):\r\n\t-                timeout = 0\r\n\t+                self.view.notify(event='error',\r\n\t+                                 msg=\"Invalid job timeout specification '%s'\" % self.view.app_args.job_timeout)\r\n\t+                return sys.exit(exit_codes.AVOCADO_FAIL)\r\n\t\t else:\r\n\t\t     timeout = 0\r\n\t\t return timeout\r\n\t@@ -152,14 +155,14 @@ class TestRunner(plugin.Plugin):\r\n\t \r\n\t\t :param args: Command line args received from the run subparser.\r\n\t\t \"\"\"\r\n\t-        view = output.View(app_args=args)\r\n\t+        self.view = output.View(app_args=args)\r\n\t\t if args.unique_job_id is not None:\r\n\t\t     try:\r\n\t\t         int(args.unique_job_id, 16)\r\n\t\t         if len(args.unique_job_id) != 40:\r\n\t\t             raise ValueError\r\n\t\t     except ValueError:\r\n\t-                view.notify(event='error', msg='Unique Job ID needs to be a 40 digit hex number')\r\n\t+                self.view.notify(event='error', msg='Unique Job ID needs to be a 40 digit hex number')\r\n\t\t         return sys.exit(exit_codes.AVOCADO_FAIL)\r\n\t\t args.job_timeout = self._validate_job_timeout(args.job_timeout)\r\n\t\t job_instance = job.Job(args)"}}