Fixed page 404 test. Added a cat image to the banned page.

boards/templates/boards/staticpages/banned.html

@@ -7,5 +7,6 b''
 {% endblock %}
 
 {% block staticcontent %}
-    {% trans 'Your IP address has been banned. Contact the administrator' %}
+    <p><img src="{{ STATIC_URL }}images/banned.png" width="200" /></p>
+    <p>{% trans 'Your IP address has been banned. Contact the administrator' %}</p>
 {% endblock %}
  No newline at end of file

boards/tests.py

@@ -113,13 +113,18 b' class BoardTests(TestCase):'
         self.assertEqual(1, Post.objects.count(), msg='The validation passed '
                                                       'where it should fail')
 
+        # Change posting delay so we don't have to wait for 30 seconds or more
+        old_posting_delay = settings.POSTING_DELAY
         # Wait fot the posting delay or we won't be able to post
+        settings.POSTING_DELAY = 1
         time.sleep(settings.POSTING_DELAY + 1)
         response = client.post(THREAD_PAGE_ONE, {'text': TEST_TEXT,
                                                  'tags': valid_tags})
         self.assertEqual(HTTP_CODE_REDIRECT, response.status_code,
                          msg=u'Posting new message failed: got code ' +
                              str(response.status_code))
+        # Restore posting delay
+        settings.POSTING_DELAY = old_posting_delay
 
         self.assertEqual(2, Post.objects.count(),
                          msg=u'No posts were created')
@@ -127,8 +132,6 b' class BoardTests(TestCase):'
         # Restore captcha setting
         settings.ENABLE_CAPTCHA = captcha_enabled
 
-    # TODO This test fails for now. We must check for 404.html instead of
-    # code 404
     def test_404(self):
         """Test receiving error 404 when opening a non-existent page"""
 
@@ -149,9 +152,8 b' class BoardTests(TestCase):'
 
         response_not_existing = client.get(THREAD_PAGE + str(
             existing_post_id + 1) + '/')
-        response_not_existing.get_full_path()
-        self.assertEqual(HTTP_CODE_NOT_FOUND,
-                         response_not_existing.status_code,
+        self.assertEqual('boards/404.html',
+                         response_not_existing.templates[0].name,
                          u'Not existing thread is opened')
 
         response_existing = client.get(TAG_PAGE + tag_name + '/')
@@ -160,14 +162,14 b' class BoardTests(TestCase):'
                          u'Cannot open existing tag')
 
         response_not_existing = client.get(TAG_PAGE + u'not_tag' + '/')
-        self.assertEqual(HTTP_CODE_NOT_FOUND,
-                         response_not_existing.status_code,
+        self.assertEqual('boards/404.html',
+                         response_not_existing.templates[0].name,
                          u'Not existing tag is opened')
 
         reply_id = Post.objects.create_post('', TEST_TEXT,
-                                            parent_id=existing_post_id)
+                                            thread=Post.objects.all()[0])
         response_not_existing = client.get(THREAD_PAGE + str(
             reply_id) + '/')
-        self.assertEqual(HTTP_CODE_NOT_FOUND,
-                         response_not_existing.status_code,
-                         u'Not existing thread is opened')
+        self.assertEqual('boards/404.html',
+                         response_not_existing.templates[0].name,
+                         u'Reply is opened as a thread')

boards/views.py

@@ -82,9 +82,6 b' def _new_post(request, form, thread_id=b'
                     tag, created = Tag.objects.get_or_create(name=tag_name)
                     tags.append(tag)
 
-    # TODO Add a possibility to define a link image instead of an image file.
-    # If a link is given, download the image automatically.
-
     op = None if thread_id == boards.models.NO_PARENT else \
         get_object_or_404(Post, id=thread_id)
     post = Post.objects.create_post(title=title, text=text, ip=ip,