config: Add allow_playlists config

Fixes #25

README.rst

@@ -114,6 +114,9 @@ The following configuration values are available:
 - ``spotify/allow_network``: Whether to allow network access or not. Defaults
   to ``true``.
 
+- ``spotify/allow_playlists``: Whether or not playlists should be exposed.
+  Defaults to ``true``.
+
 
 Project resources
 =================
@@ -144,6 +147,10 @@ v2.0.0 (UNRELEASED)
   Mopidy-Spotify to stay offline. This is mostly useful for testing during
   development.
 
+- Add ``spotify/allow_playlists`` config which can be used to disable all
+  access to playlists on the Spotify account. Useful where Mopidy is shared by
+  multiple users. (Fixes: #25)
+
 **Lookup**
 
 - Adding an artist by URI will now first find all albums by the artist and

mopidy_spotify/__init__.py

@@ -35,9 +35,7 @@ def get_config_schema(self):
         schema['toplist_countries'] = config.List(optional=True)
 
         schema['allow_network'] = config.Boolean()
-
-        # TODO Add more configrations:
-        # - show_playlists = true/false, requested in #25
+        schema['allow_playlists'] = config.Boolean()
 
         return schema
 

mopidy_spotify/backend.py

@@ -59,7 +59,11 @@ def __init__(self, config, audio):
         self.library = library.SpotifyLibraryProvider(backend=self)
         self.playback = playback.SpotifyPlaybackProvider(
             audio=audio, backend=self)
-        self.playlists = playlists.SpotifyPlaylistsProvider(backend=self)
+
+        if config['spotify']['allow_playlists']:
+            self.playlists = playlists.SpotifyPlaylistsProvider(backend=self)
+        else:
+            self.playlists = None
 
         self.uri_schemes = ['spotify']
 

mopidy_spotify/ext.conf

@@ -9,3 +9,4 @@ cache_dir = $XDG_CACHE_DIR/mopidy/spotify
 settings_dir = $XDG_CONFIG_DIR/mopidy/spotify
 toplist_countries = AD,AR,AU,AT,BE,CO,CY,DK,EE,FI,FR,DE,GR,HK,IS,IE,IT,LV,LI,LT,LU,MY,MX,MC,NL,NZ,NO,PT,ES,SG,SE,CH,TW,TR,GB,US
 allow_network = true
+allow_playlists = true

tests/conftest.py

@@ -21,6 +21,7 @@ def config():
             'cache_dir': '/my/cache/dir',
             'settings_dir': '/my/settings/dir',
             'allow_network': True,
+            'allow_playlists': True,
         }
     }
 

tests/test_backend.py

@@ -92,6 +92,14 @@ def test_init_sets_up_the_providers(spotify_mock, config):
     assert isinstance(backend.playlists, playlists.SpotifyPlaylistsProvider)
 
 
+def test_init_disables_playlists_provider_if_not_allowed(spotify_mock, config):
+    config['spotify']['allow_playlists'] = False
+
+    backend = get_backend(config)
+
+    assert backend.playlists is None
+
+
 def test_on_start_starts_the_pyspotify_event_loop(spotify_mock, config):
     backend = get_backend(config)
     backend.on_start()

tests/test_extension.py

@@ -26,6 +26,7 @@ def test_get_config_schema():
     assert 'settings_dir' in schema
     assert 'toplist_countries' in schema
     assert 'allow_network' in schema
+    assert 'allow_playlists' in schema
 
 
 def test_setup():