Fix ToolboxBuilder not using li tag

cubesystems · Jan 13, 2015 · e1f7d86 · e1f7d86
1 parent 03d060c
commit e1f7d86
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/releaf-core/app/helpers/releaf/builders/toolbox.rb b/releaf-core/app/helpers/releaf/builders/toolbox.rb
@@ -2,12 +2,14 @@ module Releaf::Builders::Toolbox
 
   def output
     safe_join do
-      items
+      items.map do |item|
+        tag('li', item)
+      end
     end
   end
 
   def items
     []
   end
 
-end
+end
diff --git a/releaf-core/spec/helpers/builders/toolbox_spec.rb b/releaf-core/spec/helpers/builders/toolbox_spec.rb
@@ -16,7 +16,7 @@ class ToolboxTestIncluder
   describe "#output" do
     it "returns safely joined items" do
       allow(subject).to receive(:items).and_return([ '<', ActiveSupport::SafeBuffer.new(">")])
-      expect(subject.output).to eq("&lt;>")
+      expect(subject.output).to eq("<li>&lt;</li><li>></li>")
     end
   end