Add a source location for UnexpectedEndOfInput parse error (#504)

CHANGELOG.md

@@ -8,6 +8,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 ### Changed
+- *BREAKING* partiql-parser: Added a source location to `ParseError::UnexpectedEndOfInput`
 
 ### Added
 

partiql-parser/src/error.rs

@@ -50,7 +50,7 @@ where
 
     /// There were not enough tokens to complete a parse
     #[error("Unexpected end of input")]
-    UnexpectedEndOfInput,
+    UnexpectedEndOfInput(Loc),
 
     /// An otherwise un-categorized error occurred
     #[error("Unknown parse error at `{}`", _0)]
@@ -83,14 +83,14 @@ where
     Loc: Display,
 {
     /// Maps an `ParserError<Loc>` to `ParserError<Loc2>` by applying a function to each variant
-    pub fn map_loc<F, Loc2>(self, tx: F) -> ParseError<'input, Loc2>
+    pub fn map_loc<F, Loc2>(self, mut tx: F) -> ParseError<'input, Loc2>
     where
         Loc2: Display,
         F: FnMut(Loc) -> Loc2,
     {
         match self {
             ParseError::SyntaxError(l) => ParseError::SyntaxError(l.map_loc(tx)),
-            ParseError::UnexpectedEndOfInput => ParseError::UnexpectedEndOfInput,
+            ParseError::UnexpectedEndOfInput(l) => ParseError::UnexpectedEndOfInput((tx)(l)),
             ParseError::UnexpectedToken(l) => ParseError::UnexpectedToken(l.map_loc(tx)),
             ParseError::LexicalError(l) => ParseError::LexicalError(l.map_loc(tx)),
             ParseError::IllegalState(s) => ParseError::IllegalState(s),

partiql-parser/src/parse/mod.rs

@@ -119,9 +119,10 @@ impl<'input> From<LalrpopError<'input>> for ParseError<'input, BytePosition> {
             }
 
             // TODO do something with UnrecognizedEof.expected
-            lalrpop_util::ParseError::UnrecognizedEof { expected: _, .. } => {
-                ParseError::UnexpectedEndOfInput
-            }
+            lalrpop_util::ParseError::UnrecognizedEof {
+                location,
+                expected: _,
+            } => ParseError::UnexpectedEndOfInput(location.into()),
 
             lalrpop_util::ParseError::ExtraToken {
                 token: (start, token, end),
@@ -769,7 +770,10 @@ mod tests {
             assert!(res.is_err());
             let err_data = res.unwrap_err();
             assert_eq!(1, err_data.errors.len());
-            assert_eq!(err_data.errors[0], ParseError::UnexpectedEndOfInput);
+            assert_eq!(
+                err_data.errors[0],
+                ParseError::UnexpectedEndOfInput(BytePosition::from(6))
+            );
         }
 
         #[test]