Tests: Cover OPTION FOR Display propagation and non-identifier REVOKE

Author: LucaCappelletti94

tests/sqlparser_common.rs

@@ -22636,4 +22636,33 @@ fn parse_revoke_role_option_for() {
             .unwrap_err(),
         ParserError::ParserError("Expected: FROM, found: TO".to_string())
     );
+    assert_eq!(
+        all_dialects()
+            .parse_sql_statements("REVOKE 1 FROM anon")
+            .unwrap_err(),
+        ParserError::ParserError("Expected: a privilege keyword, found: 1".to_string())
+    );
+}
+
+#[test]
+fn display_revoke_role_option_for_propagates_write_errors() {
+    use core::fmt::{self, Write};
+
+    struct RejectingWriter<'a>(&'a str);
+
+    impl Write for RejectingWriter<'_> {
+        fn write_str(&mut self, value: &str) -> fmt::Result {
+            if value.contains(self.0) {
+                Err(fmt::Error)
+            } else {
+                Ok(())
+            }
+        }
+    }
+
+    let statement = all_dialects().verified_stmt("REVOKE ADMIN OPTION FOR bypasser FROM anon");
+    for rejected in ["REVOKE ", "ADMIN", " OPTION FOR "] {
+        let mut writer = RejectingWriter(rejected);
+        assert_eq!(write!(writer, "{statement}"), Err(fmt::Error));
+    }
 }