Add test for mustBeValidTime

Author: ehennestad

code/internal/property_validation/mustBeValidTime.m

@@ -1,5 +1,8 @@
 function mustBeValidTime(value)
-    isValid = all(value.Year == 0 & value.Month == 0 & value.Day == 0);
+    
+    [h, m, s] = hms(value); hmsArray = [h; m; s];
+    isValid = all(value.Year == -1) && all( sum(hmsArray, 1) ~= 0);
+
     if ~isValid
         warning('OPENMINDS_MATLAB:PropertyValidators:InvalidTime', ...
             'Value must be a datetime value without date information')

tools/tests/unitTests/PropertyValidatorTest.m

@@ -113,5 +113,26 @@ function testMustBeValidStringLength(testCase)
                 @() mustBeValidStringLength(testValue, 10, 15), ...
                 'OPENMINDS_MATLAB:PropertyValidators:InvalidStringLength')
         end
+
+        function testMustBeValidTime(testCase)
+
+            validTime = datetime(0,0,0,18,30,10);
+            inValidTime =  datetime(2024,1,1,18,30,10);
+            inValidTimeNoTime =  datetime(2024,1,1,0,0,0);
+
+
+            % Test function when value is a valid time
+            mustBeValidTime(validTime)
+
+            % Test function when datetime contains a year
+            testCase.verifyWarning(...
+                @() mustBeValidTime(inValidTime), ...
+                'OPENMINDS_MATLAB:PropertyValidators:InvalidTime')
+
+            % Test function when there is no time info
+            testCase.verifyWarning(...
+                @() mustBeValidTime(inValidTimeNoTime), ...
+                'OPENMINDS_MATLAB:PropertyValidators:InvalidTime')
+        end
     end
 end