add unit test for showing error message in case data-markdown is prov…

…ided with data-separator-by-heading
opf · Jul 25, 2024 · 23cad6f · 23cad6f
1 parent 60f69f3
commit 23cad6f
Show file tree

Hide file tree

Showing 4 changed files with 27 additions and 4 deletions.
diff --git a/plugin/awesoMD/awesoMD.esm.js b/plugin/awesoMD/awesoMD.esm.js
diff --git a/plugin/awesoMD/awesoMD.js b/plugin/awesoMD/awesoMD.js
diff --git a/plugin/awesoMD/plugin.js b/plugin/awesoMD/plugin.js
@@ -86,7 +86,7 @@ const plugin = () => {
                 }
             }
 
-            if (animateLists === true) {
+            if (animateLists) {
                 renderer.listitem = (text) => `<li class="fragment">${text}</li>`
             }
 
@@ -207,7 +207,13 @@ const plugin = () => {
             // add slide separator in the case heading indicates the new slide
             if (options.separateByHeading) {
                 if (options.hasDataSeparator) {
-                    return '<section ' + options.attributes + ' data-markdown>' + 'Please do not specify "data-markdown" when "data-separator-by-heading" is used.' + '</section>'
+                    return (
+                        '<section ' +
+                        options.attributes +
+                        ' data-markdown>' +
+                        'Please do not specify "data-markdown" when "data-separator-by-heading" is used.' +
+                        '</section>'
+                    )
                 }
                 options['slideSeparator'] = '---'
                 markdown = this.addSlideSeparator(markdown, options)

diff --git a/tests/unit/jest/extractMetadata.spec.js b/tests/unit/jest/extractMetadata.spec.js
@@ -219,3 +219,20 @@ some content
         expect(updatedMarkdownContent).toEqual(expectedMarkdownContent)
     })
 })
+
+describe('slidify', () => {
+    it('should return error message when data-separator is provided with data-separator-by-heading', () => {
+        const options = {
+            separateByHeading: true,
+            hasDataSeparator: true,
+        }
+        const expectedMarkdownSection =
+            '<section  data-markdown>' +
+            'Please do not specify "data-markdown" when "data-separator-by-heading" is used.' +
+            '</section>'
+
+        const markdownContent = fs.readFileSync('tests/unit/testFiles/noSlideSeparator.md', 'utf-8')
+        const returnedMarkdownSection = mdPlugin.slidify(markdownContent, { ...options })
+        expect(returnedMarkdownSection).toEqual(expectedMarkdownSection)
+    })
+})