fix: cluster configuration validation for bool type

As bool is a subtype of int, True/False was considered as 1/0

Author: Ygnas

src/codeflare_sdk/common/utils/unit_test_support.py

@@ -55,7 +55,7 @@ def createClusterWrongType():
     config = ClusterConfiguration(
         name="unit-test-cluster",
         namespace="ns",
-        num_workers=2,
+        num_workers=True,
         worker_cpu_requests=[],
         worker_cpu_limits=4,
         worker_memory_requests=5,

src/codeflare_sdk/ray/cluster/config.py

@@ -242,13 +242,15 @@ def _memory_to_resource(self):
 
     def _validate_types(self):
         """Validate the types of all fields in the ClusterConfiguration dataclass."""
+        errors = []
         for field_info in fields(self):
             value = getattr(self, field_info.name)
             expected_type = field_info.type
             if not self._is_type(value, expected_type):
-                raise TypeError(
-                    f"'{field_info.name}' should be of type {expected_type}"
-                )
+                errors.append(f"'{field_info.name}' should be of type {expected_type}.")
+
+        if errors:
+            raise TypeError("Type validation failed:\n" + "\n".join(errors))
 
     @staticmethod
     def _is_type(value, expected_type):
@@ -268,6 +270,11 @@ def check_type(value, expected_type):
                 )
             if origin_type is tuple:
                 return all(check_type(elem, etype) for elem, etype in zip(value, args))
-            return isinstance(value, expected_type)
+            if isinstance(expected_type, type):
+                if expected_type is int:
+                    return isinstance(value, int) and not isinstance(value, bool)
+                if expected_type is bool:
+                    return isinstance(value, bool)
+                return isinstance(value, expected_type)
 
         return check_type(value, expected_type)