Now using expected_at and simplified crash case

Author: LucaCappelletti94

src/parser/mod.rs

@@ -14510,17 +14510,21 @@ impl<'a> Parser<'a> {
             }
             .into());
         } else if self.parse_keyword(Keyword::AUTHORIZATION) {
+            let scope = match scope {
+                Some(s) => s,
+                None => {
+                    return self.expected_at(
+                        "SESSION, LOCAL, or other scope modifier before AUTHORIZATION",
+                        self.get_current_index(),
+                    )
+                }
+            };
             let auth_value = if self.parse_keyword(Keyword::DEFAULT) {
                 SetSessionAuthorizationParamKind::Default
             } else {
                 let value = self.parse_identifier()?;
                 SetSessionAuthorizationParamKind::User(value)
             };
-            let scope = scope.ok_or_else(|| {
-                ParserError::ParserError(
-                    "Expected a scope modifier (e.g. SESSION) before AUTHORIZATION".to_string(),
-                )
-            })?;
             return Ok(Set::SetSessionAuthorization(SetSessionAuthorizationParam {
                 scope,
                 kind: auth_value,

tests/sqlparser_common.rs

@@ -18283,9 +18283,7 @@ fn test_parse_set_session_authorization() {
 #[test]
 fn test_set_authorization_without_scope_errors() {
     // This should return a parser error, not panic.
-    let res = parse_sql_statements(
-        "\tSET\t\t\t\t\t\t\t\t\t\tAUTHORIZATION\tTIME\t\t\t\t\t\tTIME\u{fffd}\u{fffd}v\u{1}\0\0\t74843EUTI>\0\0\0\0\0\0\0\0\0\0\0\0\0\0\0USER_RNCLUDE_NULL\0\0\0\0\0\0\0\0\0\t\t\t\t^^^^^^^^\tWHI\t\tIN"
-    );
+    let res = parse_sql_statements("SET AUTHORIZATION TIME TIME");
     assert!(
         res.is_err(),
         "SET AUTHORIZATION without a scope modifier (e.g. SESSION) should error"