From 2d7b6bdef9f4aead59576b7bcbb2f42ba9c92ad7 Mon Sep 17 00:00:00 2001 From: Joar Wandborg Date: Sun, 7 Apr 2013 23:17:23 +0200 Subject: New notifications - Added request.notifications - Email configuration fixes - Set config_spec default SMTP port to `0` and switch to SSL/non-SSL default if `port == 0` - Added email_smtp_use_ssl configuration setting - Added migrations for notification tables - Added __repr__ to MediaComment(Mixin) - Added MediaComment.get_entry => MediaEntry - Added CommentSubscription, CommentNotification, Notification, ProcessingNotification tables - Added notifications.task to celery init - Fixed a bug in the video transcoder where pygst would hijack the --help argument. - Added notifications - views - silence - subscribe - routes - utility methods - celery task - Added half-hearted .active comment CSS style - Added quick JS to show header_dropdown - Added fragment template to show notifications in header_dropdown - Added fragment template to show subscribe/unsubscribe buttons on media/comment pages - Updated celery setup tests with notifications.task - Tried to fix test_misc tests that I broke - Added notification tests - Added and extended tests.tools fixtures - Integrated new notifications into media_home, media_post_comment views - Bumped SQLAlchemy dependency to >= 0.8.0 since we need polymorphic for the notifications to work --- mediagoblin/tests/test_misc.py | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) (limited to 'mediagoblin/tests/test_misc.py') diff --git a/mediagoblin/tests/test_misc.py b/mediagoblin/tests/test_misc.py index 755d863f..6af6bf92 100644 --- a/mediagoblin/tests/test_misc.py +++ b/mediagoblin/tests/test_misc.py @@ -28,8 +28,10 @@ def test_user_deletes_other_comments(test_app): user_a = fixture_add_user(u"chris_a") user_b = fixture_add_user(u"chris_b") - media_a = fixture_media_entry(uploader=user_a.id, save=False) - media_b = fixture_media_entry(uploader=user_b.id, save=False) + media_a = fixture_media_entry(uploader=user_a.id, save=False, + expunge=False) + media_b = fixture_media_entry(uploader=user_b.id, save=False, + expunge=False) Session.add(media_a) Session.add(media_b) Session.flush() @@ -79,7 +81,7 @@ def test_user_deletes_other_comments(test_app): def test_media_deletes_broken_attachment(test_app): user_a = fixture_add_user(u"chris_a") - media = fixture_media_entry(uploader=user_a.id, save=False) + media = fixture_media_entry(uploader=user_a.id, save=False, expunge=False) media.attachment_files.append(dict( name=u"some name", filepath=[u"does", u"not", u"exist"], -- cgit v1.2.3 From 56d1346789c2b628aa1af986c101010c426ecc56 Mon Sep 17 00:00:00 2001 From: Christopher Allan Webber Date: Sat, 22 Jun 2013 17:37:30 -0500 Subject: Fix the data integrity bug in test_misc. It turns out this has to do with some additions to the fixture adding of media entries. Since the new fake_upload field is True by default, adding multiple entries that have media_files at once meant that multiple additions of FileKeynames would be added at once if they were not saved at the same time. Tricky! Thankfully, Elrond helped us figure this one out. Thanks, Elrond :) And thanks also to Aapo Rantalainen for sponsoring this commit! --- mediagoblin/tests/test_misc.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'mediagoblin/tests/test_misc.py') diff --git a/mediagoblin/tests/test_misc.py b/mediagoblin/tests/test_misc.py index 6af6bf92..43ad0b6d 100644 --- a/mediagoblin/tests/test_misc.py +++ b/mediagoblin/tests/test_misc.py @@ -29,9 +29,9 @@ def test_user_deletes_other_comments(test_app): user_b = fixture_add_user(u"chris_b") media_a = fixture_media_entry(uploader=user_a.id, save=False, - expunge=False) + expunge=False, fake_upload=False) media_b = fixture_media_entry(uploader=user_b.id, save=False, - expunge=False) + expunge=False, fake_upload=False) Session.add(media_a) Session.add(media_b) Session.flush() -- cgit v1.2.3