Backup_Repos/beets
1
0
Fork 0
mirror of https://github.com/beetbox/beets.git synced 2025-12-10 10:32:34 +01:00
Code Issues Projects Releases Wiki Activity

Replace capture_stdout in test_config_command

Browse source 
I didn't see a good reason not to use `run_with_command` here
This commit is contained in:
Johnny Robeson 2016-09-18 03:29:04 -04:00
parent ddaa94003b
commit ec4eb8368e
1 changed files with 9 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

19
test/test_config_command.py
View file

@ -12,7 +12,7 @@ from beets import ui
from beets import config
from test._common import unittest
from test.helper import TestHelper, capture_stdout
from test.helper import TestHelper
from beets.library import Library
import six
@ -44,9 +44,8 @@ class ConfigCommandTest(unittest.TestCase, TestHelper):
rmtree(self.temp_dir)
def _run_with_yaml_output(self, *args):
with capture_stdout() as output:
self.run_command(*args)
return yaml.load(output.getvalue())
output = self.run_with_output(*args)
return yaml.load(output)
def test_show_user_config(self):
output = self._run_with_yaml_output('config', '-c')
@ -83,16 +82,16 @@ class ConfigCommandTest(unittest.TestCase, TestHelper):
self.assertEqual(output['import']['timid'], False)
def test_config_paths(self):
with capture_stdout() as output:
self.run_command('config', '-p')
paths = output.getvalue().split('\n')
output = self.run_with_output('config', '-p')
paths = output.split('\n')
self.assertEqual(len(paths), 2)
self.assertEqual(paths[0], self.config_path)
def test_config_paths_with_cli(self):
with capture_stdout() as output:
self.run_command('--config', self.cli_config_path, 'config', '-p')
paths = output.getvalue().split('\n')
output = self.run_with_output('--config', self.cli_config_path,
'config', '-p')
paths = output.split('\n')
self.assertEqual(len(paths), 3)
self.assertEqual(paths[0], self.cli_config_path)