Remove unused show_landing_strip?
helper method (#29618)
This commit is contained in:
parent
ecdbf15ebe
commit
92855948a0
3 changed files with 1 additions and 35 deletions
|
@ -32,10 +32,6 @@ module ApplicationHelper
|
||||||
paths.any? { |path| current_page?(path) } ? 'active' : ''
|
paths.any? { |path| current_page?(path) } ? 'active' : ''
|
||||||
end
|
end
|
||||||
|
|
||||||
def show_landing_strip?
|
|
||||||
!user_signed_in? && !single_user_mode?
|
|
||||||
end
|
|
||||||
|
|
||||||
def open_registrations?
|
def open_registrations?
|
||||||
Setting.registrations_mode == 'open'
|
Setting.registrations_mode == 'open'
|
||||||
end
|
end
|
||||||
|
|
|
@ -99,36 +99,6 @@ describe ApplicationHelper do
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe 'show_landing_strip?', :without_verify_partial_doubles do
|
|
||||||
describe 'when signed in' do
|
|
||||||
before do
|
|
||||||
allow(helper).to receive(:user_signed_in?).and_return(true)
|
|
||||||
end
|
|
||||||
|
|
||||||
it 'does not show landing strip' do
|
|
||||||
expect(helper.show_landing_strip?).to be false
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe 'when signed out' do
|
|
||||||
before do
|
|
||||||
allow(helper).to receive(:user_signed_in?).and_return(false)
|
|
||||||
end
|
|
||||||
|
|
||||||
it 'does not show landing strip on single user instance' do
|
|
||||||
allow(helper).to receive(:single_user_mode?).and_return(true)
|
|
||||||
|
|
||||||
expect(helper.show_landing_strip?).to be false
|
|
||||||
end
|
|
||||||
|
|
||||||
it 'shows landing strip on multi user instance' do
|
|
||||||
allow(helper).to receive(:single_user_mode?).and_return(false)
|
|
||||||
|
|
||||||
expect(helper.show_landing_strip?).to be true
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe 'available_sign_up_path' do
|
describe 'available_sign_up_path' do
|
||||||
context 'when registrations are closed' do
|
context 'when registrations are closed' do
|
||||||
before do
|
before do
|
||||||
|
|
|
@ -4,7 +4,7 @@ require 'rails_helper'
|
||||||
|
|
||||||
describe 'statuses/show.html.haml', :without_verify_partial_doubles do
|
describe 'statuses/show.html.haml', :without_verify_partial_doubles do
|
||||||
before do
|
before do
|
||||||
allow(view).to receive_messages(api_oembed_url: '', show_landing_strip?: true, site_title: 'example site', site_hostname: 'example.com', full_asset_url: '//asset.host/image.svg', current_account: nil, single_user_mode?: false)
|
allow(view).to receive_messages(api_oembed_url: '', site_title: 'example site', site_hostname: 'example.com', full_asset_url: '//asset.host/image.svg', current_account: nil, single_user_mode?: false)
|
||||||
allow(view).to receive(:local_time)
|
allow(view).to receive(:local_time)
|
||||||
allow(view).to receive(:local_time_ago)
|
allow(view).to receive(:local_time_ago)
|
||||||
assign(:instance_presenter, InstancePresenter.new)
|
assign(:instance_presenter, InstancePresenter.new)
|
||||||
|
|
Loading…
Reference in a new issue