4
72%
dev: 72%

Ran 12 Dec 2018 08:41PM UTC

Files 185

Run time 23s

Badge

Embed ▾

Committed 12 Dec 2018 08:37PM UTC coverage: 26.827% (+0.04%) from 26.786%

Job # TOXENV=cov

Build Type

push

travis-ci-com

Committed by

maxandersen

Commit Message

Fix bad url creation and don't ignore 401

In #64 it was found that `hass-cli info` would return empty result
when there was 401 auth errors AND for some reason setting HASSIO_TOKEN was not
enough to make use of http://hassio/homeassistant as base url.

401 was easy fix by simply ensuring we will throw
an exception if error occurs + added basic test for this.

Hassio issue was because urljoin was used for combing baseurl with api
path. Now moved to basic string concat so it will work as expected, i.e.

http://hassio/homeassistant + /api/discovery_info gives http://hassio/homeassistant/api/discovery_info

And not http://hassio/api/discovery_info which was the cause of the bugs.

Run Details

9513 of 35460 relevant lines covered (26.83%)

0.27 hits per line

home-assistant / home-assistant-cli / 66 / 4
72%
dev: 72%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 66.4 (TOXENV=cov)

home-assistant / home-assistant-cli / 66 / 4 72% dev: 72%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 66.4 (TOXENV=cov)

home-assistant / home-assistant-cli / 66 / 4
72%
dev: 72%

README BADGES
x