Remove unused active_nav_class helper method (#29617)
				
					
				
			This commit is contained in:
		
							parent
							
								
									d5063072c3
								
							
						
					
					
						commit
						6d2986017e
					
				@ -28,10 +28,6 @@ module ApplicationHelper
 | 
				
			|||||||
    number_to_human(number, **options)
 | 
					    number_to_human(number, **options)
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
					
 | 
				
			||||||
  def active_nav_class(*paths)
 | 
					 | 
				
			||||||
    paths.any? { |path| current_page?(path) } ? 'active' : ''
 | 
					 | 
				
			||||||
  end
 | 
					 | 
				
			||||||
 | 
					 | 
				
			||||||
  def open_registrations?
 | 
					  def open_registrations?
 | 
				
			||||||
    Setting.registrations_mode == 'open'
 | 
					    Setting.registrations_mode == 'open'
 | 
				
			||||||
  end
 | 
					  end
 | 
				
			||||||
 | 
				
			|||||||
@ -3,30 +3,6 @@
 | 
				
			|||||||
require 'rails_helper'
 | 
					require 'rails_helper'
 | 
				
			||||||
 | 
					
 | 
				
			||||||
describe ApplicationHelper do
 | 
					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
 | 
					  describe 'body_classes' do
 | 
				
			||||||
    context 'with a body class string from a controller' do
 | 
					    context 'with a body class string from a controller' do
 | 
				
			||||||
      before { helper.extend controller_helpers }
 | 
					      before { helper.extend controller_helpers }
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user