#280: Ignore requirement in Markdown code blocks

doc/changes/changes.md

@@ -1,5 +1,6 @@
 # Changes
 
+* [4.2.0](changes_4.2.0.md)
 * [4.1.0](changes_4.1.0.md)
 * [4.0.2](changes_4.0.2.md)
 * [4.0.1](changes_4.0.1.md)

doc/changes/changes_4.2.0.md

@@ -0,0 +1,11 @@
+# OpenFastTrace 4.2.0, released ???
+
+Code name: Markdown code blocks
+
+## Summary
+
+In this release we changed the behavior of the Markdown importer, so that if we are inside a code block, no OFT specification items are found. This is a corner case, but we think that this behavior is what users would expect (#480). 
+
+## Features
+
+* #480: Ignore specification items inside Markdown code blocks

importer/lightweightmarkup/src/main/java/org/itsallcode/openfasttrace/importer/lightweightmarkup/statemachine/LineParserState.java

@@ -29,6 +29,8 @@ public enum LineParserState
     TITLE,
     /** Found tags */
     TAGS,
+    /** Code block */
+    CODE_BLOCK,
     /** Reached the end of the file */
     EOF
 }

importer/markdown/src/main/java/org/itsallcode/openfasttrace/importer/markdown/MarkdownImporter.java

@@ -42,6 +42,7 @@ protected Transition[] configureTransitions()
                 transition(START      , SPEC_ITEM  , MdPattern.ID         , this::beginItem                    ),
                 transition(START      , TITLE      , SECTION_TITLE        , this::rememberTitle                ),
                 transition(START      , START      , MdPattern.FORWARD    , this::forward                      ),
+                transition(START      , CODE_BLOCK , MdPattern.CODE_BEGIN , () -> {}                            ),
                 transition(START      , START      , MdPattern.EVERYTHING , () -> {}                           ),
 
                 transition(TITLE      , SPEC_ITEM  , MdPattern.ID         , this::beginItem                    ),
@@ -99,7 +100,6 @@ protected Transition[] configureTransitions()
                 transition(COMMENT    , TAGS       , MdPattern.TAGS       , () -> {}                           ),
                 transition(COMMENT    , COMMENT    , MdPattern.EVERYTHING , this::appendComment                ),
 
-
                 // [impl->dsn~md.covers-list~1]
                 transition(COVERS     , SPEC_ITEM  , MdPattern.ID         , this::beginItem                    ),
                 transition(COVERS     , TITLE      , SECTION_TITLE        , () -> {endItem(); rememberTitle();}),
@@ -156,7 +156,9 @@ protected Transition[] configureTransitions()
                 transition(TAGS       , COVERS     , MdPattern.COVERS     , () -> {}                           ),
                 transition(TAGS       , TAGS       , MdPattern.TAGS       , () -> {}                           ),
                 transition(TAGS       , TAGS       , MdPattern.TAGS_INT   , this::addTag                       ),
-                transition(TAGS       , START      , MdPattern.FORWARD    , () -> {endItem(); forward();}      )
+                transition(TAGS       , START      , MdPattern.FORWARD    , () -> {endItem(); forward();}      ),
+
+                transition(CODE_BLOCK , START      , MdPattern.CODE_END   , () -> {}                           )
         };
         // @formatter:on
     }

importer/markdown/src/main/java/org/itsallcode/openfasttrace/importer/markdown/MdPattern.java

@@ -15,6 +15,8 @@ enum MdPattern
     // [impl->dsn~md.artifact-forwarding-notation~1]
 
     // @formatter:off
+    CODE_BEGIN("```\\w*\\s*"),
+    CODE_END("```"),
     COMMENT("Comment:\\s*"),
     COVERS("Covers:\\s*"),
     COVERS_REF(PatternConstants.REFERENCE_AFTER_BULLET),

importer/markdown/src/test/java/org/itsallcode/openfasttrace/importer/markdown/TestMarkdownMarkupImporter.java

@@ -3,6 +3,7 @@
 import static org.itsallcode.matcher.auto.AutoMatcher.contains;
 import static org.itsallcode.openfasttrace.testutil.core.ItemBuilderFactory.item;
 
+import org.hamcrest.Matchers;
 import org.itsallcode.openfasttrace.api.core.SpecificationItemId;
 import org.itsallcode.openfasttrace.api.importer.ImporterFactory;
 import org.itsallcode.openfasttrace.testutil.importer.lightweightmarkup.AbstractLightWeightMarkupImporterTest;
@@ -138,4 +139,37 @@ void testLessThenThreeUnderliningCharactersAreNotDetectedAsTitleUnderlines()
                         .location("z", 3)
                         .build()));
     }
+
+    @Test
+    void testWhenInsideMarkdownCodeBlockThenNoSpecificationItemMustBeDetected() {
+        assertImport("file_with_code_block.md", """
+                ```
+                This is a code block, the following requirement must be ignored.
+
+                req~example~1
+                ```
+                """,
+                Matchers.emptyIterable());
+    }
+
+    // Unit Test
+    @Test
+    void testWhenCodeBlockIsInsideCommentSectionThenItIsImportedAsPartOfComment() {
+        assertImport("file_with_code_block_in_comment.md", """
+                req~comment_with_code_block~1
+                Comment:
+
+                ```
+                This is a code block inside a comment.
+                ```
+                """,
+                contains(item()
+                        .id(SpecificationItemId.createId("req", "comment_with_code_block", 1))
+                        .comment("""
+                                ```
+                                This is a code block inside a comment.
+                                ```""")
+                        .location("file_with_code_block_in_comment.md", 1)
+                        .build()));
+    }
 }