MarshallOfSound / Google-Play-Music-Desktop-Player-UNOFFICIAL- / 1678 / 1

Ran 03 Jan 2017 06:03AM UTC

Files 47

Run time 2s

Badge

Embed ▾

Committed 03 Jan 2017 05:54AM UTC coverage: 92.242% (+0.1%) from 92.096%

Job # 1678.1

Build Type

push

travis-ci

Committed by

MarshallOfSound

Commit Message

Add a resolver for "Go To URL" modal to follow redirects (#2036)

* Add a resolver for "Go To URL" modal to follow redirects

* WIP Refactor based on requested changes

I believe this covers most of the changes that were requested. I haven't
refactored to using fetch/net yet, as it looks like that is a larger
change. I also haven't written new tests, although the pass-quick logic
does make the failing tests pass again.

* Rename fireURL to goToURL

minor change

* More small fixes (typos and linter issues)

* Replace request calls with fetch

This should be the last thing for basic redirect support, except writing
tests for coverage.

The internal function for resolveURL recursively follows redirects up to
10 times, and returns the resolved URL or error.

* Adding test-cases for resolver, refactored to be testable

* Test for invalid url, refactor for easier testability

* Fix a few formatting issues, add testcase for too many redirects

* Fix test description for stubbed fetch requests

Run Details

1296 of 1405 relevant lines covered (92.24%)

28.46 hits per line

MarshallOfSound / Google-Play-Music-Desktop-Player-UNOFFICIAL- / 1678 / 1

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1678.1

MarshallOfSound / Google-Play-Music-Desktop-Player-UNOFFICIAL- / 1678 / 1

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 1678.1

README BADGES
x