diff --git a/app/helpers/lightbox_helper.rb b/app/helpers/lightbox_helper.rb index a31ab8a..01f0ea8 100644 --- a/app/helpers/lightbox_helper.rb +++ b/app/helpers/lightbox_helper.rb @@ -9,7 +9,7 @@ module LightboxHelper end def lightbox_close(text, options = {}) - link_to(text, '#', lightbox_options(options, 'lbAction').merge(:rel => 'deactivate')) + button(:close, text, '#', lightbox_options(options, 'lbAction').merge(:rel => 'deactivate')) end def lightbox_button(type, label, url, options = {}) diff --git a/test/unit/lightbox_helper_text.rb b/test/unit/lightbox_helper_text.rb index 6a3b2d4..5d4b660 100644 --- a/test/unit/lightbox_helper_text.rb +++ b/test/unit/lightbox_helper_text.rb @@ -27,13 +27,13 @@ class LightboxHelperTest < Test::Unit::TestCase end should 'provide link to close lightbox' do - expects(:link_to).with('text', '#', { :class => 'lbAction', :rel => 'deactivate', :id => 'my-id' }).returns('[close-lightbox]') + expects(:button).with(:close, 'text', '#', { :class => 'lbAction', :rel => 'deactivate', :id => 'my-id' }).returns('[close-lightbox]') assert_equal '[close-lightbox]', lightbox_close('text', :id => 'my-id') end should 'merge existing :class option in lightbox_close' do - expects(:link_to).with('text', '#', { :class => 'lbAction my-class', :rel => 'deactivate', :id => 'my-id' }).returns('[close-lightbox]') + expects(:button).with(:close, 'text', '#', { :class => 'lbAction my-class', :rel => 'deactivate', :id => 'my-id' }).returns('[close-lightbox]') assert_equal '[close-lightbox]', lightbox_close('text', :class => 'my-class', :id => 'my-id' ) end -- libgit2 0.21.2