{"action":"created","issue":{"url":"https://api.github.com/repos/home-assistant/home-assistant/issues/6092","repository_url":"https://api.github.com/repos/home-assistant/home-assistant","labels_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/6092/labels{/name}","comments_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/6092/comments","events_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/6092/events","html_url":"https://github.com/home-assistant/home-assistant/pull/6092","id":208677908,"number":6092,"title":"WIP:Added tests for Efergy sensor.","user":{"login":"miniconfig","id":13227865,"avatar_url":"https://avatars.githubusercontent.com/u/13227865?v=3","gravatar_id":"","url":"https://api.github.com/users/miniconfig","html_url":"https://github.com/miniconfig","followers_url":"https://api.github.com/users/miniconfig/followers","following_url":"https://api.github.com/users/miniconfig/following{/other_user}","gists_url":"https://api.github.com/users/miniconfig/gists{/gist_id}","starred_url":"https://api.github.com/users/miniconfig/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/miniconfig/subscriptions","organizations_url":"https://api.github.com/users/miniconfig/orgs","repos_url":"https://api.github.com/users/miniconfig/repos","events_url":"https://api.github.com/users/miniconfig/events{/privacy}","received_events_url":"https://api.github.com/users/miniconfig/received_events","type":"User","site_admin":false},"labels":[{"id":527871105,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/cla-signed","name":"cla-signed","color":"008000","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2017-02-18T22:50:16Z","updated_at":"2017-02-18T22:50:20Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/6092","html_url":"https://github.com/home-assistant/home-assistant/pull/6092","diff_url":"https://github.com/home-assistant/home-assistant/pull/6092.diff","patch_url":"https://github.com/home-assistant/home-assistant/pull/6092.patch"},"body":"**Description:**\r\nI'm in the process of adding support for multiple sensors on one account, but I need to add test cases for it.  This is just the tests for the current functionality, but I was hoping someone could look at it and give me some feedback.  I'm particularly uncertain about the part where I set up the config  - I had to put in values that I would normally expect voluptuous to supply the defaults for, and I'm not sure how to make that better.  \r\n\r\n**Checklist:**\r\n\r\nIf user exposed functionality or configuration variables are added/changed:\r\n  - [ ] Documentation added/updated in [home-assistant.github.io](https://github.com/home-assistant/home-assistant.github.io)\r\n\r\nIf the code communicates with devices, web services, or third-party tools:\r\n  - [x] Local tests with `tox` run successfully. **Your PR cannot be merged unless tests pass**\r\n  - [ ] New dependencies have been added to the `REQUIREMENTS` variable ([example][ex-requir]).\r\n  - [ ] New dependencies are only imported inside functions that use them ([example][ex-import]).\r\n  - [ ] New dependencies have been added to `requirements_all.txt` by running `script/gen_requirements_all.py`.\r\n  - [ ] New files were added to `.coveragerc`.\r\n\r\nIf the code does not interact with devices:\r\n  - [ ] Local tests with `tox` run successfully. **Your PR cannot be merged unless tests pass**\r\n  - [ ] Tests have been added to verify that the new code works.\r\n\r\n[ex-requir]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L16\r\n[ex-import]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L51\r\n"},"comment":{"url":"https://api.github.com/repos/home-assistant/home-assistant/issues/comments/280881195","html_url":"https://github.com/home-assistant/home-assistant/pull/6092#issuecomment-280881195","issue_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/6092","id":280881195,"user":{"login":"mention-bot","id":15710697,"avatar_url":"https://avatars.githubusercontent.com/u/15710697?v=3","gravatar_id":"","url":"https://api.github.com/users/mention-bot","html_url":"https://github.com/mention-bot","followers_url":"https://api.github.com/users/mention-bot/followers","following_url":"https://api.github.com/users/mention-bot/following{/other_user}","gists_url":"https://api.github.com/users/mention-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/mention-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mention-bot/subscriptions","organizations_url":"https://api.github.com/users/mention-bot/orgs","repos_url":"https://api.github.com/users/mention-bot/repos","events_url":"https://api.github.com/users/mention-bot/events{/privacy}","received_events_url":"https://api.github.com/users/mention-bot/received_events","type":"User","site_admin":false},"created_at":"2017-02-18T22:50:20Z","updated_at":"2017-02-18T22:50:20Z","body":"@miniconfig, thanks for your PR! By analyzing the history of the files in this pull request, we identified @konikvranik, @balloob and @fabaff to be potential reviewers."}}
{"actor":{"display_login":"mention-bot"}}