Skip to content

Commit

Permalink
Remove unused active_nav_class helper method (mastodon#29617)
Browse files Browse the repository at this point in the history
  • Loading branch information
mjankowski authored and lutoma committed Mar 19, 2024
1 parent 69ea5c4 commit f84cfed
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 28 deletions.
4 changes: 0 additions & 4 deletions app/helpers/application_helper.rb
Expand Up @@ -28,10 +28,6 @@ def friendly_number_to_human(number, **options)
number_to_human(number, **options)
end

def active_nav_class(*paths)
paths.any? { |path| current_page?(path) } ? 'active' : ''
end

def open_registrations?
Setting.registrations_mode == 'open'
end
Expand Down
24 changes: 0 additions & 24 deletions spec/helpers/application_helper_spec.rb
Expand Up @@ -3,30 +3,6 @@
require 'rails_helper'

describe ApplicationHelper do
describe 'active_nav_class' do
it 'returns active when on the current page' do
allow(helper).to receive(:current_page?).and_return(true)

result = helper.active_nav_class('/test')
expect(result).to eq 'active'
end

it 'returns active when on a current page' do
allow(helper).to receive(:current_page?).with('/foo').and_return(false)
allow(helper).to receive(:current_page?).with('/test').and_return(true)

result = helper.active_nav_class('/foo', '/test')
expect(result).to eq 'active'
end

it 'returns empty string when not on current page' do
allow(helper).to receive(:current_page?).and_return(false)

result = helper.active_nav_class('/test')
expect(result).to eq ''
end
end

describe 'body_classes' do
context 'with a body class string from a controller' do
before { helper.extend controller_helpers }
Expand Down

0 comments on commit f84cfed

Please sign in to comment.