diff options
Diffstat (limited to 'spec')
4 files changed, 10 insertions, 8 deletions
diff --git a/spec/controllers/dashboard/todos_controller_spec.rb b/spec/controllers/dashboard/todos_controller_spec.rb index a8c08a6e87b..288984cfba9 100644 --- a/spec/controllers/dashboard/todos_controller_spec.rb +++ b/spec/controllers/dashboard/todos_controller_spec.rb @@ -13,10 +13,11 @@ describe Dashboard::TodosController do context 'when using pagination' do let(:last_page) { user.todos.page().total_pages } - let!(:issues) { create_list(:issue, 30, project: project, assignee: user) } + let!(:issues) { create_list(:issue, 2, project: project, assignee: user) } before do issues.each { |issue| todo_service.new_issue(issue, user) } + allow(Kaminari.config).to receive(:default_per_page).and_return(1) end it 'redirects to last_page if page number is larger than number of pages' do diff --git a/spec/controllers/projects/issues_controller_spec.rb b/spec/controllers/projects/issues_controller_spec.rb index 60b21cb4ace..e2321f2034b 100644 --- a/spec/controllers/projects/issues_controller_spec.rb +++ b/spec/controllers/projects/issues_controller_spec.rb @@ -55,11 +55,12 @@ describe Projects::IssuesController do context 'with page param' do let(:last_page) { project.issues.page().total_pages } - let!(:issue_list) { create_list(:issue, 30, project: project) } + let!(:issue_list) { create_list(:issue, 2, project: project) } before do sign_in(user) project.team << [user, :developer] + allow(Kaminari.config).to receive(:default_per_page).and_return(1) end it 'redirects to last_page if page number is larger than number of pages' do @@ -68,7 +69,7 @@ describe Projects::IssuesController do project_id: project.path.to_param, page: (last_page + 1).to_param - expect(response).to redirect_to(namespace_project_issues_path(page: last_page)) + expect(response).to redirect_to(namespace_project_issues_path(page: last_page, state: controller.params[:state], scope: controller.params[:scope])) end it 'redirects to specified page' do diff --git a/spec/controllers/projects/merge_requests_controller_spec.rb b/spec/controllers/projects/merge_requests_controller_spec.rb index 92392172af0..2a411d78395 100644 --- a/spec/controllers/projects/merge_requests_controller_spec.rb +++ b/spec/controllers/projects/merge_requests_controller_spec.rb @@ -127,9 +127,6 @@ describe Projects::MergeRequestsController do end describe 'GET index' do - let(:last_page) { project.merge_requests.page().total_pages } - let!(:merge_request) { create(:merge_request_with_diffs, target_project: project, source_project: project) } - def get_merge_requests(page = nil) get :index, namespace_id: project.namespace.to_param, @@ -138,10 +135,13 @@ describe Projects::MergeRequestsController do end context 'when page param' do + let(:last_page) { project.merge_requests.page().total_pages } + let!(:merge_request) { create(:merge_request_with_diffs, target_project: project, source_project: project) } + it 'redirects to last_page if page number is larger than number of pages' do get_merge_requests(last_page + 1) - expect(response).to redirect_to(namespace_project_merge_requests_path(page: last_page)) + expect(response).to redirect_to(namespace_project_merge_requests_path(page: last_page, state: controller.params[:state], scope: controller.params[:scope])) end it 'redirects to specified page' do diff --git a/spec/controllers/projects/snippets_controller_spec.rb b/spec/controllers/projects/snippets_controller_spec.rb index 82e57e4d7dc..32b0e42c3cd 100644 --- a/spec/controllers/projects/snippets_controller_spec.rb +++ b/spec/controllers/projects/snippets_controller_spec.rb @@ -26,7 +26,7 @@ describe Projects::SnippetsController do it 'redirects to specified page' do get :index, namespace_id: project.namespace.path, - project_id: project.path, page: (last_page).to_param + project_id: project.path, page: last_page.to_param expect(assigns(:snippets).current_page).to eq(last_page) expect(response).to have_http_status(200) |