File: test_library.py

package info (click to toggle)
mopidy 3.4.2-4
  • links: PTS, VCS
  • area: main
  • in suites: forky
  • size: 2,616 kB
  • sloc: python: 16,656; sh: 159; makefile: 126
file content (598 lines) | stat: -rw-r--r-- 25,032 bytes parent folder | download | duplicates (4)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
426
427
428
429
430
431
432
433
434
435
436
437
438
439
440
441
442
443
444
445
446
447
448
449
450
451
452
453
454
455
456
457
458
459
460
461
462
463
464
465
466
467
468
469
470
471
472
473
474
475
476
477
478
479
480
481
482
483
484
485
486
487
488
489
490
491
492
493
494
495
496
497
498
499
500
501
502
503
504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
566
567
568
569
570
571
572
573
574
575
576
577
578
579
580
581
582
583
584
585
586
587
588
589
590
591
592
593
594
595
596
597
598
import unittest
from unittest import mock

from mopidy import backend, core
from mopidy.internal import validation
from mopidy.models import Image, Ref, SearchResult, Track


class BaseCoreLibraryTest(unittest.TestCase):
    def setUp(self):  # noqa: N802
        dummy1_root = Ref.directory(uri="dummy1:directory", name="dummy1")
        self.backend1 = mock.Mock()
        self.backend1.uri_schemes.get.return_value = ["dummy1"]
        self.backend1.actor_ref.actor_class.__name__ = "DummyBackend1"
        self.library1 = mock.Mock(spec=backend.LibraryProvider)
        self.library1.get_images.return_value.get.return_value = {}
        self.library1.root_directory.get.return_value = dummy1_root
        self.backend1.library = self.library1
        self.backend1.has_playlists.return_value.get.return_value = False

        dummy2_root = Ref.directory(uri="dummy2:directory", name="dummy2")
        self.backend2 = mock.Mock()
        self.backend2.uri_schemes.get.return_value = ["dummy2", "du2"]
        self.backend2.actor_ref.actor_class.__name__ = "DummyBackend2"
        self.library2 = mock.Mock(spec=backend.LibraryProvider)
        self.library2.get_images.return_value.get.return_value = {}
        self.library2.root_directory.get.return_value = dummy2_root
        self.backend2.library = self.library2
        self.backend2.has_playlists.return_value.get.return_value = False

        # A backend without the optional library provider
        self.backend3 = mock.Mock()
        self.backend3.uri_schemes.get.return_value = ["dummy3"]
        self.backend3.actor_ref.actor_class.__name__ = "DummyBackend3"
        self.backend3.has_library.return_value.get.return_value = False
        self.backend3.has_library_browse.return_value.get.return_value = False

        self.core = core.Core(
            mixer=None, backends=[self.backend1, self.backend2, self.backend3]
        )


