From 74a65896687059ab057e8c5c11cb18512ac92422 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sun, 18 Nov 2012 16:11:18 +0100 Subject: [PATCH 1/4] tests: Fix typo in class name --- tests/models_test.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/models_test.py b/tests/models_test.py index d5d58ace..6c1520cb 100644 --- a/tests/models_test.py +++ b/tests/models_test.py @@ -7,7 +7,7 @@ from mopidy.models import Artist, Album, TlTrack, Track, Playlist from tests import unittest -class GenericCopyTets(unittest.TestCase): +class GenericCopyTest(unittest.TestCase): def compare(self, orig, other): self.assertEqual(orig, other) self.assertNotEqual(id(orig), id(other)) From f237736f877031a36b1126425b5e937749b1f0fa Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sun, 18 Nov 2012 16:27:04 +0100 Subject: [PATCH 2/4] models: Add '__type__' attribute to serialized models --- mopidy/models.py | 1 + tests/models_test.py | 25 +++++++++++++++---------- 2 files changed, 16 insertions(+), 10 deletions(-) diff --git a/mopidy/models.py b/mopidy/models.py index 17616f9d..6a2938ad 100644 --- a/mopidy/models.py +++ b/mopidy/models.py @@ -78,6 +78,7 @@ class ImmutableObject(object): def serialize(self): data = {} + data['__type__'] = self.__class__.__name__ for key in self.__dict__.keys(): public_key = key.lstrip('_') value = self.__dict__[key] diff --git a/tests/models_test.py b/tests/models_test.py index 6c1520cb..5fbd4dd3 100644 --- a/tests/models_test.py +++ b/tests/models_test.py @@ -83,7 +83,7 @@ class ArtistTest(unittest.TestCase): def test_serialize(self): self.assertDictEqual( - {'uri': 'uri', 'name': 'name'}, + {'__type__': 'Artist', 'uri': 'uri', 'name': 'name'}, Artist(uri='uri', name='name').serialize()) def test_eq_name(self): @@ -195,13 +195,14 @@ class AlbumTest(unittest.TestCase): def test_serialize_without_artists(self): self.assertDictEqual( - {'uri': 'uri', 'name': 'name'}, + {'__type__': 'Album', 'uri': 'uri', 'name': 'name'}, Album(uri='uri', name='name').serialize()) def test_serialize_with_artists(self): artist = Artist(name='foo') self.assertDictEqual( - {'uri': 'uri', 'name': 'name', 'artists': [artist.serialize()]}, + {'__type__': 'Album', 'uri': 'uri', 'name': 'name', 'artists': + [artist.serialize()]}, Album(uri='uri', name='name', artists=[artist]).serialize()) def test_eq_name(self): @@ -386,19 +387,21 @@ class TrackTest(unittest.TestCase): def test_serialize_without_artists(self): self.assertDictEqual( - {'uri': 'uri', 'name': 'name'}, + {'__type__': 'Track', 'uri': 'uri', 'name': 'name'}, Track(uri='uri', name='name').serialize()) def test_serialize_with_artists(self): artist = Artist(name='foo') self.assertDictEqual( - {'uri': 'uri', 'name': 'name', 'artists': [artist.serialize()]}, + {'__type__': 'Track', 'uri': 'uri', 'name': 'name', + 'artists': [artist.serialize()]}, Track(uri='uri', name='name', artists=[artist]).serialize()) def test_serialize_with_album(self): album = Album(name='foo') self.assertDictEqual( - {'uri': 'uri', 'name': 'name', 'album': album.serialize()}, + {'__type__': 'Track', 'uri': 'uri', 'name': 'name', + 'album': album.serialize()}, Track(uri='uri', name='name', album=album).serialize()) def test_eq_uri(self): @@ -590,9 +593,10 @@ class TlTrackTest(unittest.TestCase): repr(TlTrack(tlid=123, track=Track(uri='uri')))) def test_serialize(self): + track = Track(uri='uri', name='name') self.assertDictEqual( - {'tlid': 123, 'track': {'uri': 'uri', 'name': 'name'}}, - TlTrack(tlid=123, track=Track(uri='uri', name='name')).serialize()) + {'__type__': 'TlTrack', 'tlid': 123, 'track': track.serialize()}, + TlTrack(tlid=123, track=track).serialize()) def test_eq(self): tlid = 123 @@ -719,13 +723,14 @@ class PlaylistTest(unittest.TestCase): def test_serialize_without_tracks(self): self.assertDictEqual( - {'uri': 'uri', 'name': 'name'}, + {'__type__': 'Playlist', 'uri': 'uri', 'name': 'name'}, Playlist(uri='uri', name='name').serialize()) def test_serialize_with_tracks(self): track = Track(name='foo') self.assertDictEqual( - {'uri': 'uri', 'name': 'name', 'tracks': [track.serialize()]}, + {'__type__': 'Playlist', 'uri': 'uri', 'name': 'name', + 'tracks': [track.serialize()]}, Playlist(uri='uri', name='name', tracks=[track]).serialize()) def test_eq_name(self): From 68e4b207cb99be65df7d096691ce2f07b5aba8b3 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sun, 18 Nov 2012 16:50:17 +0100 Subject: [PATCH 3/4] models: Support automatic serialization to and deserialization from JSON --- mopidy/models.py | 45 ++++++++++++++++++++++++++++++++++++++++++++ tests/models_test.py | 37 +++++++++++++++++++++++++++++++++++- 2 files changed, 81 insertions(+), 1 deletion(-) diff --git a/mopidy/models.py b/mopidy/models.py index 6a2938ad..9eadb314 100644 --- a/mopidy/models.py +++ b/mopidy/models.py @@ -1,5 +1,7 @@ from __future__ import unicode_literals +import json + class ImmutableObject(object): """ @@ -91,6 +93,49 @@ class ImmutableObject(object): return data +class ModelJSONEncoder(json.JSONEncoder): + """ + Automatically serialize Mopidy models to JSON. + + Usage:: + + >>> import json + >>> json.dumps({'a_track': Track(name='name')}, cls=ModelJSONEncoder) + '{"a_track": {"__type__": "Track", "name": "name"}}' + + """ + def default(self, obj): + if isinstance(obj, ImmutableObject): + return obj.serialize() + return json.JSONEncoder.default(self, obj) + + +def model_json_decoder(dct): + """ + Automatically deserialize Mopidy models from JSON. + + Usage:: + + >>> import json + >>> json.loads( + ... '{"a_track": {"__type__": "Track", "name": "name"}}', + ... object_hook=model_json_decoder) + {u'a_track': Track(artists=[], name=u'name')} + + """ + if '__type__' in dct: + obj_type = dct.pop('__type__') + if obj_type == 'Album': + return Album(**dct) + if obj_type == 'Artist': + return Artist(**dct) + if obj_type == 'Playlist': + return Playlist(**dct) + if obj_type == 'Track': + return Track(**dct) + return dct + + class Artist(ImmutableObject): """ :param uri: artist URI diff --git a/tests/models_test.py b/tests/models_test.py index 5fbd4dd3..c49142a8 100644 --- a/tests/models_test.py +++ b/tests/models_test.py @@ -1,8 +1,11 @@ from __future__ import unicode_literals import datetime +import json -from mopidy.models import Artist, Album, TlTrack, Track, Playlist +from mopidy.models import ( + Artist, Album, TlTrack, Track, Playlist, + ModelJSONEncoder, model_json_decoder) from tests import unittest @@ -86,6 +89,12 @@ class ArtistTest(unittest.TestCase): {'__type__': 'Artist', 'uri': 'uri', 'name': 'name'}, Artist(uri='uri', name='name').serialize()) + def test_to_json_and_Back(self): + artist1 = Artist(uri='uri', name='name') + serialized = json.dumps(artist1, cls=ModelJSONEncoder) + artist2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(artist1, artist2) + def test_eq_name(self): artist1 = Artist(name='name') artist2 = Artist(name='name') @@ -205,6 +214,12 @@ class AlbumTest(unittest.TestCase): [artist.serialize()]}, Album(uri='uri', name='name', artists=[artist]).serialize()) + def test_to_json_and_back(self): + album1 = Album(uri='uri', name='name', artists=[Artist(name='foo')]) + serialized = json.dumps(album1, cls=ModelJSONEncoder) + album2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(album1, album2) + def test_eq_name(self): album1 = Album(name='name') album2 = Album(name='name') @@ -404,6 +419,14 @@ class TrackTest(unittest.TestCase): 'album': album.serialize()}, Track(uri='uri', name='name', album=album).serialize()) + def test_to_json_and_back(self): + track1 = Track( + uri='uri', name='name', album=Album(name='foo'), + artists=[Artist(name='foo')]) + serialized = json.dumps(track1, cls=ModelJSONEncoder) + track2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(track1, track2) + def test_eq_uri(self): track1 = Track(uri='uri1') track2 = Track(uri='uri1') @@ -598,6 +621,12 @@ class TlTrackTest(unittest.TestCase): {'__type__': 'TlTrack', 'tlid': 123, 'track': track.serialize()}, TlTrack(tlid=123, track=track).serialize()) + def test_to_json_and_back(self): + track1 = Track(uri='uri', name='name') + serialized = json.dumps(track1, cls=ModelJSONEncoder) + track2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(track1, track2) + def test_eq(self): tlid = 123 track = Track() @@ -733,6 +762,12 @@ class PlaylistTest(unittest.TestCase): 'tracks': [track.serialize()]}, Playlist(uri='uri', name='name', tracks=[track]).serialize()) + def test_to_json_and_back(self): + playlist1 = Playlist(uri='uri', name='name') + serialized = json.dumps(playlist1, cls=ModelJSONEncoder) + playlist2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(playlist1, playlist2) + def test_eq_name(self): playlist1 = Playlist(name='name') playlist2 = Playlist(name='name') From 3bc4126b45f952c254de82263fbc33b906679ea5 Mon Sep 17 00:00:00 2001 From: Stein Magnus Jodal Date: Sun, 18 Nov 2012 17:28:37 +0100 Subject: [PATCH 4/4] models: Fix TlTrack deserialization --- mopidy/models.py | 2 ++ tests/models_test.py | 8 ++++---- 2 files changed, 6 insertions(+), 4 deletions(-) diff --git a/mopidy/models.py b/mopidy/models.py index 9eadb314..4861ef0d 100644 --- a/mopidy/models.py +++ b/mopidy/models.py @@ -131,6 +131,8 @@ def model_json_decoder(dct): return Artist(**dct) if obj_type == 'Playlist': return Playlist(**dct) + if obj_type == 'TlTrack': + return TlTrack(**dct) if obj_type == 'Track': return Track(**dct) return dct diff --git a/tests/models_test.py b/tests/models_test.py index c49142a8..21ad7ead 100644 --- a/tests/models_test.py +++ b/tests/models_test.py @@ -622,10 +622,10 @@ class TlTrackTest(unittest.TestCase): TlTrack(tlid=123, track=track).serialize()) def test_to_json_and_back(self): - track1 = Track(uri='uri', name='name') - serialized = json.dumps(track1, cls=ModelJSONEncoder) - track2 = json.loads(serialized, object_hook=model_json_decoder) - self.assertEqual(track1, track2) + tl_track1 = TlTrack(tlid=123, track=Track(uri='uri', name='name')) + serialized = json.dumps(tl_track1, cls=ModelJSONEncoder) + tl_track2 = json.loads(serialized, object_hook=model_json_decoder) + self.assertEqual(tl_track1, tl_track2) def test_eq(self): tlid = 123