mirror of
https://github.com/beetbox/beets.git
synced 2026-01-01 13:33:02 +01:00
Merge pull request #1842 from pkess/extend_test_for_modify
Extend test for modify
This commit is contained in:
commit
6ee7c4f7c7
1 changed files with 26 additions and 2 deletions
|
|
@ -158,10 +158,13 @@ class ModifyTest(unittest.TestCase, TestHelper):
|
|||
def tearDown(self):
|
||||
self.teardown_beets()
|
||||
|
||||
def modify(self, *args):
|
||||
with control_stdin('y'):
|
||||
def modify_inp(self, inp, *args):
|
||||
with control_stdin(inp):
|
||||
ui._raw_main(['modify'] + list(args), self.lib)
|
||||
|
||||
def modify(self, *args):
|
||||
self.modify_inp('y', *args)
|
||||
|
||||
# Item tests
|
||||
|
||||
def test_modify_item(self):
|
||||
|
|
@ -169,6 +172,20 @@ class ModifyTest(unittest.TestCase, TestHelper):
|
|||
item = self.lib.items().get()
|
||||
self.assertEqual(item.title, 'newTitle')
|
||||
|
||||
def test_modify_item_abort(self):
|
||||
item = self.lib.items().get()
|
||||
title = item.title
|
||||
self.modify_inp('n', "title=newTitle")
|
||||
item = self.lib.items().get()
|
||||
self.assertEqual(item.title, title)
|
||||
|
||||
def test_modify_item_no_change(self):
|
||||
title = "Tracktitle"
|
||||
item = self.add_item_fixture(title=title)
|
||||
self.modify_inp('y', "title", "title={0}".format(title))
|
||||
item = self.lib.items(title).get()
|
||||
self.assertEqual(item.title, title)
|
||||
|
||||
def test_modify_write_tags(self):
|
||||
self.modify("title=newTitle")
|
||||
item = self.lib.items().get()
|
||||
|
|
@ -191,6 +208,13 @@ class ModifyTest(unittest.TestCase, TestHelper):
|
|||
item = self.lib.items().get()
|
||||
self.assertNotIn(b'newTitle', item.path)
|
||||
|
||||
def test_no_write_no_move(self):
|
||||
self.modify("--nomove", "--nowrite", "title=newTitle")
|
||||
item = self.lib.items().get()
|
||||
item.read()
|
||||
self.assertNotIn(b'newTitle', item.path)
|
||||
self.assertNotEqual(item.title, 'newTitle')
|
||||
|
||||
def test_update_mtime(self):
|
||||
item = self.item
|
||||
old_mtime = item.mtime
|
||||
|
|
|
|||
Loading…
Reference in a new issue