Refactored tests and added a breakpoint to the merge_request_tabs
This commit is contained in:
parent
502bd6c033
commit
a7335c1188
|
@ -144,6 +144,9 @@ import BlobForkSuggestion from './blob/blob_fork_suggestion';
|
|||
this.resetViewContainer();
|
||||
this.mountPipelinesView();
|
||||
} else {
|
||||
if (Breakpoints.get().getBreakpointSize() !== 'xs') {
|
||||
this.expandView();
|
||||
}
|
||||
this.resetViewContainer();
|
||||
this.destroyPipelinesView();
|
||||
}
|
||||
|
|
|
@ -0,0 +1,30 @@
|
|||
require 'rails_helper'
|
||||
|
||||
describe 'Issue Sidebar on Mobile' do
|
||||
include MobileHelpers
|
||||
|
||||
let(:project) { create(:project, :public) }
|
||||
let(:merge_request) { create(:merge_request, source_project: project) }
|
||||
let(:issue) { create(:issue, project: project) }
|
||||
let!(:user) { create(:user)}
|
||||
|
||||
before do
|
||||
sign_in(user)
|
||||
end
|
||||
|
||||
context 'mobile sidebar on merge requests', js: true do
|
||||
before do
|
||||
visit namespace_project_merge_request_path(merge_request.project.namespace, merge_request.project, merge_request)
|
||||
end
|
||||
|
||||
it_behaves_like "issue sidebar stays collapsed on mobile"
|
||||
end
|
||||
|
||||
context 'mobile sidebar on issues', js: true do
|
||||
before do
|
||||
visit namespace_project_issue_path(project.namespace, project, issue)
|
||||
end
|
||||
|
||||
it_behaves_like "issue sidebar stays collapsed on mobile"
|
||||
end
|
||||
end
|
|
@ -6,7 +6,6 @@ feature 'Issue Sidebar', feature: true do
|
|||
let(:group) { create(:group, :nested) }
|
||||
let(:project) { create(:project, :public, namespace: group) }
|
||||
let(:issue) { create(:issue, project: project) }
|
||||
let(:merge_request) { create(:merge_request, source_project: project) }
|
||||
let!(:user) { create(:user)}
|
||||
let!(:label) { create(:label, project: project, title: 'bug') }
|
||||
|
||||
|
@ -155,22 +154,6 @@ feature 'Issue Sidebar', feature: true do
|
|||
end
|
||||
end
|
||||
|
||||
context 'as a allowed mobile user', js: true do
|
||||
before do
|
||||
project.team << [user, :developer]
|
||||
resize_screen_xs
|
||||
end
|
||||
|
||||
context 'mobile sidebar' do
|
||||
it 'collapses the sidebar for small screens on an issue/merge_request' do
|
||||
visit_issue(project, issue)
|
||||
expect(page).not_to have_css('aside.right-sidebar.right-sidebar-collapsed')
|
||||
visit namespace_project_merge_request_path(merge_request.project.namespace, merge_request.project, merge_request)
|
||||
expect(page).not_to have_css('aside.right-sidebar.right-sidebar-collapsed')
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
context 'as a guest' do
|
||||
before do
|
||||
project.team << [user, :guest]
|
||||
|
|
|
@ -0,0 +1,9 @@
|
|||
shared_examples 'issue sidebar stays collapsed on mobile' do
|
||||
before do
|
||||
resize_screen_xs
|
||||
end
|
||||
|
||||
it 'keeps the sidebar collapsed' do
|
||||
expect(page).not_to have_css('.right-sidebar.right-sidebar-collapsed')
|
||||
end
|
||||
end
|
Loading…
Reference in New Issue