Revert "Non-ASCII input will be caught anyway, because it's never a valid character"

This reverts commit fe1911e.

Author: Techassi

src/duration/mod.rs

@@ -30,8 +30,8 @@ mod serde_impl;
 #[derive(Debug, Snafu, PartialEq)]
 #[snafu(module)]
 pub enum DurationParseError {
-    #[snafu(display("empty input"))]
-    EmptyInput,
+    #[snafu(display("invalid input, either empty or contains non-ascii characters"))]
+    InvalidInput,
 
     #[snafu(display("unexpected character {chr:?}"))]
     UnexpectedCharacter { chr: char },
@@ -65,8 +65,9 @@ impl FromStr for Duration {
         use duration_parse_error::*;
         let input = s.trim();
 
-        if input.is_empty() {
-            return EmptyInputSnafu.fail();
+        // An empty or non-ascii input is invalid
+        if input.is_empty() || !input.is_ascii() {
+            return Err(DurationParseError::InvalidInput);
         }
 
         let mut chars = input.char_indices().peekable();
@@ -284,8 +285,8 @@ mod test {
     #[rstest]
     #[case("1D", DurationParseError::ParseUnitError{unit: "D".into()})]
     #[case("2d2", DurationParseError::NoUnit{value: 2})]
-    #[case("1ä", DurationParseError::EmptyInput)]
-    #[case(" ", DurationParseError::EmptyInput)]
+    #[case("1ä", DurationParseError::InvalidInput)]
+    #[case(" ", DurationParseError::InvalidInput)]
     fn parse_invalid(#[case] input: &str, #[case] expected_err: DurationParseError) {
         let err = Duration::from_str(input).unwrap_err();
         assert_eq!(err, expected_err)