Skip over some known divergence between implementations

pulldown-cmark · Jan 21, 2024 · 9b5cd31 · 9b5cd31
1 parent 0e406a1
commit 9b5cd31
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 6 deletions.
diff --git a/fuzz/fuzz_targets/commonmark_js.rs b/fuzz/fuzz_targets/commonmark_js.rs
@@ -25,11 +25,6 @@ fuzz_target!(|text: String| {
         return;
     }
 
-    // https://github.com/raphlinus/pulldown-cmark/issues/659
-    if text.contains("<") {
-        return;
-    }
-
     // There are some trivial differences due to trailing whitespace.
     let text = text
         .lines()

diff --git a/fuzz/fuzz_targets/pandoc.rs b/fuzz/fuzz_targets/pandoc.rs
@@ -48,6 +48,9 @@ fuzz_target!(|text: String| {
     if text.contains("[\r") {
         return;
     }
+    if text.contains("`") {
+        return;
+    }
 
     if text.bytes().any(|b| b > 127) {
         return;

diff --git a/fuzz/src/lib.rs b/fuzz/src/lib.rs
@@ -871,10 +871,13 @@ pub fn normalize(events: Vec<Event<'_>>) -> Vec<Event<'_>> {
                 dest_url,
                 title,
                 id: _,
+                link_type,
                 ..
             }) => vec![Event::Start(Tag::Link {
                 link_type: LinkType::Inline,
-                dest_url: dest_url.clone(),
+                // pandoc and commonmark.js both turn <i@e> into <mailto:i@e> and don't
+                // distinguish between them
+                dest_url: format!("{e}{dest_url}", e = if link_type == LinkType::Email { "mailto:" } else { "" }).into(),
                 title: title.clone(),
                 // commonmark.js does not record the id, either
                 id: "".into(),