# TODO: split by method
class CoreLibraryTest(BaseCoreLibraryTest):
    def test_get_images_returns_empty_dict_for_no_uris(self):
        assert {} == self.core.library.get_images([])

    def test_get_images_returns_empty_result_for_unknown_uri(self):
        result = self.core.library.get_images(["dummy4:track"])
        assert {"dummy4:track": ()} == result

    def test_get_images_returns_empty_result_for_library_less_uri(self):
        result = self.core.library.get_images(["dummy3:track"])
        assert {"dummy3:track": ()} == result

    def test_get_images_maps_uri_to_backend(self):
        self.core.library.get_images(["dummy1:track"])
        self.library1.get_images.assert_called_once_with(["dummy1:track"])
        self.library2.get_images.assert_not_called()

    def test_get_images_maps_uri_to_backends(self):
        self.core.library.get_images(["dummy1:track", "dummy2:track"])
        self.library1.get_images.assert_called_once_with(["dummy1:track"])
        self.library2.get_images.assert_called_once_with(["dummy2:track"])

    def test_get_images_returns_images(self):
        self.library1.get_images.return_value.get.return_value = {
            "dummy1:track": [Image(uri="uri")]
        }

        result = self.core.library.get_images(["dummy1:track"])
        assert {"dummy1:track": (Image(uri="uri"),)} == result

    def test_get_images_merges_results(self):
        self.library1.get_images.return_value.get.return_value = {
            "dummy1:track": [Image(uri="uri1")]
        }
        self.library2.get_images.return_value.get.return_value = {
            "dummy2:track": [Image(uri="uri2")]
        }

        result = self.core.library.get_images(
            ["dummy1:track", "dummy2:track", "dummy3:track", "dummy4:track"]
        )
        expected = {
            "dummy1:track": (Image(uri="uri1"),),
            "dummy2:track": (Image(uri="uri2"),),
            "dummy3:track": (),
            "dummy4:track": (),
        }
        assert expected == result

    def test_browse_root_returns_dir_ref_for_each_lib_with_root_dir_name(self):
        result = self.core.library.browse(None)

        assert result == [
            Ref.directory(uri="dummy1:directory", name="dummy1"),
            Ref.directory(uri="dummy2:directory", name="dummy2"),
        ]
        assert not self.library1.browse.called
        assert not self.library2.browse.called
        assert not self.backend3.library.browse.called

    def test_browse_empty_string_returns_nothing(self):
        result = self.core.library.browse("")

        assert result == []
        assert not self.library1.browse.called
        assert not self.library2.browse.called

    def test_browse_dummy1_selects_dummy1_backend(self):
        self.library1.browse.return_value.get.return_value = [
            Ref.directory(uri="dummy1:directory:/foo/bar", name="bar"),
            Ref.track(uri="dummy1:track:/foo/baz.mp3", name="Baz"),
        ]

        self.core.library.browse("dummy1:directory:/foo")

        assert self.library1.browse.call_count == 1
        assert self.library2.browse.call_count == 0
        self.library1.browse.assert_called_with("dummy1:directory:/foo")

    def test_browse_dummy2_selects_dummy2_backend(self):
        self.library2.browse.return_value.get.return_value = [
            Ref.directory(uri="dummy2:directory:/bar/baz", name="quux"),
            Ref.track(uri="dummy2:track:/bar/foo.mp3", name="Baz"),
        ]

        self.core.library.browse("dummy2:directory:/bar")

        assert self.library1.browse.call_count == 0
        assert self.library2.browse.call_count == 1
        self.library2.browse.assert_called_with("dummy2:directory:/bar")

    def test_browse_dummy3_returns_nothing(self):
        result = self.core.library.browse("dummy3:test")

        assert result == []
        assert self.library1.browse.call_count == 0
        assert self.library2.browse.call_count == 0

    def test_browse_dir_returns_subdirs_and_tracks(self):
        self.library1.browse.return_value.get.return_value = [
            Ref.directory(uri="dummy1:directory:/foo/bar", name="Bar"),
            Ref.track(uri="dummy1:track:/foo/baz.mp3", name="Baz"),
        ]

        result = self.core.library.browse("dummy1:directory:/foo")
        assert result == [
            Ref.directory(uri="dummy1:directory:/foo/bar", name="Bar"),
            Ref.track(uri="dummy1:track:/foo/baz.mp3", name="Baz"),
        ]

    def test_lookup_returns_empty_dict_for_no_uris(self):
        assert {} == self.core.library.lookup(uris=[])

    def test_lookup_can_handle_uris(self):
        track1 = Track(uri="dummy1:a", name="abc")
        track2 = Track(uri="dummy2:a", name="def")

        self.library1.lookup().get.return_value = [track1]
        self.library2.lookup().get.return_value = [track2]

        result = self.core.library.lookup(uris=["dummy1:a", "dummy2:a"])
        assert result == {
            "dummy2:a": [track2],
            "dummy1:a": [track1],
        }

    def test_lookup_uris_returns_empty_list_for_dummy3_track(self):
        result = self.core.library.lookup(uris=["dummy3:a"])

        assert result == {
            "dummy3:a": [],
        }
        assert not self.library1.lookup.called
        assert not self.library2.lookup.called

    def test_lookup_ignores_tracks_without_uri_set(self):
        track1 = Track(uri="dummy1:a", name="abc")
        track2 = Track()

        self.library1.lookup().get.return_value = [track1, track2]

        result = self.core.library.lookup(uris=["dummy1:a"])
        assert result == {
            "dummy1:a": [track1],
        }

    def test_refresh_with_uri_selects_dummy1_backend(self):
        self.core.library.refresh("dummy1:a")

        self.library1.refresh.assert_called_once_with("dummy1:a")
        assert not self.library2.refresh.called

    def test_refresh_with_uri_selects_dummy2_backend(self):
        self.core.library.refresh("dummy2:a")

        assert not self.library1.refresh.called
        self.library2.refresh.assert_called_once_with("dummy2:a")

    def test_refresh_with_uri_fails_silently_for_dummy3_uri(self):
        self.core.library.refresh("dummy3:a")

        assert not self.library1.refresh.called
        assert not self.library2.refresh.called

    def test_refresh_without_uri_calls_all_backends(self):
        self.core.library.refresh()

        self.library1.refresh.return_value.get.assert_called_once_with()
        self.library2.refresh.return_value.get.assert_called_once_with()

    def test_search_combines_results_from_all_backends(self):
        track1 = Track(uri="dummy1:a")
        track2 = Track(uri="dummy2:a")
        result1 = SearchResult(tracks=[track1])
        result2 = SearchResult(tracks=[track2])

        self.library1.search.return_value.get.return_value = result1
        self.library2.search.return_value.get.return_value = result2

        result = self.core.library.search({"any": ["a"]})

        assert result1 in result
        assert result2 in result
        self.library1.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )
        self.library2.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )

    def test_search_with_uris_selects_dummy1_backend(self):
        self.core.library.search(
            query={"any": ["a"]}, uris=["dummy1:", "dummy1:foo", "dummy3:"]
        )

        self.library1.search.assert_called_once_with(
            query={"any": ["a"]}, uris=["dummy1:", "dummy1:foo"], exact=False
        )
        assert not self.library2.search.called

    def test_search_with_uris_selects_both_backends(self):
        self.core.library.search(
            query={"any": ["a"]}, uris=["dummy1:", "dummy1:foo", "dummy2:"]
        )

        self.library1.search.assert_called_once_with(
            query={"any": ["a"]}, uris=["dummy1:", "dummy1:foo"], exact=False
        )
        self.library2.search.assert_called_once_with(
            query={"any": ["a"]}, uris=["dummy2:"], exact=False
        )

    def test_search_filters_out_none(self):
        track1 = Track(uri="dummy1:a")
        result1 = SearchResult(tracks=[track1])

        self.library1.search.return_value.get.return_value = result1
        self.library2.search.return_value.get.return_value = None

        result = self.core.library.search({"any": ["a"]})

        assert result1 in result
        assert None not in result
        self.library1.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )
        self.library2.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )

    def test_search_accepts_query_dict_instead_of_kwargs(self):
        track1 = Track(uri="dummy1:a")
        track2 = Track(uri="dummy2:a")
        result1 = SearchResult(tracks=[track1])
        result2 = SearchResult(tracks=[track2])

        self.library1.search.return_value.get.return_value = result1
        self.library2.search.return_value.get.return_value = result2

        result = self.core.library.search({"any": ["a"]})

        assert result1 in result
        assert result2 in result
        self.library1.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )
        self.library2.search.assert_called_once_with(
            query={"any": ["a"]}, uris=None, exact=False
        )

    def test_search_normalises_bad_queries(self):
        self.core.library.search({"any": "foobar"})
        self.library1.search.assert_called_once_with(
            query={"any": ["foobar"]}, uris=None, exact=False
        )


