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

BPD tests: be more careful with paths

Browse source
This commit is contained in:
Carl Suster 2019-03-29 11:49:20 +11:00
parent 7c8bbd3011
commit 14d0cc3b85
1 changed files with 12 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

23
test/test_player.py
View file

@ -18,6 +18,9 @@
from __future__ import division, absolute_import, print_function
import unittest
from test.helper import TestHelper
import os
import sys
import multiprocessing as mp
import socket
@ -29,7 +32,6 @@ from contextlib import contextmanager
from beets.util import confit, py3_path
from beetsplug import bpd
from test.helper import TestHelper
# Mock GstPlayer so that the forked process doesn't attempt to import gi:
import mock
@ -225,6 +227,8 @@ class BPDTest(unittest.TestCase, TestHelper):
self.item1 = self.add_item(title='Track One Title',
album='Album Title', artist='Artist Name',
track=1)
self.item1_path = os.path.join(
'Artist Name', 'Album Title', '01 Track One Title.mp3')
self.item2 = self.add_item(title='Track Two Title',
album='Album Title', artist='Artist Name',
track=2)
@ -243,14 +247,15 @@ class BPDTest(unittest.TestCase, TestHelper):
"""
# Create a config file:
config = {
'pluginpath': [self.temp_dir.decode('utf-8')],
'pluginpath': [os.fsdecode(self.temp_dir)],
'plugins': 'bpd',
'bpd': {'host': host, 'port': port},
}
if password:
config['bpd']['password'] = password
config_file = tempfile.NamedTemporaryFile(
mode='wb', dir=self.temp_dir, suffix=b'.yaml', delete=False)
mode='wb', dir=os.fsdecode(self.temp_dir), suffix='.yaml',
delete=False)
config_file.write(
yaml.dump(config, Dumper=confit.Dumper, encoding='utf-8'))
config_file.close()
@ -304,7 +309,7 @@ class BPDTest(unittest.TestCase, TestHelper):
def test_cmd_play(self):
with self.run_bpd() as client:
responses = client.send_commands(
('add', 'Artist Name/Album Title/01 Track One Title.mp3'),
('add', self.item1_path),
('status',),
('play',),
('status',))
@ -322,15 +327,13 @@ class BPDTest(unittest.TestCase, TestHelper):
def test_cmd_add(self):
with self.run_bpd() as client:
response = client.send_command(
'add',
'Artist Name/Album Title/01 Track One Title.mp3')
response = client.send_command('add', self.item1_path)
self.assertTrue(response.ok)
def test_cmd_playlistinfo(self):
with self.run_bpd() as client:
responses = client.send_commands(
('add', 'Artist Name/Album Title/01 Track One Title.mp3'),
('add', self.item1_path),
('playlistinfo',),
('playlistinfo', '0'))
response = client.send_command('playlistinfo', '200')
@ -358,9 +361,7 @@ class BPDTest(unittest.TestCase, TestHelper):
def test_cmd_search(self):
with self.run_bpd() as client:
response = client.send_command('search', 'track', '1')
self.assertEqual(
'Artist Name/Album Title/01 Track One Title.mp3',
response.data['file'])
self.assertEqual(self.item1_path, response.data['file'])
def test_cmd_list_simple(self):
with self.run_bpd() as client: