I dug into the project history to figure out how to test the error path of splitDateTime() and discovered that the code that motivated the error path was replaced but the error handling it required was not removed. Error handling added to parseDate() override in commit 4496926. Refactored to splitDateTime() in e7c7d40. Happy path implementation replaced in b838a21.

Author: srvance

jquery-ui-timepicker-addon.js

@@ -1833,46 +1833,22 @@
 	* Returns {dateString: dateString, timeString: timeString}
 	*/
 	var splitDateTime = function(dateFormat, dateTimeString, dateSettings, timeSettings) {
-		try {
-			// The idea is to get the number separator occurrences in datetime and the time format requested (since time has
-			// fewer unknowns, mostly numbers and am/pm). We will use the time pattern to split.
-			var separator = computeEffectiveSetting(timeSettings, 'separator'),
-				format = computeEffectiveSetting(timeSettings, 'timeFormat'),
-				timeParts = format.split(separator), // how many occurrences of separator may be in our format?
-				timePartsLen = timeParts.length,
-				allParts = dateTimeString.split(separator),
-				allPartsLen = allParts.length;
-
-			if (allPartsLen > 1) {
-				return {
-					dateString: allParts.splice(0,allPartsLen-timePartsLen).join(separator),
-					timeString: allParts.splice(0,timePartsLen).join(separator)
-				};
-			}
-
-		} catch (err) {
-			$.timepicker.log('Could not split the date from the time. Please check the following datetimepicker options' +
-					"\nthrown error: " + err +
-					"\ndateTimeString" + dateTimeString +
-					"\ndateFormat = " + dateFormat +
-					"\nseparator = " + timeSettings.separator +
-					"\ntimeFormat = " + timeSettings.timeFormat);
-
-			if (err.indexOf(":") >= 0) {
-				// Hack!  The error message ends with a colon, a space, and
-				// the "extra" characters.  We rely on that instead of
-				// attempting to perfectly reproduce the parsing algorithm.
-				var dateStringLength = dateTimeString.length - (err.length - err.indexOf(':') - 2);
-
-				return {
-					dateString: $.trim(dateTimeString.substring(0, dateStringLength)),
-					timeString: $.trim(dateTimeString.substring(dateStringLength))
-				};
-
-			} else {
-				throw err;
-			}
+		// The idea is to get the number separator occurrences in datetime and the time format requested (since time has
+		// fewer unknowns, mostly numbers and am/pm). We will use the time pattern to split.
+		var separator = computeEffectiveSetting(timeSettings, 'separator'),
+			format = computeEffectiveSetting(timeSettings, 'timeFormat'),
+			timeParts = format.split(separator), // how many occurrences of separator may be in our format?
+			timePartsLen = timeParts.length,
+			allParts = dateTimeString.split(separator),
+			allPartsLen = allParts.length;
+
+		if (allPartsLen > 1) {
+			return {
+				dateString: allParts.splice(0,allPartsLen-timePartsLen).join(separator),
+				timeString: allParts.splice(0,timePartsLen).join(separator)
+			};
 		}
+
 		return {
 			dateString: dateTimeString,
 			timeString: ''

test/jquery-ui-timepicker-addon_spec.js

@@ -263,8 +263,6 @@ describe('datetimepicker', function() {
 
 				expect(result).toEqual({dateString: expectedDateString, timeString: ''});
 			});
-
-			// TODO: Should test the error path, but not sure what throws the error or what the message looks like.
 		});
 
 		describe('parseDateTimeInternal', function() {