fix: convert DAPR_HEALTH_TIMEOUT to float (#692)

* fix: convert DAPR_HEALTH_TIMEOUT to float

Signed-off-by: KentHsu <chiahaohsu9@gmail.com>

* test: update healthcheck timeout mock to a float value

Signed-off-by: KentHsu <chiahaohsu9@gmail.com>

---------

Signed-off-by: KentHsu <chiahaohsu9@gmail.com>
Co-authored-by: Bernd Verst <github@bernd.dev>

dapr/clients/health.py

@@ -28,7 +28,7 @@ def wait_until_ready():
         headers = {USER_AGENT_HEADER: DAPR_USER_AGENT}
         if settings.DAPR_API_TOKEN is not None:
             headers[DAPR_API_TOKEN_HEADER] = settings.DAPR_API_TOKEN
-        timeout = settings.DAPR_HEALTH_TIMEOUT
+        timeout = float(settings.DAPR_HEALTH_TIMEOUT)
 
         start = time.time()
         while True:

tests/clients/test_heatlhcheck.py

@@ -62,7 +62,7 @@ def test_wait_until_ready_success_with_api_token(self, mock_urlopen):
         self.assertIn('Dapr-api-token', headers)
         self.assertEqual(headers['Dapr-api-token'], 'mytoken')
 
-    @patch.object(settings, 'DAPR_HEALTH_TIMEOUT', 2)
+    @patch.object(settings, 'DAPR_HEALTH_TIMEOUT', '2.5')
     @patch('urllib.request.urlopen')
     def test_wait_until_ready_timeout(self, mock_urlopen):
         mock_urlopen.return_value.__enter__.return_value = MagicMock(status=500)
@@ -72,5 +72,5 @@ def test_wait_until_ready_timeout(self, mock_urlopen):
         with self.assertRaises(TimeoutError):
             DaprHealth.wait_until_ready()
 
-        self.assertGreaterEqual(time.time() - start, 2)
+        self.assertGreaterEqual(time.time() - start, 2.5)
         self.assertGreater(mock_urlopen.call_count, 1)