Changed wording export/restore to save/load
This commit is contained in:
parent
00c47117d5
commit
b55996da6a
@ -25,7 +25,7 @@ class HistoryTrack(ValidatedImmutableObject):
|
|||||||
class HistoryState(ValidatedImmutableObject):
|
class HistoryState(ValidatedImmutableObject):
|
||||||
"""
|
"""
|
||||||
State of the history controller.
|
State of the history controller.
|
||||||
Internally used for import/export of current state.
|
Internally used for save/load state.
|
||||||
|
|
||||||
:param history: the track history
|
:param history: the track history
|
||||||
:type history: list of :class:`HistoryTrack`
|
:type history: list of :class:`HistoryTrack`
|
||||||
@ -38,7 +38,7 @@ class HistoryState(ValidatedImmutableObject):
|
|||||||
class MixerState(ValidatedImmutableObject):
|
class MixerState(ValidatedImmutableObject):
|
||||||
"""
|
"""
|
||||||
State of the mixer controller.
|
State of the mixer controller.
|
||||||
Internally used for import/export of current state.
|
Internally used for save/load state.
|
||||||
|
|
||||||
:param volume: the volume
|
:param volume: the volume
|
||||||
:type volume: int
|
:type volume: int
|
||||||
@ -56,7 +56,7 @@ class MixerState(ValidatedImmutableObject):
|
|||||||
class PlaybackState(ValidatedImmutableObject):
|
class PlaybackState(ValidatedImmutableObject):
|
||||||
"""
|
"""
|
||||||
State of the playback controller.
|
State of the playback controller.
|
||||||
Internally used for import/export of current state.
|
Internally used for save/load state.
|
||||||
|
|
||||||
:param tlid: current track tlid
|
:param tlid: current track tlid
|
||||||
:type tlid: int
|
:type tlid: int
|
||||||
@ -80,7 +80,7 @@ class TracklistState(ValidatedImmutableObject):
|
|||||||
|
|
||||||
"""
|
"""
|
||||||
State of the tracklist controller.
|
State of the tracklist controller.
|
||||||
Internally used for import/export of current state.
|
Internally used for save/load state.
|
||||||
|
|
||||||
:param repeat: the repeat mode
|
:param repeat: the repeat mode
|
||||||
:type repeat: bool
|
:type repeat: bool
|
||||||
@ -119,7 +119,7 @@ class CoreState(ValidatedImmutableObject):
|
|||||||
|
|
||||||
"""
|
"""
|
||||||
State of all Core controller.
|
State of all Core controller.
|
||||||
Internally used for import/export of current state.
|
Internally used for save/load state.
|
||||||
|
|
||||||
:param history: State of the history controller
|
:param history: State of the history controller
|
||||||
:type history: :class:`HistorState`
|
:type history: :class:`HistorState`
|
||||||
|
|||||||
@ -52,7 +52,7 @@ class CoreActorTest(unittest.TestCase):
|
|||||||
self.assertEqual(self.core.version, versioning.get_version())
|
self.assertEqual(self.core.version, versioning.get_version())
|
||||||
|
|
||||||
|
|
||||||
class CoreActorExportRestoreTest(unittest.TestCase):
|
class CoreActorSaveLoadStateTest(unittest.TestCase):
|
||||||
|
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
self.temp_dir = tempfile.mkdtemp()
|
self.temp_dir = tempfile.mkdtemp()
|
||||||
|
|||||||
@ -49,7 +49,7 @@ class PlaybackHistoryTest(unittest.TestCase):
|
|||||||
self.assertIn(artist.name, ref.name)
|
self.assertIn(artist.name, ref.name)
|
||||||
|
|
||||||
|
|
||||||
class CoreHistoryExportRestoreTest(unittest.TestCase):
|
class CoreHistorySaveLoadStateTest(unittest.TestCase):
|
||||||
|
|
||||||
def setUp(self): # noqa: N802
|
def setUp(self): # noqa: N802
|
||||||
self.tracks = [
|
self.tracks = [
|
||||||
@ -64,7 +64,7 @@ class CoreHistoryExportRestoreTest(unittest.TestCase):
|
|||||||
|
|
||||||
self.history = HistoryController()
|
self.history = HistoryController()
|
||||||
|
|
||||||
def test_export(self):
|
def test_save(self):
|
||||||
self.history._add_track(self.tracks[2])
|
self.history._add_track(self.tracks[2])
|
||||||
self.history._add_track(self.tracks[1])
|
self.history._add_track(self.tracks[1])
|
||||||
|
|
||||||
@ -75,7 +75,7 @@ class CoreHistoryExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual(value.history[0].track, self.refs[1])
|
self.assertEqual(value.history[0].track, self.refs[1])
|
||||||
self.assertEqual(value.history[1].track, self.refs[2])
|
self.assertEqual(value.history[1].track, self.refs[2])
|
||||||
|
|
||||||
def test_import(self):
|
def test_load(self):
|
||||||
state = HistoryState(history=[
|
state = HistoryState(history=[
|
||||||
HistoryTrack(timestamp=34, track=self.refs[0]),
|
HistoryTrack(timestamp=34, track=self.refs[0]),
|
||||||
HistoryTrack(timestamp=45, track=self.refs[2]),
|
HistoryTrack(timestamp=45, track=self.refs[2]),
|
||||||
@ -98,9 +98,9 @@ class CoreHistoryExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual(hist[2], (45, self.refs[2]))
|
self.assertEqual(hist[2], (45, self.refs[2]))
|
||||||
self.assertEqual(hist[3], (56, self.refs[1]))
|
self.assertEqual(hist[3], (56, self.refs[1]))
|
||||||
|
|
||||||
def test_import_invalid_type(self):
|
def test_load_invalid_type(self):
|
||||||
with self.assertRaises(TypeError):
|
with self.assertRaises(TypeError):
|
||||||
self.history._load_state(11, None)
|
self.history._load_state(11, None)
|
||||||
|
|
||||||
def test_import_none(self):
|
def test_load_none(self):
|
||||||
self.history._load_state(None, None)
|
self.history._load_state(None, None)
|
||||||
|
|||||||
@ -157,13 +157,13 @@ class SetMuteBadBackendTest(MockBackendCoreMixerBase):
|
|||||||
self.assertFalse(self.core.mixer.set_mute(True))
|
self.assertFalse(self.core.mixer.set_mute(True))
|
||||||
|
|
||||||
|
|
||||||
class CoreMixerExportRestoreTest(unittest.TestCase):
|
class CoreMixerSaveLoadStateTest(unittest.TestCase):
|
||||||
|
|
||||||
def setUp(self): # noqa: N802
|
def setUp(self): # noqa: N802
|
||||||
self.mixer = dummy_mixer.create_proxy()
|
self.mixer = dummy_mixer.create_proxy()
|
||||||
self.core = core.Core(mixer=self.mixer, backends=[])
|
self.core = core.Core(mixer=self.mixer, backends=[])
|
||||||
|
|
||||||
def test_export_mute(self):
|
def test_save_mute(self):
|
||||||
volume = 32
|
volume = 32
|
||||||
mute = False
|
mute = False
|
||||||
target = MixerState(volume=volume, mute=mute)
|
target = MixerState(volume=volume, mute=mute)
|
||||||
@ -172,7 +172,7 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
value = self.core.mixer._save_state()
|
value = self.core.mixer._save_state()
|
||||||
self.assertEqual(target, value)
|
self.assertEqual(target, value)
|
||||||
|
|
||||||
def test_export_unmute(self):
|
def test_save_unmute(self):
|
||||||
volume = 33
|
volume = 33
|
||||||
mute = True
|
mute = True
|
||||||
target = MixerState(volume=volume, mute=mute)
|
target = MixerState(volume=volume, mute=mute)
|
||||||
@ -181,7 +181,7 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
value = self.core.mixer._save_state()
|
value = self.core.mixer._save_state()
|
||||||
self.assertEqual(target, value)
|
self.assertEqual(target, value)
|
||||||
|
|
||||||
def test_import(self):
|
def test_load(self):
|
||||||
self.core.mixer.set_volume(11)
|
self.core.mixer.set_volume(11)
|
||||||
volume = 45
|
volume = 45
|
||||||
target = MixerState(volume=volume)
|
target = MixerState(volume=volume)
|
||||||
@ -189,7 +189,7 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
self.core.mixer._load_state(target, coverage)
|
self.core.mixer._load_state(target, coverage)
|
||||||
self.assertEqual(volume, self.core.mixer.get_volume())
|
self.assertEqual(volume, self.core.mixer.get_volume())
|
||||||
|
|
||||||
def test_import_not_covered(self):
|
def test_load_not_covered(self):
|
||||||
self.core.mixer.set_volume(21)
|
self.core.mixer.set_volume(21)
|
||||||
self.core.mixer.set_mute(True)
|
self.core.mixer.set_mute(True)
|
||||||
target = MixerState(volume=56, mute=False)
|
target = MixerState(volume=56, mute=False)
|
||||||
@ -198,7 +198,7 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual(21, self.core.mixer.get_volume())
|
self.assertEqual(21, self.core.mixer.get_volume())
|
||||||
self.assertEqual(True, self.core.mixer.get_mute())
|
self.assertEqual(True, self.core.mixer.get_mute())
|
||||||
|
|
||||||
def test_import_mute_on(self):
|
def test_load_mute_on(self):
|
||||||
self.core.mixer.set_mute(False)
|
self.core.mixer.set_mute(False)
|
||||||
self.assertEqual(False, self.core.mixer.get_mute())
|
self.assertEqual(False, self.core.mixer.get_mute())
|
||||||
target = MixerState(mute=True)
|
target = MixerState(mute=True)
|
||||||
@ -206,7 +206,7 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
self.core.mixer._load_state(target, coverage)
|
self.core.mixer._load_state(target, coverage)
|
||||||
self.assertEqual(True, self.core.mixer.get_mute())
|
self.assertEqual(True, self.core.mixer.get_mute())
|
||||||
|
|
||||||
def test_import_mute_off(self):
|
def test_load_mute_off(self):
|
||||||
self.core.mixer.set_mute(True)
|
self.core.mixer.set_mute(True)
|
||||||
self.assertEqual(True, self.core.mixer.get_mute())
|
self.assertEqual(True, self.core.mixer.get_mute())
|
||||||
target = MixerState(mute=False)
|
target = MixerState(mute=False)
|
||||||
@ -214,9 +214,9 @@ class CoreMixerExportRestoreTest(unittest.TestCase):
|
|||||||
self.core.mixer._load_state(target, coverage)
|
self.core.mixer._load_state(target, coverage)
|
||||||
self.assertEqual(False, self.core.mixer.get_mute())
|
self.assertEqual(False, self.core.mixer.get_mute())
|
||||||
|
|
||||||
def test_import_invalid_type(self):
|
def test_load_invalid_type(self):
|
||||||
with self.assertRaises(TypeError):
|
with self.assertRaises(TypeError):
|
||||||
self.core.mixer._load_state(11, None)
|
self.core.mixer._load_state(11, None)
|
||||||
|
|
||||||
def test_import_none(self):
|
def test_load_none(self):
|
||||||
self.core.mixer._load_state(None, None)
|
self.core.mixer._load_state(None, None)
|
||||||
|
|||||||
@ -1132,9 +1132,9 @@ class TestBug1177Regression(unittest.TestCase):
|
|||||||
b.playback.change_track.assert_called_once_with(track2)
|
b.playback.change_track.assert_called_once_with(track2)
|
||||||
|
|
||||||
|
|
||||||
class TesetCorePlaybackExportRestore(BaseTest):
|
class TestCorePlaybackSaveLoadState(BaseTest):
|
||||||
|
|
||||||
def test_export(self):
|
def test_save(self):
|
||||||
tl_tracks = self.core.tracklist.get_tl_tracks()
|
tl_tracks = self.core.tracklist.get_tl_tracks()
|
||||||
|
|
||||||
self.core.playback.play(tl_tracks[1])
|
self.core.playback.play(tl_tracks[1])
|
||||||
@ -1146,7 +1146,7 @@ class TesetCorePlaybackExportRestore(BaseTest):
|
|||||||
|
|
||||||
self.assertEqual(state, value)
|
self.assertEqual(state, value)
|
||||||
|
|
||||||
def test_import(self):
|
def test_load(self):
|
||||||
tl_tracks = self.core.tracklist.get_tl_tracks()
|
tl_tracks = self.core.tracklist.get_tl_tracks()
|
||||||
|
|
||||||
self.core.playback.stop()
|
self.core.playback.stop()
|
||||||
@ -1163,7 +1163,7 @@ class TesetCorePlaybackExportRestore(BaseTest):
|
|||||||
self.assertEqual(tl_tracks[2],
|
self.assertEqual(tl_tracks[2],
|
||||||
self.core.playback.get_current_tl_track())
|
self.core.playback.get_current_tl_track())
|
||||||
|
|
||||||
def test_import_not_covered(self):
|
def test_load_not_covered(self):
|
||||||
tl_tracks = self.core.tracklist.get_tl_tracks()
|
tl_tracks = self.core.tracklist.get_tl_tracks()
|
||||||
|
|
||||||
self.core.playback.stop()
|
self.core.playback.stop()
|
||||||
@ -1180,11 +1180,11 @@ class TesetCorePlaybackExportRestore(BaseTest):
|
|||||||
self.assertEqual(None,
|
self.assertEqual(None,
|
||||||
self.core.playback.get_current_tl_track())
|
self.core.playback.get_current_tl_track())
|
||||||
|
|
||||||
def test_import_invalid_type(self):
|
def test_load_invalid_type(self):
|
||||||
with self.assertRaises(TypeError):
|
with self.assertRaises(TypeError):
|
||||||
self.core.playback._load_state(11, None)
|
self.core.playback._load_state(11, None)
|
||||||
|
|
||||||
def test_import_none(self):
|
def test_load_none(self):
|
||||||
self.core.playback._load_state(None, None)
|
self.core.playback._load_state(None, None)
|
||||||
|
|
||||||
|
|
||||||
|
|||||||
@ -180,7 +180,7 @@ class TracklistIndexTest(unittest.TestCase):
|
|||||||
self.assertEqual(2, self.core.tracklist.index())
|
self.assertEqual(2, self.core.tracklist.index())
|
||||||
|
|
||||||
|
|
||||||
class TracklistExportRestoreTest(unittest.TestCase):
|
class TracklistSaveLoadStateTest(unittest.TestCase):
|
||||||
|
|
||||||
def setUp(self): # noqa: N802
|
def setUp(self): # noqa: N802
|
||||||
config = {
|
config = {
|
||||||
@ -211,7 +211,7 @@ class TracklistExportRestoreTest(unittest.TestCase):
|
|||||||
|
|
||||||
self.core.playback = mock.Mock(spec=core.PlaybackController)
|
self.core.playback = mock.Mock(spec=core.PlaybackController)
|
||||||
|
|
||||||
def test_export(self):
|
def test_save(self):
|
||||||
tl_tracks = self.core.tracklist.add(uris=[
|
tl_tracks = self.core.tracklist.add(uris=[
|
||||||
t.uri for t in self.tracks])
|
t.uri for t in self.tracks])
|
||||||
consume = True
|
consume = True
|
||||||
@ -226,7 +226,7 @@ class TracklistExportRestoreTest(unittest.TestCase):
|
|||||||
value = self.core.tracklist._save_state()
|
value = self.core.tracklist._save_state()
|
||||||
self.assertEqual(target, value)
|
self.assertEqual(target, value)
|
||||||
|
|
||||||
def test_import(self):
|
def test_load(self):
|
||||||
old_version = self.core.tracklist.get_version()
|
old_version = self.core.tracklist.get_version()
|
||||||
target = TracklistState(consume=False,
|
target = TracklistState(consume=False,
|
||||||
repeat=True,
|
repeat=True,
|
||||||
@ -245,12 +245,12 @@ class TracklistExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual(self.tl_tracks, self.core.tracklist.get_tl_tracks())
|
self.assertEqual(self.tl_tracks, self.core.tracklist.get_tl_tracks())
|
||||||
self.assertGreater(self.core.tracklist.get_version(), old_version)
|
self.assertGreater(self.core.tracklist.get_version(), old_version)
|
||||||
|
|
||||||
# after import, adding more tracks must be possible
|
# after load, adding more tracks must be possible
|
||||||
self.core.tracklist.add(uris=[self.tracks[1].uri])
|
self.core.tracklist.add(uris=[self.tracks[1].uri])
|
||||||
self.assertEqual(13, self.core.tracklist._next_tlid)
|
self.assertEqual(13, self.core.tracklist._next_tlid)
|
||||||
self.assertEqual(5, self.core.tracklist.get_length())
|
self.assertEqual(5, self.core.tracklist.get_length())
|
||||||
|
|
||||||
def test_import_mode_only(self):
|
def test_load_mode_only(self):
|
||||||
old_version = self.core.tracklist.get_version()
|
old_version = self.core.tracklist.get_version()
|
||||||
target = TracklistState(consume=False,
|
target = TracklistState(consume=False,
|
||||||
repeat=True,
|
repeat=True,
|
||||||
@ -269,7 +269,7 @@ class TracklistExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual([], self.core.tracklist.get_tl_tracks())
|
self.assertEqual([], self.core.tracklist.get_tl_tracks())
|
||||||
self.assertEqual(self.core.tracklist.get_version(), old_version)
|
self.assertEqual(self.core.tracklist.get_version(), old_version)
|
||||||
|
|
||||||
def test_import_tracklist_only(self):
|
def test_load_tracklist_only(self):
|
||||||
old_version = self.core.tracklist.get_version()
|
old_version = self.core.tracklist.get_version()
|
||||||
target = TracklistState(consume=False,
|
target = TracklistState(consume=False,
|
||||||
repeat=True,
|
repeat=True,
|
||||||
@ -288,9 +288,9 @@ class TracklistExportRestoreTest(unittest.TestCase):
|
|||||||
self.assertEqual(self.tl_tracks, self.core.tracklist.get_tl_tracks())
|
self.assertEqual(self.tl_tracks, self.core.tracklist.get_tl_tracks())
|
||||||
self.assertGreater(self.core.tracklist.get_version(), old_version)
|
self.assertGreater(self.core.tracklist.get_version(), old_version)
|
||||||
|
|
||||||
def test_import_invalid_type(self):
|
def test_load_invalid_type(self):
|
||||||
with self.assertRaises(TypeError):
|
with self.assertRaises(TypeError):
|
||||||
self.core.tracklist._load_state(11, None)
|
self.core.tracklist._load_state(11, None)
|
||||||
|
|
||||||
def test_import_none(self):
|
def test_load_none(self):
|
||||||
self.core.tracklist._load_state(None, None)
|
self.core.tracklist._load_state(None, None)
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user