From b82562fe8ab14623675a6c3482685ab76f875bdd Mon Sep 17 00:00:00 2001 From: dalance Date: Tue, 17 Mar 2020 17:11:12 +0900 Subject: [PATCH] Fix SUMMARY's parse_numbered with comment --- src/book/summary.rs | 31 +++++++++++++++++++++++++++++++ 1 file changed, 31 insertions(+) diff --git a/src/book/summary.rs b/src/book/summary.rs index 33e8126d..1e130537 100644 --- a/src/book/summary.rs +++ b/src/book/summary.rs @@ -377,6 +377,10 @@ impl<'a> SummaryParser<'a> { items.push(item); } Some(Event::Start(Tag::List(..))) => { + // Skip this tag after comment bacause it is not nested. + if items.is_empty() { + continue; + } // recurse to parse the nested list let (_, last_item) = get_last_link(&mut items)?; let last_item_number = last_item @@ -696,6 +700,33 @@ mod tests { assert_eq!(got, should_be); } + #[test] + fn parse_numbered_chapters_separated_by_comment() { + let src = "- [First](./first.md)\n\n- [Second](./second.md)"; + + let should_be = vec![ + SummaryItem::Link(Link { + name: String::from("First"), + location: PathBuf::from("./first.md"), + number: Some(SectionNumber(vec![1])), + nested_items: Vec::new(), + }), + SummaryItem::Link(Link { + name: String::from("Second"), + location: PathBuf::from("./second.md"), + number: Some(SectionNumber(vec![2])), + nested_items: Vec::new(), + }), + ]; + + let mut parser = SummaryParser::new(src); + let _ = parser.stream.next(); + + let got = parser.parse_numbered().unwrap(); + + assert_eq!(got, should_be); + } + /// This test ensures the book will continue to pass because it breaks the /// `SUMMARY.md` up using level 2 headers ([example]). ///