Skip to content

Commit

Permalink
Update test_get_musicbrainz_server() test case
Browse files Browse the repository at this point in the history
Signed-off-by: JoeLametta <JoeLametta@users.noreply.github.com>
  • Loading branch information
JoeLametta committed Jan 16, 2020
1 parent 0b41a6e commit 26df7f7
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions whipper/test/test_common_config.py
Original file line number Diff line number Diff line change
Expand Up @@ -69,25 +69,25 @@ def testDefeatsCache(self):

def test_get_musicbrainz_server(self):
self.assertEqual(self._config.get_musicbrainz_server(),
'musicbrainz.org',
{'scheme':'https', 'netloc':'musicbrainz.org'},
msg='Default value is correct')

self._config._parser.add_section('musicbrainz')

self._config._parser.set('musicbrainz', 'server',
'192.168.2.141:5000')
'http://192.168.2.141:5000')
self._config.write()
self.assertEqual(self._config.get_musicbrainz_server(),
'192.168.2.141:5000',
{'scheme':'http', 'netloc':'192.168.2.141:5000'},
msg='Correctly returns user-set value')

self._config._parser.set('musicbrainz', 'server',
'192.168.2.141:5000/hello/world')
# Test for unsupported scheme
self._config._parser.set('musicbrainz', 'server', 'ftp://example.com')
self._config.write()
self.assertRaises(KeyError, self._config.get_musicbrainz_server)

self._config._parser.set('musicbrainz', 'server',
'http://192.168.2.141:5000')
# Test for absent scheme
self._config._parser.set('musicbrainz', 'server', 'example.com')
self._config.write()
self.assertRaises(KeyError, self._config.get_musicbrainz_server)

Expand Down

0 comments on commit 26df7f7

Please sign in to comment.