class GetDistinctTest(BaseCoreLibraryTest):
    def test_with_query(self):
        self.library1.get_distinct.return_value.get.return_value = {}
        self.library2.get_distinct.return_value.get.return_value = {}

        result = self.core.library.get_distinct("album", {"any": ["a"]})

        self.library1.get_distinct.assert_called_with("album", {"any": ["a"]})
        self.library2.get_distinct.assert_called_with("album", {"any": ["a"]})
        assert result == set()

    def test_combines_results_from_all_backends(self):
        result1 = "foo"
        result2 = "bar"
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {result2}

        result = self.core.library.get_distinct("artist")

        assert result1 in result
        assert result2 in result

    def test_combined_results_are_unique(self):
        result1 = "foo"
        result2 = "foo"
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {result2}

        result = self.core.library.get_distinct("artist")

        assert result1 in result
        assert len(result) == 1

    @mock.patch.object(core.library.validation, "check_choice")
    def test_checks_field_is_valid(self, check_choice_mock):
        self.core.library.get_distinct("artist")
        check_choice_mock.assert_called_with(
            "artist", validation.DISTINCT_FIELDS.keys()
        )

    def test_any_field_raises_valueerror(self):
        with self.assertRaises(ValueError):
            self.core.library.get_distinct("any")

    def test_unknown_tag_in_query_raises_valueerror(self):
        with self.assertRaises(ValueError):
            self.core.library.get_distinct("album", {"track": ["a"]})

    def test_track_name_field_maps_to_track_for_backwards_compatibility(self):
        result1 = "foo"
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {}

        result = self.core.library.get_distinct("track_name")

        self.library1.get_distinct.assert_called_with("track", None)
        self.library2.get_distinct.assert_called_with("track", None)
        assert result == {result1}

    @mock.patch("mopidy.internal.deprecation.warn")
    def test_track_field_is_deprecated(self, deprecate_warn_mock):
        result1 = "bar"
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {}

        assert self.core.library.get_distinct("track") == {result1}
        deprecate_warn_mock.assert_called_once_with(
            "core.library.get_distinct:field_arg:track", pending=mock.ANY
        )

    @mock.patch("mopidy.core.library.logger")
    def test_validate_integer_results(self, logger_mock):
        result1 = 99
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {}

        assert self.core.library.get_distinct("track_no") == {result1}
        assert self.core.library.get_distinct("disc_no") == {result1}
        logger_mock.error.assert_not_called()

        assert self.core.library.get_distinct("uri") == set()
        logger_mock.error.assert_called_once()

    @mock.patch("mopidy.core.library.logger")
    def test_wrong_result_types_removed_and_logged(self, logger_mock):
        result1 = 99
        self.library1.get_distinct.return_value.get.return_value = {result1}
        self.library2.get_distinct.return_value.get.return_value = {}

        assert self.core.library.get_distinct("track_no") == {result1}
        logger_mock.error.assert_not_called()

        result2 = "foo"
        self.library2.get_distinct.return_value.get.return_value = {result2}

        assert self.core.library.get_distinct("track_no") == {result1}
        logger_mock.error.assert_called_once()

        logger_mock.error.reset_mock()
        assert self.core.library.get_distinct("uri") == {result2}
        logger_mock.error.assert_called()


