Transform rouge options keys into symbols.

fnando · Feb 20, 2022 · bfa68f0 · bfa68f0
1 parent eafbabb
commit bfa68f0
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,9 @@
 # Changelog
 
+## Unreleased
+
+- Fix Rouge options keys; symbols are required.
+
 ## v3.0.2
 
 - Fix `styles` directory copying.

diff --git a/lib/kitabu/extensions/rouge.rb b/lib/kitabu/extensions/rouge.rb
@@ -5,6 +5,8 @@ module Plugins
     module Redcarpet
       def rouge_formatter(lexer)
         options = lexer.respond_to?(:options) ? lexer.options : {}
+        options = options.keys.map(&:to_sym).zip(options.values).to_h
+        options[:start_line] = options.fetch(:start_line, 1).to_i
 
         Formatters::HTMLLegacy.new(
           {css_class: "highlight #{lexer.tag}"}.merge(options)

diff --git a/spec/kitabu/markdown_spec.rb b/spec/kitabu/markdown_spec.rb
@@ -24,6 +24,17 @@ class User
     expect(html).to include('<span class="k">echo</span>')
   end
 
+  it "enables line numbers" do
+    html = Kitabu::Markdown.render <<-TEXT.strip_heredoc
+    ```ruby?line_numbers=true
+    class User
+    end
+    ```
+    TEXT
+
+    expect(html).to include(%[<table class="rouge-table">])
+  end
+
   it "does not raise with unknown lexers" do
     expect do
       Kitabu::Markdown.render <<-TEXT.strip_heredoc