Change variable name to avoid confusion between DB and HTTP connections.

[plomtask] / tests / test_days.py

diff --git a/tests/test_days.py b/tests/test_days.py
index 7e06198788c734aea2d44de50585bb859811b85e..cc0f80db857b2fdf4ad83124d117b39c2cdb5135 100644 (file)
--- a/tests/test_days.py
+++ b/tests/test_days.py
@@ -2,6 +2,7 @@
 from unittest import TestCase
 from threading import Thread
 from http.client import HTTPConnection
+from urllib.parse import urlencode
 from datetime import datetime
 from os import remove as remove_file
 from plomtask.http import TaskHandler, TaskServer
@@ -112,17 +113,35 @@ class TestsWithServer(TestsWithDB):
         self.server_thread.daemon = True
         self.server_thread.start()
 
+    def test_do_POST_day(self):
+        """Test POST /day and its effect on the database."""
+        http_conn = HTTPConnection(*self.httpd.server_address)
+        form_data = {'comment': 'foo'}
+        encoded_form_data = urlencode(form_data).encode('utf-8')
+        headers = {'Content-Type': 'application/x-www-form-urlencoded',
+                   'Content-Length': str(len(encoded_form_data))}
+        http_conn.request('POST', '/day?date=FOO',
+                          body=encoded_form_data, headers=headers)
+        self.assertEqual(http_conn.getresponse().status, 400)
+        self.assertEqual(Day.all(self.db_conn), [])
+        http_conn.request('POST', '/day?date=2024-01-01',
+                          body=encoded_form_data, headers=headers)
+        self.assertEqual(http_conn.getresponse().status, 302)
+        retrieved = Day.by_date(self.db_conn, '2024-01-01')
+        self.assertEqual(retrieved.comment, 'foo')
+        self.assertEqual(Day.all(self.db_conn), [retrieved])
+
     def test_do_GET(self):
         """Test /day and /calendar response codes."""
-        conn = HTTPConnection(*self.httpd.server_address)
-        conn.request('GET', '/day')
-        self.assertEqual(conn.getresponse().status, 200)
-        conn.request('GET', '/day?date=3000-01-01')
-        self.assertEqual(conn.getresponse().status, 200)
-        conn.request('GET', '/day?date=FOO')
-        self.assertEqual(conn.getresponse().status, 400)
-        conn.request('GET', '/calendar')
-        self.assertEqual(conn.getresponse().status, 200)
+        http_conn = HTTPConnection(*self.httpd.server_address)
+        http_conn.request('GET', '/day')
+        self.assertEqual(http_conn.getresponse().status, 200)
+        http_conn.request('GET', '/day?date=3000-01-01')
+        self.assertEqual(http_conn.getresponse().status, 200)
+        http_conn.request('GET', '/day?date=FOO')
+        self.assertEqual(http_conn.getresponse().status, 400)
+        http_conn.request('GET', '/calendar')
+        self.assertEqual(http_conn.getresponse().status, 200)
 
     def tearDown(self):
         self.httpd.shutdown()