New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add uris kwarg to tracklist.add() #1065

merged 2 commits into from Mar 20, 2015


2 participants

jodal commented Mar 20, 2015

See commit messages.

jodal added some commits Mar 20, 2015

core: Make lookup(uris=...) return dict with all uris
All uris given to lookup should be in the result even if there is no
backend to handle the uri, and the lookup result thus is an empty list.

As a side effect, future.get() is now called in the order of the URIs
in the `uris` list, making it easier to mock out
backend.library.lookup() in core layer tests.

@jodal jodal added this to the v1.0 - Audio cleanup 1 milestone Mar 20, 2015

adamcik added a commit that referenced this pull request Mar 20, 2015

Merge pull request #1065 from jodal/fix/1060-tracklist-add-uris
core: Add uris kwarg to tracklist.add()

@adamcik adamcik merged commit 31d12c7 into mopidy:develop Mar 20, 2015

3 checks passed

Scrutinizer 2 updated code elements
continuous-integration/travis-ci/pr The Travis CI build passed
coverage/coveralls Coverage increased (+0.04%) to 76.4%

@jodal jodal deleted the jodal:fix/1060-tracklist-add-uris branch Mar 20, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment