[STYLE] Expanded locale testing stability (- WIP PR #448 -)

Changes in file .github/workflows/CI-BUILD.yml:
 * added fr_CA.UTF-8 to build testing
 * small style changes

Changes in file tests/test_hear_cleanup.py:
 * new test for negitive case

Changes in file tox.ini:
 * More style changes

Author: reactive-firewall

.github/workflows/CI-BUILD.yml

@@ -23,6 +23,7 @@ env:
   # Define Python versions at the top level -- Expected format: X.Y (e.g., 3.13)
   PYTHON_DEFAULT: "${{ vars.PYTHON_DEFAULT }}"
   PYTHON_OLD_MIN: "${{ vars.PYTHON_OLD_MIN }}"  # For Oldest Python versions
+  PYTHON_OLD_EXTRA: "${{ vars.PYTHON_OLD_EXTRA }}"  # For Oldest Python versions (Extra coverage)
   PYTHON_EXPERIMENTAL: "${{ vars.PYTHON_EXPERIMENTAL }}"  # For future Python versions
 
 jobs:
@@ -149,14 +150,17 @@ jobs:
       fail-fast: false
       matrix:
         python-version: ["${{ vars.PYTHON_OLD_MIN }}", "${{ vars.PYTHON_DEFAULT }}", "${{ vars.PYTHON_EXPERIMENTAL }}"]
-        lang-var: ["de.utf-8", "jp.utf-8"]
+        lang-var: ["de_DE.UTF-8", "ja_JP.UTF-8", "fr_CA.UTF-8"]
         experimental: [true]
         include:
           - python-version: "${{ vars.PYTHON_DEFAULT }}"
-            lang-var: "de.utf-8"
+            lang-var: "de_DE.UTF-8"
             experimental: false
           - python-version: "${{ vars.PYTHON_DEFAULT }}"
-            lang-var: "jp.utf-8"
+            lang-var: "ja_JP.UTF-8"
+            experimental: false
+          - python-version: "${{ vars.PYTHON_DEFAULT }}"
+            lang-var: "fr_CA.UTF-8"
             experimental: false
           - python-version: "${{ vars.PYTHON_OLD_MIN }}"
             lang-var: "en_US.utf-8"

tests/test_hear_cleanup.py

@@ -207,6 +207,50 @@ def get_default_ip() -> str:
 				s.close()
 		return ip
 
+	def test_should_not_invoke_kill_func_when_handle_error_not_called(self) -> None:
+		"""Test that handle_error only conditionally calls kill_func on stop keyword.
+
+		Verifies that the server properly handles mocked requests without
+		the STOP command and never calls the kill_func to free up server resources early.
+
+		Args:
+			None (self is implicit)
+
+		Returns:
+			None
+
+		Raises:
+			AssertionError: If the test conditions are not met.
+		"""
+		theResult = False
+		fail_fixture = "Mock(MSG) --> Handler-HEAR --> early shutdown"
+		_fixture_port_num = self._the_test_port
+		try:
+			self.assertIsNotNone(_fixture_port_num)
+			self.assertIsInstance(_fixture_port_num, int)
+			# Create an instance of McastServer
+			server_address = (self.TEST_MULTICAST_GROUP, _fixture_port_num)
+			self.server = multicast.hear.McastServer(server_address, None, False)
+			self.server.shutdown = MagicMock()  # Mock the shutdown method
+			client_address = (self.get_default_ip(), _fixture_port_num)
+			# Mock a request not containing "STOP"
+			request = ("Any other message with O, P, S, T", multicast.genSocket())
+			# Add assertions for initial state
+			self.assertIsNotNone(request[1], "Socket should be created")
+			self.assertIsInstance(request[0], str, "Request should be a string")
+			try:
+				self.server.handle_error(request, client_address)
+				# Assert that the shutdown method was called
+				self.server.shutdown.assert_not_called()
+				theResult = True
+			finally:
+				# Clean up
+				self.server.server_close()
+		except Exception as _cause:
+			context.debugtestError(_cause)
+			self.fail(fail_fixture)
+		self.assertTrue(theResult, fail_fixture)
+
 	def test_should_invoke_kill_func_when_handle_error_called(self) -> None:
 		"""Test that kill_func calls shutdown on the server instance.
 

tox.ini

@@ -53,8 +53,8 @@ alwayscopy = True
 passenv =
     {[base]passenv}
 setenv =
-    LC_CTYPE=C.utf8
-    LC_ALL=en_US.utf8
+    LC_CTYPE=C.UTF-8
+    LC_ALL=en_US.UTF-8
     PYTHON={envpython}
     PYTHONCOERCECLOCALE=UTF-8
     PYTHONUTF8=1
@@ -106,10 +106,10 @@ skipsdist = False
 recreate = True
 alwayscopy = True
 passenv =
-    LC_CTYPE=C.utf8
-    LC_ALL=en_US.utf8
     {[base]passenv}
 setenv =
+    LC_CTYPE=C.UTF-8
+    LC_ALL=en_US.UTF-8
     PYTHON={envpython}
     PYTHONCOERCECLOCALE=UTF-8
     PYTHONUTF8=1
@@ -134,10 +134,10 @@ skipsdist = False
 recreate = True
 alwayscopy = True
 passenv =
-    LC_CTYPE=C.utf8
-    LC_ALL=en_US.utf8
     {[base]passenv}
 setenv =
+    LC_CTYPE=C.UTF-8
+    LC_ALL=en_US.UTF-8
     PYTHON={envpython}
     PYTHONCOERCECLOCALE=UTF-8
     PYTHONUTF8=1