added custom http responses.

Author: guettli

README.md

@@ -20,7 +20,7 @@ I need to validate my data on the server anyway. So why should I validate them o
 
 The Django Forms library has all you need to write database focused applications.
 
-Sending HTML fragements over the wire keeps my application simple.
+Sending HTML fragments over the wire keeps my application simple.
 
 There is just one thing which is outdated (although it is still perfectly fine). The need
 for a full page refresh after submitting a form.

diary/models.py

@@ -1,6 +1,5 @@
 from django.db import models
 from django.utils import timezone
-from django.utils.html import format_html
 
 
 class Note(models.Model):

diary/tests/test_views.py

@@ -1,5 +1,9 @@
+import pytest
+from html_form_to_dict import html_form_to_dict
+
+from diary.models import Note
+from diary.utils import HttpResponseUnprocessableEntity, HttpResponseCreated
 from diary.views.note import get_next_or_none
-from django.urls import reverse
 
 
 def test_get_next_or_none__last(note):
@@ -9,7 +13,27 @@ def test_get_next_or_none__last(note):
 def test_get_next_or_none__next_exists(note2):
     assert get_next_or_none(note2).title == 'My Title'
 
+@pytest.mark.django_db
+def test_start_page__invalid(client):
+    response = client.get('/')
+    assert response.status_code == 200
+    data = html_form_to_dict(response.content)
+    assert list(data.keys()) == ['datetime', 'initial-datetime', 'title', 'text']
+    data['text'] = 'my text'
+    response = data.submit(client)
+    assert response.status_code == HttpResponseUnprocessableEntity.status_code
+    assert response.description == ('<ul class="errorlist"><li>title<ul class="errorlist"><li>This field is '
+                                    'required.</li></ul></li></ul>')
 
-def test_start_page(client, note):
+@pytest.mark.django_db
+def test_start_page__valid(client):
     response = client.get('/')
     assert response.status_code == 200
+    data = html_form_to_dict(response.content)
+    data['text'] = 'my text'
+    data['title'] = 'my title'
+    response = data.submit(client)
+    assert isinstance(response, HttpResponseCreated)
+    note = Note.objects.get(id=response.pk)
+    assert note.title == 'my title'
+    assert note.text == 'my text'

diary/utils.py

@@ -0,0 +1,23 @@
+from django.http import HttpResponse
+
+
+class HttpResponseUnprocessableEntity(HttpResponse):
+    description: str = ''
+
+    def __init__(self, content=b'', description=None, **kwargs):
+        assert description is not None, 'Please provide a description. For example form.errors'
+        super().__init__(content, **kwargs)
+        self.description = str(description)
+
+    status_code = 422
+
+
+class HttpResponseCreated(HttpResponse):
+    pk: str = ''
+
+    def __init__(self, content=b'', pk=None, **kwargs):
+        assert pk is not None, 'Please provide a pk (primary key) if the object which got saved'
+        super().__init__(content, **kwargs)
+        self.pk = pk
+
+    status_code = 201

diary/views/note.py

@@ -1,12 +1,13 @@
-from diary.models import Note
 from django.forms import ModelForm, Textarea
 from django.http import HttpResponse
 from django.shortcuts import get_object_or_404
 from django.urls import reverse
 from django.utils.html import format_html
-
 from django.views.decorators.http import require_POST
 
+from diary.models import Note
+from diary.utils import HttpResponseUnprocessableEntity, HttpResponseCreated
+
 
 class NoteCreateForm(ModelForm):
     class Meta:
@@ -56,8 +57,8 @@ def create_note_hxpost(request):
     form = NoteCreateForm(request.POST)
     if form.is_valid():
         note = form.save()
-        return HttpResponse(format_html('{} {}', note_add_html(), note_html(note)))
-    return HttpResponse(note_form_html(form))
+        return HttpResponseCreated(format_html('{} {}', note_add_html(), note_html(note)), note.pk)
+    return HttpResponseUnprocessableEntity(note_form_html(form), form.errors)
 
 
 def note_and_next_html(note):
@@ -79,7 +80,7 @@ def get_next_or_none(note):
     # distinguished with ".first()". Grrr ugly loop is needed.
     found = False
     for next in Note.objects.filter(datetime__lte=note.datetime).order_by(
-        '-datetime', '-id'
+            '-datetime', '-id'
     ):
         if found:
             return next

setup.py

@@ -19,7 +19,7 @@
         "Operating System :: OS Independent",
     ],
     python_requires='>=3.6',
-    install_requires=['Django', 'pytest-django'],
+    install_requires=['Django', 'pytest-django', 'html_form_to_dict'],
     scripts=[
         'mysite/manage.py',
     ],