Add additional /timezone test

OpenOversight/tests/routes/route_helpers.py

@@ -1,10 +1,12 @@
+from faker import Faker
 from flask import url_for
 
 from OpenOversight.app.auth.forms import LoginForm
 
 
 ADMIN_EMAIL = "[email protected]"
 ADMIN_PASSWORD = "testtest"
+FAKER = Faker()
 
 
 def login_user(client):

OpenOversight/tests/routes/test_other.py

@@ -5,7 +5,7 @@
 from flask import current_app, url_for
 
 from OpenOversight.app.utils.constants import ENCODING_UTF_8, KEY_TIMEZONE
-from OpenOversight.tests.routes.route_helpers import login_user
+from OpenOversight.tests.routes.route_helpers import FAKER, login_user
 
 
 @pytest.mark.parametrize(
@@ -58,12 +58,26 @@ def test_user_can_access_profile_differently_cased(mockdata, client, session):
 
 def test_timezone_setting(client):
     with current_app.test_request_context():
+        test_tz = FAKER.timezone()
         with client.session_transaction() as session:
             assert KEY_TIMEZONE not in session
         rv = client.post(
             url_for("main.set_session_timezone"),
-            data="A TIMEZONE".encode(ENCODING_UTF_8),
+            data=test_tz.encode(ENCODING_UTF_8),
         )
         assert rv.status_code == HTTPStatus.OK
         with client.session_transaction() as session:
-            assert session[KEY_TIMEZONE] == "A TIMEZONE"
+            assert session[KEY_TIMEZONE] == test_tz
+
+
+def test_timezone_setting_empty_string(client):
+    with current_app.test_request_context():
+        with client.session_transaction() as session:
+            assert KEY_TIMEZONE not in session
+        rv = client.post(
+            url_for("main.set_session_timezone"),
+            data="".encode(ENCODING_UTF_8),
+        )
+        assert rv.status_code == HTTPStatus.OK
+        with client.session_transaction() as session:
+            assert session[KEY_TIMEZONE] == current_app.config.get(KEY_TIMEZONE)