From fdeb05ebc8bb9ef4a2dd30490152dbb5ed953987 Mon Sep 17 00:00:00 2001 From: AntonioTerceiro Date: Mon, 10 Sep 2007 19:23:25 +0000 Subject: [PATCH] ActionItem18: creating a proper list on RecentDocumentsBlock --- app/models/recent_documents_block.rb | 2 +- test/unit/recent_documents_block_test.rb | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/app/models/recent_documents_block.rb b/app/models/recent_documents_block.rb index 5e8f2a4..6884794 100644 --- a/app/models/recent_documents_block.rb +++ b/app/models/recent_documents_block.rb @@ -4,7 +4,7 @@ class RecentDocumentsBlock < Design::Block lambda do content_tag( 'ul', - profile.recent_documents.map {|item| link_to_document(item) }.join("\n") + profile.recent_documents.map {|item| content_tag('li', link_to_document(item)) }.join("\n") ) end end diff --git a/test/unit/recent_documents_block_test.rb b/test/unit/recent_documents_block_test.rb index 52fcccd..44cbfa5 100644 --- a/test/unit/recent_documents_block_test.rb +++ b/test/unit/recent_documents_block_test.rb @@ -15,8 +15,11 @@ class RecentDocumentsBlockTest < Test::Unit::TestCase helper = mock helper.expects(:profile).returns(profile) helper.expects(:link_to_document).with(doc1).returns('doc1') + helper.expects(:content_tag).with('li', 'doc1').returns('doc1') helper.expects(:link_to_document).with(doc2).returns('doc2') + helper.expects(:content_tag).with('li', 'doc2').returns('doc2') helper.expects(:link_to_document).with(doc3).returns('doc3') + helper.expects(:content_tag).with('li', 'doc3').returns('doc3') helper.expects(:content_tag).with('ul', "doc1\ndoc2\ndoc3").returns('the_tag') assert_equal('the_tag', helper.instance_eval(&RecentDocumentsBlock.new.content)) -- libgit2 0.21.2