class LegacyFindExactToSearchLibraryTest(unittest.TestCase):
    def setUp(self):  # noqa: N802
        self.backend = mock.Mock()
        self.backend.actor_ref.actor_class.__name__ = "DummyBackend"
        self.backend.uri_schemes.get.return_value = ["dummy"]
        self.backend.library = mock.Mock(spec=backend.LibraryProvider)
        self.core = core.Core(mixer=None, backends=[self.backend])

    def test_core_search_call_backend_search_with_exact(self):
        self.core.library.search(query={"any": ["a"]})
        self.backend.library.search.assert_called_once_with(
            query=dict(any=["a"]), uris=None, exact=False
        )

    def test_core_search_with_exact_call_backend_search_with_exact(self):
        self.core.library.search(query={"any": ["a"]}, exact=True)
        self.backend.library.search.assert_called_once_with(
            query=dict(any=["a"]), uris=None, exact=True
        )

    def test_core_search_with_handles_legacy_backend(self):
        self.backend.library.search.return_value.get.side_effect = TypeError
        self.core.library.search(query={"any": ["a"]}, exact=True)
        # We are just testing that this doesn't fail.


class MockBackendCoreLibraryBase(unittest.TestCase):
    def setUp(self):  # noqa: N802
        dummy_root = Ref.directory(uri="dummy:directory", name="dummy")

        self.library = mock.Mock(spec=backend.LibraryProvider)
        self.library.root_directory.get.return_value = dummy_root

        self.backend = mock.Mock()
        self.backend.actor_ref.actor_class.__name__ = "DummyBackend"
        self.backend.uri_schemes.get.return_value = ["dummy"]
        self.backend.library = self.library

        self.core = core.Core(mixer=None, backends=[self.backend])


@mock.patch("mopidy.core.library.logger")
class BrowseBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception_for_root(self, logger):
        # Might happen if root_directory is a property for some weird reason.
        self.library.root_directory.get.side_effect = Exception
        assert [] == self.core.library.browse(None)
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_returns_none_for_root(self, logger):
        self.library.root_directory.get.return_value = None
        assert [] == self.core.library.browse(None)
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_wrong_type_for_root(self, logger):
        self.library.root_directory.get.return_value = 123
        assert [] == self.core.library.browse(None)
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_raises_exception_for_browse(self, logger):
        self.library.browse.return_value.get.side_effect = Exception
        assert [] == self.core.library.browse("dummy:directory")
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_returns_wrong_type_for_browse(self, logger):
        self.library.browse.return_value.get.return_value = [123]
        assert [] == self.core.library.browse("dummy:directory")
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)


