Skip to content

Commit dc22749

Browse files
committed
Address PR feedback
Test Plan: rspec
1 parent 4ef5be7 commit dc22749

File tree

3 files changed

+7
-6
lines changed

3 files changed

+7
-6
lines changed

lib/split/configuration.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ class Configuration
2525
attr_accessor :beta_probability_simulations
2626
attr_accessor :winning_alternative_recalculation_interval
2727
attr_accessor :redis
28-
attr_accessor :dashboard_pagination_default_per
28+
attr_accessor :dashboard_pagination_default_per_page
2929

3030
attr_reader :experiments
3131

@@ -227,7 +227,7 @@ def initialize
227227
@beta_probability_simulations = 10000
228228
@winning_alternative_recalculation_interval = 60 * 60 * 24 # 1 day
229229
@redis = ENV.fetch(ENV.fetch('REDIS_PROVIDER', 'REDIS_URL'), 'redis://localhost:6379')
230-
@dashboard_pagination_default_per = 10
230+
@dashboard_pagination_default_per_page = 10
231231
end
232232

233233
def redis_url=(value)

lib/split/dashboard/pagination_helpers.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,8 @@
44
module Split
55
module DashboardPaginationHelpers
66
def pagination_per
7-
default_per = Split.configuration.dashboard_pagination_default_per
8-
@pagination_per ||= (params[:per] || default_per).to_i
7+
default_per_page = Split.configuration.dashboard_pagination_default_per_page
8+
@pagination_per ||= (params[:per] || default_per_page).to_i
99
end
1010

1111
def page_number

spec/dashboard/pagination_helpers_spec.rb

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,9 @@
1010
context 'when params empty' do
1111
let(:params) { Hash[] }
1212

13-
it 'returns 10' do
14-
expect(pagination_per).to eql 10
13+
it 'returns the default' do
14+
default_per_page = Split.configuration.dashboard_pagination_default_per_page
15+
expect(pagination_per).to eql default_per_page
1516
end
1617
end
1718

0 commit comments

Comments
 (0)