Fixed #14176 -- Added forwards compatibility to the legacy syndication feed view. This allows class-based feeds to be deployed using the old-style feed view, as long as the feed requires no arguments (i.e., get_object returns None). Thanks to psychcf for the report, cwhaines for the investigation, and Andrew Godwin for the assist.

git-svn-id: http://code.djangoproject.com/svn/django/trunk@15189 bcc190cf-cafb-0310-a4f2-bffc1f526a37

3 files changed, 43 insertions, 0 deletions

diff --git a/tests/regressiontests/comment_tests/tests/__init__.py b/tests/regressiontests/comment_tests/tests/__init__.py
index 449fea471d..88c6f33203 100644
--- a/tests/regressiontests/comment_tests/tests/__init__.py
+++ b/tests/regressiontests/comment_tests/tests/__init__.py
@@ -81,6 +81,7 @@ class CommentTestCase(TestCase):
         return d
 
 from regressiontests.comment_tests.tests.app_api_tests import *
+from regressiontests.comment_tests.tests.feed_tests import *
 from regressiontests.comment_tests.tests.model_tests import *
 from regressiontests.comment_tests.tests.comment_form_tests import *
 from regressiontests.comment_tests.tests.templatetag_tests import *
diff --git a/tests/regressiontests/comment_tests/tests/feed_tests.py b/tests/regressiontests/comment_tests/tests/feed_tests.py
new file mode 100644
index 0000000000..0bc6618d9a
--- /dev/null
+++ b/tests/regressiontests/comment_tests/tests/feed_tests.py
@@ -0,0 +1,33 @@
+import warnings
+
+from django.test.utils import get_warnings_state, restore_warnings_state
+
+from regressiontests.comment_tests.tests import CommentTestCase
+
+
+class CommentFeedTests(CommentTestCase):
+    urls = 'regressiontests.comment_tests.urls'
+    feed_url = '/rss/comments/'
+
+    def test_feed(self):
+        response = self.client.get(self.feed_url)
+        self.assertEquals(response.status_code, 200)
+        self.assertEquals(response['Content-Type'], 'application/rss+xml')
+        self.assertContains(response, '<rss xmlns:atom="http://www.w3.org/2005/Atom" version="2.0">')
+        self.assertContains(response, '<title>example.com comments</title>')
+        self.assertContains(response, '<link>http://example.com/</link>')
+        self.assertContains(response, '</rss>')
+
+
+class LegacyCommentFeedTests(CommentFeedTests):
+    feed_url = '/rss/legacy/comments/'
+
+    def setUp(self):
+        self._warnings_state = get_warnings_state()
+        warnings.filterwarnings("ignore", category=DeprecationWarning,
+                                module='django.contrib.syndication.views')
+        warnings.filterwarnings("ignore", category=DeprecationWarning,
+                                module='django.contrib.syndication.feeds')
+
+    def tearDown(self):
+        restore_warnings_state(self._warnings_state)
diff --git a/tests/regressiontests/comment_tests/urls.py b/tests/regressiontests/comment_tests/urls.py
index 0058689657..052f7cf81d 100644
--- a/tests/regressiontests/comment_tests/urls.py
+++ b/tests/regressiontests/comment_tests/urls.py
@@ -1,4 +1,9 @@
 from django.conf.urls.defaults import *
+from django.contrib.comments.feeds import LatestCommentFeed
+
+feeds = {
+     'comments': LatestCommentFeed,
+}
 
 urlpatterns = patterns('regressiontests.comment_tests.custom_comments.views',
     url(r'^post/$',          'custom_submit_comment'),
@@ -7,3 +12,7 @@ urlpatterns = patterns('regressiontests.comment_tests.custom_comments.views',
     url(r'^approve/(\d+)/$', 'custom_approve_comment'),
 )
 
+urlpatterns += patterns('',
+    (r'^rss/legacy/(?P<url>.*)/$', 'django.contrib.syndication.views.feed', {'feed_dict': feeds}),
+    (r'^rss/comments/$', LatestCommentFeed()),
+)