@mock.patch("mopidy.core.library.logger")
class GetDistinctBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception(self, logger):
        self.library.get_distinct.return_value.get.side_effect = Exception
        assert set() == self.core.library.get_distinct("artist")
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_returns_none(self, logger):
        self.library.get_distinct.return_value.get.return_value = None
        assert set() == self.core.library.get_distinct("artist")
        assert not logger.error.called

    def test_backend_returns_wrong_type(self, logger):
        self.library.get_distinct.return_value.get.return_value = "abc"
        assert set() == self.core.library.get_distinct("artist")
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_iterable_containing_wrong_types(self, logger):
        self.library.get_distinct.return_value.get.return_value = [1, 2, 3]
        assert set() == self.core.library.get_distinct("artist")
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)


@mock.patch("mopidy.core.library.logger")
class GetImagesBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.side_effect = Exception
        assert {uri: ()} == self.core.library.get_images([uri])
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_returns_none(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.return_value = None
        assert {uri: ()} == self.core.library.get_images([uri])
        assert not logger.error.called

    def test_backend_returns_wrong_type(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.return_value = "abc"
        assert {uri: ()} == self.core.library.get_images([uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_mapping_containing_wrong_types(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.return_value = {uri: "abc"}
        assert {uri: ()} == self.core.library.get_images([uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_mapping_containing_none(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.return_value = {uri: None}
        assert {uri: ()} == self.core.library.get_images([uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_unknown_uri(self, logger):
        uri = "dummy:/1"
        self.library.get_images.return_value.get.return_value = {"foo": []}
        assert {uri: ()} == self.core.library.get_images([uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)


@mock.patch("mopidy.core.library.logger")
class LookupByUrisBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception(self, logger):
        uri = "dummy:/1"
        self.library.lookup.return_value.get.side_effect = Exception
        assert {uri: []} == self.core.library.lookup(uris=[uri])
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_returns_none(self, logger):
        uri = "dummy:/1"
        self.library.lookup.return_value.get.return_value = None
        assert {uri: []} == self.core.library.lookup(uris=[uri])
        assert not logger.error.called

    def test_backend_returns_wrong_type(self, logger):
        uri = "dummy:/1"
        self.library.lookup.return_value.get.return_value = "abc"
        assert {uri: []} == self.core.library.lookup(uris=[uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)

    def test_backend_returns_iterable_containing_wrong_types(self, logger):
        uri = "dummy:/1"
        self.library.lookup.return_value.get.return_value = [123]
        assert {uri: []} == self.core.library.lookup(uris=[uri])
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)


@mock.patch("mopidy.core.library.logger")
class RefreshBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception(self, logger):
        self.library.refresh.return_value.get.side_effect = Exception
        self.core.library.refresh()
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_raises_exception_with_uri(self, logger):
        self.library.refresh.return_value.get.side_effect = Exception
        self.core.library.refresh("dummy:/1")
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")


@mock.patch("mopidy.core.library.logger")
class SearchBadBackendTest(MockBackendCoreLibraryBase):
    def test_backend_raises_exception(self, logger):
        self.library.search.return_value.get.side_effect = Exception
        assert [] == self.core.library.search(query={"any": ["foo"]})
        logger.exception.assert_called_with(mock.ANY, "DummyBackend")

    def test_backend_raises_lookuperror(self, logger):
        # TODO: is this behavior desired? Do we need to continue handling
        # LookupError case specially.
        self.library.search.return_value.get.side_effect = LookupError
        with self.assertRaises(LookupError):
            self.core.library.search(query={"any": ["foo"]})

    def test_backend_returns_none(self, logger):
        self.library.search.return_value.get.return_value = None
        assert [] == self.core.library.search(query={"any": ["foo"]})
        assert not logger.error.called

    def test_backend_returns_wrong_type(self, logger):
        self.library.search.return_value.get.return_value = "abc"
        assert [] == self.core.library.search(query={"any": ["foo"]})
        logger.error.assert_called_with(mock.ANY, "DummyBackend", mock.ANY)