Syntax fixes in markdown test helpers

Author: webstackdev

src/lib/markdown/README.md

@@ -71,6 +71,8 @@ Working plugins that pass all tests:
 - rehype-autolink-headings
 - rehype-tailwind-classes
 
+### Rehype Tailwind Classes
+
 Conditional elements (have checks):
 
 - a (checks for .btn and .heading-anchor)
@@ -81,6 +83,45 @@ Conditional elements (have checks):
 - iframe
 - various special cases
 
+### Duplicate test code across units, units_with_default_astro, and e2e
+
+- rehypeAccessibleEmojis.spec.tsx
+
+Summary: What Should Change
+
+Remove:
+
+Layer 2: "should use custom footnote labels from Astro config" test (doesn't test emojis)
+
+Add to Layer 2 (Astro):
+
+Test emoji shortcodes (:heart:) in addition to Unicode
+"Text without emojis" edge case
+DOM-based validation with axe (optional, but valuable)
+Empty aria-label verification
+
+Add to Layer 4 (E2E):
+
+Unicode emoji tests in addition to shortcodes
+"Text without emojis" edge case
+Expand emoji.md fixture to include more edge cases
+
+Keep Layer 1 (Units) as-is:
+
+It's focused and tests the isolated plugin correctly
+Uses Unicode emojis which is appropriate
+Has good edge case coverage
+
+Fixture Enhancement Needed
+
+The emoji.md fixture should include:
+
+Unicode emojis (not just shortcodes)
+Text without emojis
+Emoji in headings
+Emoji in various list types
+More diverse emoji types
+
 ## Markdown Config Updates
 
 Details/Summary elements - These HTML elements aren't being processed by remarkGfm (they need to be raw HTML)

src/lib/markdown/__tests__/e2e/__fixtures__/attribution.md

@@ -2,6 +2,7 @@
 title: Attribution Test
 ---
 
+    <!-- markdownlint-disable -->
 # Attribution Feature Test
 
 > That's one small step for man, one giant leap for mankind.

src/lib/markdown/helpers/index.ts

@@ -0,0 +1,15 @@
+/**
+ * Barrel file for markdown helper utilities
+ *
+ * This module re-exports all helper functions for markdown processing,
+ * fixture loading, and test utilities.
+ */
+
+// Markdown fixture loader utilities for E2E testing
+export { loadFixture, MarkdownOutput, renderFixture, renderMarkdown } from './markdownLoader'
+
+// Markdown pipeline processor for testing
+export { renderMarkdown as processMarkdownPipeline } from './pipeline'
+
+// Test utilities for different pipeline layers
+export { processIsolated, processWithAstroSettings, processWithFullPipeline } from './test-utils'

src/lib/markdown/helpers/pipeline.ts

@@ -13,7 +13,7 @@ import remarkGfm from 'remark-gfm'
 import remarkRehype from 'remark-rehype'
 import rehypeStringify from 'rehype-stringify'
 import rehypeSlug from 'rehype-slug'
-import { markdownConfig } from '../../config/markdown'
+import { markdownConfig } from '@lib/config/markdown'
 
 /**
  * Renders markdown content through the complete Astro pipeline

src/lib/markdown/helpers/test-utils.ts

@@ -23,7 +23,10 @@ export async function processIsolated(
     processor.use(plugin)
   }
 
-  const result = await processor.use(remarkRehype).use(rehypeStringify).process(markdown)
+  const result = await processor
+    .use(remarkRehype)
+    .use(rehypeStringify)
+    .process(markdown)
 
   return String(result)
 }

src/lib/markdown/plugins/rehype-tailwind/__tests__/html.spec.ts

@@ -1,12 +1,12 @@
 import { describe, expect, test } from 'vitest'
 import type { Element } from 'hast'
-import type { ElementConfig } from '../@types/index.js'
+import type { ElementConfig } from '../@types/index'
 import {
   htmlElements,
   applyHtmlElementClasses,
   isSimpleHtmlElement,
   getElementConfig,
-} from '../visitors/simple.js'
+} from '../visitors/simple'
 
 describe('htmlElements configuration', () => {
   test('contains expected number of element configurations', () => {