diff options
Diffstat (limited to 'spec/views')
8 files changed, 2 insertions, 1415 deletions
diff --git a/spec/views/layouts/fullscreen.html.haml_spec.rb b/spec/views/layouts/fullscreen.html.haml_spec.rb index 46c3b08b954..c763b53ac6c 100644 --- a/spec/views/layouts/fullscreen.html.haml_spec.rb +++ b/spec/views/layouts/fullscreen.html.haml_spec.rb @@ -27,38 +27,4 @@ RSpec.describe 'layouts/fullscreen' do it_behaves_like 'a layout which reflects the application theme setting' it_behaves_like 'a layout which reflects the preferred language' - - describe 'sidebar' do - context 'when nav is set' do - before do - allow(view).to receive(:nav).and_return("admin") - render - end - - it 'renders the sidebar' do - expect(rendered).to render_template("layouts/nav/sidebar/_admin") - expect(rendered).to have_selector("aside.nav-sidebar") - end - - it 'adds the proper classes' do - expect(rendered).to have_selector(".layout-page.gl-mt-0\\!") - end - end - - describe 'when nav is not set' do - before do - allow(view).to receive(:nav).and_return(nil) - render - end - - it 'does not render the sidebar' do - expect(rendered).not_to render_template("layouts/nav/sidebar/_admin") - expect(rendered).not_to have_selector("aside.nav-sidebar") - end - - it 'not add classes' do - expect(rendered).not_to have_selector(".layout-page.gl-mt-0\\!") - end - end - end end diff --git a/spec/views/layouts/nav/sidebar/_admin.html.haml_spec.rb b/spec/views/layouts/nav/sidebar/_admin.html.haml_spec.rb deleted file mode 100644 index 3097598aaca..00000000000 --- a/spec/views/layouts/nav/sidebar/_admin.html.haml_spec.rb +++ /dev/null @@ -1,161 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe 'layouts/nav/sidebar/_admin', feature_category: :navigation do - let(:user) { build(:admin) } - - before do - allow(user).to receive(:can_admin_all_resources?).and_return(true) - allow(view).to receive(:current_user).and_return(user) - end - - shared_examples 'page has active tab' do |title| - it "activates #{title} tab" do - render - - expect(rendered).to have_selector('.nav-sidebar .sidebar-top-level-items > li.active', count: 1) - expect(rendered).to have_css('.nav-sidebar .sidebar-top-level-items > li.active', text: title) - end - end - - shared_examples 'page has active sub tab' do |title| - it "activates #{title} sub tab" do - render - - expect(rendered).to have_css('.sidebar-sub-level-items > li.active', text: title) - end - end - - context 'on home page' do - before do - allow(controller).to receive(:controller_name).and_return('dashboard') - end - - it_behaves_like 'page has active tab', 'Overview' - end - - it_behaves_like 'has nav sidebar' - - context 'on projects' do - before do - allow(controller).to receive(:controller_name).and_return('admin/projects') - allow(controller).to receive(:controller_path).and_return('admin/projects') - end - - it_behaves_like 'page has active tab', 'Overview' - it_behaves_like 'page has active sub tab', 'Projects' - end - - context 'on groups' do - before do - allow(controller).to receive(:controller_name).and_return('groups') - end - - it_behaves_like 'page has active tab', 'Overview' - it_behaves_like 'page has active sub tab', 'Groups' - end - - context 'on users' do - before do - allow(controller).to receive(:controller_name).and_return('users') - end - - it_behaves_like 'page has active tab', 'Overview' - it_behaves_like 'page has active sub tab', 'Users' - end - - context 'on topics' do - before do - allow(controller).to receive(:controller_name).and_return('admin/topics') - end - - it_behaves_like 'page has active tab', 'Overview' - it_behaves_like 'page has active sub tab', 'Topics' - end - - context 'on runners' do - before do - allow(controller).to receive(:controller_name).and_return('runners') - end - - it_behaves_like 'page has active tab', 'CI/CD' - it_behaves_like 'page has active sub tab', 'Runners' - end - - context 'on jobs' do - before do - allow(controller).to receive(:controller_name).and_return('jobs') - end - - it_behaves_like 'page has active tab', 'CI/CD' - it_behaves_like 'page has active sub tab', 'Jobs' - end - - context 'on messages' do - before do - allow(controller).to receive(:controller_name).and_return('broadcast_messages') - end - - it_behaves_like 'page has active tab', 'Messages' - end - - context 'on analytics' do - before do - allow(controller).to receive(:controller_name).and_return('dev_ops_report') - end - - it_behaves_like 'page has active tab', 'Analytics' - end - - context 'on hooks' do - before do - allow(controller).to receive(:controller_name).and_return('hooks') - end - - it_behaves_like 'page has active tab', 'Hooks' - end - - context 'on background jobs' do - before do - allow(controller).to receive(:controller_name).and_return('background_jobs') - end - - it_behaves_like 'page has active tab', 'Monitoring' - it_behaves_like 'page has active sub tab', 'Background Jobs' - end - - context 'on settings' do - let(:gitlab_com?) { false } - - before do - allow(::Gitlab).to receive(:com?) { gitlab_com? } - - render - end - - it 'includes General link' do - expect(rendered).to have_link('General', href: general_admin_application_settings_path) - end - - context 'when GitLab.com' do - let(:gitlab_com?) { true } - - it 'does not include Integrations link' do - expect(rendered).not_to have_link('Integrations', href: integrations_admin_application_settings_path) - end - end - - context 'when not GitLab.com' do - it 'includes Integrations link' do - expect(rendered).to have_link('Integrations', href: integrations_admin_application_settings_path) - end - end - - context 'when GitLab FOSS' do - it 'does not include Templates link' do - expect(rendered).not_to have_link('Templates', href: '/admin/application_settings/templates') - end - end - end -end diff --git a/spec/views/layouts/nav/sidebar/_group.html.haml_spec.rb b/spec/views/layouts/nav/sidebar/_group.html.haml_spec.rb deleted file mode 100644 index 472a2f3cb34..00000000000 --- a/spec/views/layouts/nav/sidebar/_group.html.haml_spec.rb +++ /dev/null @@ -1,187 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe 'layouts/nav/sidebar/_group' do - let_it_be(:owner) { create(:user) } - let_it_be(:group) do - create(:group).tap do |g| - g.add_owner(owner) - end - end - - before do - assign(:group, group) - - allow(view).to receive(:current_user).and_return(owner) - end - - it_behaves_like 'has nav sidebar' - it_behaves_like 'sidebar includes snowplow attributes', 'render', 'groups_side_navigation', 'groups_side_navigation' - - describe 'Group context menu' do - it 'has a link to the group path' do - render - - expect(rendered).to have_link(group.name, href: group_path(group)) - end - end - - describe 'Group information' do - it 'has a link to the group activity path' do - render - - expect(rendered).to have_link('Group information', href: activity_group_path(group)) - end - - it 'has a link to the group labels path' do - render - - expect(rendered).to have_link('Labels', href: group_labels_path(group)) - end - - it 'has a link to the members page' do - render - - expect(rendered).to have_link('Members', href: group_group_members_path(group)) - end - end - - describe 'Issues' do - it 'has a default link to the issue list path' do - render - - expect(rendered).to have_link('Issues', href: issues_group_path(group)) - end - - it 'has a link to the issue list page' do - render - - expect(rendered).to have_link('List', href: issues_group_path(group)) - end - - it 'has a link to the boards page' do - render - - expect(rendered).to have_link('Board', href: group_boards_path(group)) - end - - it 'has a link to the milestones page' do - render - - expect(rendered).to have_link('Milestones', href: group_milestones_path(group)) - end - end - - describe 'Merge Requests' do - it 'has a link to the merge request list path' do - render - - expect(rendered).to have_link('Merge requests', href: merge_requests_group_path(group)) - end - - it 'shows pill with the number of merge requests' do - render - - expect(rendered).to have_css('span.badge.badge-pill.merge_counter.js-merge-counter') - end - end - - describe 'CI/CD' do - it 'has a default link to the runners list path' do - render - - expect(rendered).to have_link('CI/CD', href: group_runners_path(group)) - end - - it 'has a link to the runners list page' do - render - - expect(rendered).to have_link('Runners', href: group_runners_path(group)) - end - end - - describe 'Kubernetes menu', :request_store do - it 'has a link to the group cluster list path' do - render - - expect(rendered).to have_link('Kubernetes', href: group_clusters_path(group)) - end - end - - describe 'Packages and registries' do - it 'has a link to the package registry page' do - stub_config(packages: { enabled: true }) - - render - - expect(rendered).to have_link('Package Registry', href: group_packages_path(group)) - end - - it 'has a link to the container registry page' do - stub_container_registry_config(enabled: true) - - render - - expect(rendered).to have_link('Container Registry', href: group_container_registries_path(group)) - end - - it 'has a link to the dependency proxy page' do - stub_config(dependency_proxy: { enabled: true }) - - render - - expect(rendered).to have_link('Dependency Proxy', href: group_dependency_proxy_path(group)) - end - end - - describe 'Settings' do - it 'default link points to edit group page' do - render - - expect(rendered).to have_link('Settings', href: edit_group_path(group)) - end - - it 'has a link to the General settings page' do - render - - expect(rendered).to have_link('General', href: edit_group_path(group)) - end - - it 'has a link to the Integrations settings page' do - render - - expect(rendered).to have_link('Integrations', href: group_settings_integrations_path(group)) - end - - it 'has a link to the group Projects settings page' do - render - - expect(rendered).to have_link('Projects', href: projects_group_path(group)) - end - - it 'has a link to the Repository settings page' do - render - - expect(rendered).to have_link('Repository', href: group_settings_repository_path(group)) - end - - it 'has a link to the CI/CD settings page' do - render - - expect(rendered).to have_link('CI/CD', href: group_settings_ci_cd_path(group)) - end - - it 'has a link to the Applications settings page' do - render - - expect(rendered).to have_link('Applications', href: group_settings_applications_path(group)) - end - - it 'has a link to the Package and registry settings page' do - render - - expect(rendered).to have_link('Packages and registries', href: group_settings_packages_and_registries_path(group)) - end - end -end diff --git a/spec/views/layouts/nav/sidebar/_profile.html.haml_spec.rb b/spec/views/layouts/nav/sidebar/_profile.html.haml_spec.rb deleted file mode 100644 index f5a0a7a935c..00000000000 --- a/spec/views/layouts/nav/sidebar/_profile.html.haml_spec.rb +++ /dev/null @@ -1,30 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe 'layouts/nav/sidebar/_profile' do - let(:user) { create(:user) } - - before do - allow(view).to receive(:current_user).and_return(user) - end - - it_behaves_like 'has nav sidebar' - it_behaves_like 'sidebar includes snowplow attributes', 'render', 'user_side_navigation', 'user_side_navigation' - - it 'has a link to access tokens' do - render - - expect(rendered).to have_link(_('Access Tokens'), href: profile_personal_access_tokens_path) - end - - context 'when personal access tokens are disabled' do - it 'does not have a link to access tokens' do - allow(::Gitlab::CurrentSettings).to receive_messages(personal_access_tokens_disabled?: true) - - render - - expect(rendered).not_to have_link(_('Access Tokens'), href: profile_personal_access_tokens_path) - end - end -end diff --git a/spec/views/layouts/nav/sidebar/_project.html.haml_spec.rb b/spec/views/layouts/nav/sidebar/_project.html.haml_spec.rb deleted file mode 100644 index 34debcab5f7..00000000000 --- a/spec/views/layouts/nav/sidebar/_project.html.haml_spec.rb +++ /dev/null @@ -1,958 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe 'layouts/nav/sidebar/_project', feature_category: :navigation do - let_it_be_with_reload(:project) { create(:project, :repository) } - - let(:user) { project.first_owner } - let(:current_ref) { 'master' } - - before do - assign(:project, project) - assign(:repository, project.repository) - - allow(view).to receive(:current_ref).and_return(current_ref) - allow(view).to receive(:can?).and_return(true) - allow(view).to receive(:current_user).and_return(user) - end - - it_behaves_like 'has nav sidebar' - - describe 'Project context' do - it 'has a link to the project path' do - render - - expect(rendered).to have_link(project.name, href: project_path(project), class: 'shortcuts-project') - expect(rendered).to have_selector("[aria-label=\"#{project.name}\"]") - end - end - - describe 'Project information' do - it 'has a link to the project activity path' do - render - - expect(rendered).to have_link('Project information', href: activity_project_path(project), class: %w[shortcuts-project-information]) - expect(rendered).to have_selector('[aria-label="Project information"]') - end - - describe 'Activity' do - it 'has a link to the project activity path' do - render - - expect(rendered).to have_link('Activity', href: activity_project_path(project), class: 'shortcuts-project-activity') - end - end - - describe 'Labels' do - let(:page) { Nokogiri::HTML.parse(rendered) } - - it 'has a link to the labels path' do - render - - expect(page.at_css('.shortcuts-project-information').parent.css('[aria-label="Labels"]')).not_to be_empty - expect(rendered).to have_link('Labels', href: project_labels_path(project)) - end - end - - describe 'Members' do - let(:page) { Nokogiri::HTML.parse(rendered) } - - it 'has a link to the members page' do - render - - expect(page.at_css('.shortcuts-project-information').parent.css('[aria-label="Members"]')).not_to be_empty - expect(rendered).to have_link('Members', href: project_project_members_path(project)) - end - end - end - - describe 'Repository' do - it 'has a link to the project tree path' do - render - - expect(rendered).to have_link('Repository', href: project_tree_path(project, current_ref), class: 'shortcuts-tree') - end - - describe 'Files' do - it 'has a link to the project tree path' do - render - - expect(rendered).to have_link('Files', href: project_tree_path(project, current_ref)) - end - end - - describe 'Commits' do - it 'has a link to the fully qualified project commits path' do - render - - expect(rendered).to have_link('Commits', href: project_commits_path(project, current_ref, ref_type: 'heads'), id: 'js-onboarding-commits-link') - end - end - - describe 'Branches' do - it 'has a link to the project branches path' do - render - - expect(rendered).to have_link('Branches', href: project_branches_path(project), id: 'js-onboarding-branches-link') - end - end - - describe 'Tags' do - it 'has a link to the project tags path' do - render - - expect(rendered).to have_link('Tags', href: project_tags_path(project)) - end - end - - describe 'Contributor statistics' do - it 'has a link to the project contributors path' do - render - - expect(rendered).to have_link('Contributor statistics', href: project_graph_path(project, current_ref, ref_type: 'heads')) - end - end - - describe 'Graph' do - it 'has a link to the project graph path' do - render - - expect(rendered).to have_link('Graph', href: project_network_path(project, current_ref)) - end - end - - describe 'Compare revisions' do - it 'has a link to the project compare path' do - render - - expect(rendered).to have_link('Compare revisions', href: project_compare_index_path(project, from: project.repository.root_ref, to: current_ref)) - end - end - end - - describe 'Issues' do - it 'has a link to the issue list path' do - render - - expect(rendered).to have_link('Issues', href: project_issues_path(project)) - end - - it 'shows pill with the number of open issues' do - render - - expect(rendered).to have_css('span.badge.badge-pill.issue_counter') - end - - describe 'Issue List' do - it 'has a link to the issue list path' do - render - - expect(rendered).to have_link('List', href: project_issues_path(project)) - end - end - - describe 'Issue Boards' do - it 'has a link to the issue boards path' do - render - - expect(rendered).to have_link('Boards', href: project_boards_path(project)) - end - end - - describe 'Service Desk' do - it 'has a link to the service desk path' do - render - - expect(rendered).to have_link('Service Desk', href: service_desk_project_issues_path(project)) - end - end - - describe 'Milestones' do - it 'has a link to the milestones path' do - render - - expect(rendered).to have_link('Milestones', href: project_milestones_path(project)) - end - end - end - - describe 'External Issue Tracker' do - let_it_be_with_refind(:project) { create(:project, has_external_issue_tracker: true) } - - context 'with custom external issue tracker' do - let(:external_issue_tracker_url) { 'http://test.com' } - - let!(:external_issue_tracker) do - create(:custom_issue_tracker_integration, active: external_issue_tracker_active, project: project, project_url: external_issue_tracker_url) - end - - context 'when external issue tracker is configured and active' do - let(:external_issue_tracker_active) { true } - - it 'has a link to the external issue tracker' do - render - - expect(rendered).to have_link(external_issue_tracker.title, href: external_issue_tracker_url) - end - end - - context 'when external issue tracker is not configured and active' do - let(:external_issue_tracker_active) { false } - - it 'does not have a link to the external issue tracker' do - render - - expect(rendered).not_to have_link(external_issue_tracker.title) - end - end - end - - context 'with Jira issue tracker' do - let_it_be(:jira) { create(:jira_integration, project: project, issues_enabled: false) } - - it 'has a link to the Jira issue tracker' do - render - - expect(rendered).to have_link('Jira', href: project.external_issue_tracker.issue_tracker_path) - end - end - end - - describe 'Merge Requests' do - it 'has a link to the merge request list path' do - render - - expect(rendered).to have_link('Merge requests', href: project_merge_requests_path(project), class: 'shortcuts-merge_requests') - end - - it 'shows pill with the number of merge requests' do - render - - expect(rendered).to have_css('span.badge.badge-pill.merge_counter.js-merge-counter') - end - end - - describe 'CI/CD' do - it 'has a link to pipelines page' do - render - - expect(rendered).to have_link('CI/CD', href: project_pipelines_path(project)) - end - - describe 'Artifacts' do - it 'has a link to the artifacts page' do - render - - expect(rendered).to have_link('Artifacts', href: project_artifacts_path(project)) - end - end - - describe 'Jobs' do - it 'has a link to the jobs page' do - render - - expect(rendered).to have_link('Jobs', href: project_jobs_path(project)) - end - end - - describe 'Pipeline Schedules' do - it 'has a link to the pipeline schedules page' do - render - - expect(rendered).to have_link('Schedules', href: pipeline_schedules_path(project)) - end - end - - describe 'Pipelines' do - it 'has a link to the pipelines page' do - render - - expect(rendered).to have_link('Pipelines', href: project_pipelines_path(project)) - end - end - - describe 'Pipeline Editor' do - context 'with a current_ref' do - it 'has a link to the pipeline editor' do - render - - expect(rendered).to have_link('Editor', href: project_ci_pipeline_editor_path(project, params: { branch_name: current_ref })) - end - end - - context 'with the default_branch' do - it 'has a link to the pipeline editor' do - render - - expect(rendered).to have_link('Editor', href: project_ci_pipeline_editor_path(project, params: { branch_name: project.default_branch })) - end - end - - context 'when user cannot access pipeline editor' do - it 'does not has a link to the pipeline editor' do - allow(view).to receive(:can_view_pipeline_editor?).and_return(false) - - render - - expect(rendered).not_to have_link('Editor', href: project_ci_pipeline_editor_path(project)) - end - end - end - end - - describe 'Security and Compliance' do - describe 'when user does not have permissions' do - before do - allow(view).to receive(:current_user).and_return(nil) - end - - it 'top level navigation link is not visible' do - render - - expect(rendered).not_to have_link('Security and Compliance') - end - end - - context 'when user has permissions' do - before do - allow(view).to receive(:current_user).and_return(user) - - render - end - - it 'top level navigation link is visible' do - expect(rendered).to have_link('Security and Compliance') - end - - it 'security configuration link is visible' do - expect(rendered).to have_link('Security configuration', href: project_security_configuration_path(project)) - end - end - end - - describe 'Deployments' do - let(:page) { Nokogiri::HTML.parse(rendered) } - - describe 'Feature flags' do - it 'has a link to the feature flags page' do - render - - expect(page.at_css('.shortcuts-deployments').parent.css('[aria-label="Feature flags"]')).not_to be_empty - expect(rendered).to have_link('Feature flags', href: project_feature_flags_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the feature flags page' do - render - - expect(rendered).not_to have_link('Feature flags') - end - end - end - - describe 'Environments' do - it 'has a link to the environments page' do - render - - expect(page.at_css('.shortcuts-deployments').parent.css('[aria-label="Environments"]')).not_to be_empty - expect(rendered).to have_link('Environments', href: project_environments_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the environments page' do - render - - expect(rendered).not_to have_link('Environments') - end - end - end - - describe 'Releases' do - it 'has a link to the project releases path' do - render - - expect(rendered).to have_link('Releases', href: project_releases_path(project), class: 'shortcuts-deployments-releases') - end - end - end - - describe 'Monitor' do - it 'top level navigation link is visible for user with permissions' do - render - - expect(rendered).to have_link('Monitor') - end - - describe 'Error Tracking' do - it 'has a link to the error tracking page' do - render - - expect(rendered).to have_link('Error Tracking', href: project_error_tracking_index_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the error tracking page' do - render - - expect(rendered).not_to have_link('Error Tracking') - end - end - end - - describe 'Alert Management' do - it 'has a link to the alert management page' do - render - - expect(rendered).to have_link('Alerts', href: project_alert_management_index_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the alert management page' do - render - - expect(rendered).not_to have_link('Alerts') - end - end - end - - describe 'Incidents' do - it 'has a link to the incidents page' do - render - - expect(rendered).to have_link('Incidents', href: project_incidents_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the incidents page' do - render - - expect(rendered).not_to have_link('Incidents') - end - end - end - end - - describe 'Infrastructure' do - describe 'Terraform' do - it 'has a link to the terraform page' do - render - - expect(rendered).to have_link('Terraform states', href: project_terraform_index_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the terraform page' do - render - - expect(rendered).not_to have_link('Terraform states') - end - end - end - - describe 'Kubernetes clusters' do - it 'has a link to the kubernetes page' do - render - - expect(rendered).to have_link('Kubernetes clusters', href: project_clusters_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the kubernetes page' do - render - - expect(rendered).not_to have_link('Kubernetes clusters') - end - end - end - - describe 'Google Cloud' do - it 'has a link to the google cloud page' do - render - expect(rendered).to have_link('Google Cloud', href: project_google_cloud_configuration_path(project)) - end - - describe 'when the user does not have access' do - let(:user) { nil } - - it 'does not have a link to the google cloud page' do - render - - expect(rendered).not_to have_link('Google Cloud') - end - end - end - end - - describe 'Packages and Registries' do - let(:registry_enabled) { true } - let(:packages_enabled) { true } - - before do - stub_container_registry_config(enabled: registry_enabled) - stub_config(packages: { enabled: packages_enabled }) - end - - it 'top level navigation link is visible and points to package registry page' do - render - - expect(rendered).to have_link('Packages and registries', href: project_packages_path(project)) - end - - describe 'Packages Registry' do - it 'shows link to package registry page' do - render - - expect(rendered).to have_link('Package Registry', href: project_packages_path(project)) - end - - context 'when packages config setting is not enabled' do - let(:packages_enabled) { false } - - it 'does not show link to package registry page' do - render - - expect(rendered).not_to have_link('Package Registry', href: project_packages_path(project)) - end - end - end - - describe 'Container Registry' do - it 'shows link to container registry page' do - render - - expect(rendered).to have_link('Container Registry', href: project_container_registry_index_path(project)) - end - - context 'when container config setting is not enabled' do - let(:registry_enabled) { false } - - it 'does not show link to package registry page' do - render - - expect(rendered).not_to have_link('Container Registry', href: project_container_registry_index_path(project)) - end - end - end - - describe 'Terraform modules' do - it 'shows link to terraform modules page' do - render - - expect(rendered).to have_link('Terraform modules', href: project_infrastructure_registry_index_path(project)) - end - - context 'when package registry config is disabled' do - it 'does not show link to package registry page' do - stub_config(packages: { enabled: false }) - - render - - expect(rendered).not_to have_link('Terraform modules', href: project_infrastructure_registry_index_path(project)) - end - end - end - end - - describe 'Analytics' do - it 'top level navigation link is visible points to the value stream page' do - render - - expect(rendered).to have_link('Analytics', href: project_cycle_analytics_path(project)) - end - - describe 'CI/CD' do - it 'has a link to the CI/CD analytics page' do - render - - expect(rendered).to have_link('CI/CD', href: charts_project_pipelines_path(project)) - end - - context 'when user does not have access' do - let(:user) { nil } - - it 'does not have a link to the CI/CD analytics page' do - render - - expect(rendered).not_to have_link('CI/CD', href: charts_project_pipelines_path(project)) - end - end - end - - describe 'Repository' do - it 'has a link to the repository analytics page' do - render - - expect(rendered).to have_link('Repository', href: charts_project_graph_path(project, 'master')) - end - - context 'when user does not have access' do - let(:user) { nil } - - it 'does not have a link to the repository analytics page' do - render - - expect(rendered).not_to have_link('Repository', href: charts_project_graph_path(project, 'master')) - end - end - end - - describe 'Value stream' do - it 'has a link to the value stream page' do - render - - expect(rendered).to have_link('Value stream', href: project_cycle_analytics_path(project)) - end - - context 'when user does not have access' do - let(:user) { nil } - - it 'does not have a link to the value stream page' do - render - - expect(rendered).not_to have_link('Value stream', href: project_cycle_analytics_path(project)) - end - end - end - end - - describe 'Confluence' do - let!(:service) { create(:confluence_integration, project: project, active: active) } - - before do - render - end - - context 'when the Confluence integration is active' do - let(:active) { true } - - it 'shows the Confluence link' do - expect(rendered).to have_link('Confluence', href: project_wikis_confluence_path(project)) - end - - it 'does not show the GitLab wiki link' do - expect(rendered).not_to have_link('Wiki') - end - end - - context 'when it is disabled' do - let(:active) { false } - - it 'does not show the Confluence link' do - expect(rendered).not_to have_link('Confluence') - end - - it 'shows the GitLab wiki link' do - expect(rendered).to have_link('Wiki', href: wiki_path(project.wiki)) - end - end - end - - describe 'Wiki' do - describe 'when wiki is enabled' do - it 'shows the wiki tab with the wiki internal link' do - render - - expect(rendered).to have_link('Wiki', href: wiki_path(project.wiki)) - end - end - - describe 'when wiki is disabled' do - let(:user) { nil } - - it 'does not show the wiki link' do - render - - expect(rendered).not_to have_link('Wiki') - end - end - end - - describe 'External Wiki' do - let(:properties) { { 'external_wiki_url' => 'https://gitlab.com' } } - let(:service_status) { true } - - before do - project.create_external_wiki_integration(active: service_status, properties: properties) - project.reload - end - - context 'when it is active' do - it 'shows the external wiki tab with the external wiki service link' do - render - - expect(rendered).to have_link('External wiki', href: properties['external_wiki_url']) - end - end - - context 'when it is disabled' do - let(:service_status) { false } - - it 'does not show the external wiki link' do - render - - expect(rendered).not_to have_link('External wiki') - end - end - end - - describe 'Snippets' do - before do - render - end - - context 'when user can access snippets' do - it 'shows Snippets link' do - expect(rendered).to have_link('Snippets', href: project_snippets_path(project)) - end - end - - context 'when user cannot access snippets' do - let(:user) { nil } - - it 'does not show Snippets link' do - expect(rendered).not_to have_link('Snippets') - end - end - end - - describe 'Settings' do - describe 'General' do - it 'has a link to the General settings' do - render - - expect(rendered).to have_link('General', href: edit_project_path(project)) - end - end - - describe 'Integrations' do - it 'has a link to the Integrations settings' do - render - - expect(rendered).to have_link('Integrations', href: project_settings_integrations_path(project)) - end - end - - describe 'WebHooks' do - it 'has a link to the WebHooks settings' do - render - - expect(rendered).to have_link('Webhooks', href: project_hooks_path(project)) - end - end - - describe 'Access Tokens' do - context 'self-managed instance' do - before do - allow(Gitlab).to receive(:com?).and_return(false) - end - - it 'has a link to the Access Tokens settings' do - render - - expect(rendered).to have_link('Access Tokens', href: project_settings_access_tokens_path(project)) - end - end - - context 'gitlab.com', :with_license do - before do - allow(Gitlab).to receive(:com?).and_return(true) - end - - it 'has a link to the Access Tokens settings' do - render - - expect(rendered).to have_link('Access Tokens', href: project_settings_access_tokens_path(project)) - end - end - end - - describe 'Repository' do - it 'has a link to the Repository settings' do - render - - expect(rendered).to have_link('Repository', href: project_settings_repository_path(project)) - end - end - - describe 'CI/CD' do - context 'when project is archived' do - before do - project.update!(archived: true) - end - - it 'does not have a link to the CI/CD settings' do - render - - expect(rendered).not_to have_link('CI/CD', href: project_settings_ci_cd_path(project)) - end - end - - context 'when project is not archived' do - it 'has a link to the CI/CD settings' do - render - - expect(rendered).to have_link('CI/CD', href: project_settings_ci_cd_path(project)) - end - end - end - - describe 'Monitor' do - context 'when project is archived' do - before do - project.update!(archived: true) - end - - it 'does not have a link to the Monitor settings' do - render - - expect(rendered).not_to have_link('Monitor', href: project_settings_operations_path(project)) - end - end - - context 'when project is not archived active' do - it 'has a link to the Monitor settings' do - render - - expect(rendered).to have_link('Monitor', href: project_settings_operations_path(project)) - end - end - end - - describe 'Pages' do - before do - stub_config(pages: { enabled: pages_enabled }) - end - - context 'when pages are enabled' do - let(:pages_enabled) { true } - - it 'has a link to the Pages settings' do - render - - expect(rendered).to have_link('Pages', href: project_pages_path(project)) - end - end - - context 'when pages are not enabled' do - let(:pages_enabled) { false } - - it 'does not have a link to the Pages settings' do - render - - expect(rendered).not_to have_link('Pages', href: project_pages_path(project)) - end - end - end - - describe 'Packages and registries' do - let(:packages_enabled) { false } - - before do - stub_container_registry_config(enabled: registry_enabled) - stub_config(packages: { enabled: packages_enabled }) - end - - context 'when registry is enabled' do - let(:registry_enabled) { true } - - it 'has a link to the Package and registry settings' do - render - - expect(rendered).to have_link('Packages and registries', href: project_settings_packages_and_registries_path(project)) - end - end - - context 'when registry is not enabled' do - let(:registry_enabled) { false } - - it 'does not have a link to the Package and registry settings' do - render - - expect(rendered).not_to have_link('Packages and registries', href: project_settings_packages_and_registries_path(project)) - end - end - - context 'when packages config is enabled' do - let(:registry_enabled) { false } - let(:packages_enabled) { true } - - it 'has a link to the Package and registry settings' do - render - - expect(rendered).to have_link('Packages and registries', href: project_settings_packages_and_registries_path(project)) - end - end - end - - describe 'Usage Quotas' do - it 'has a link to Usage Quotas' do - render - - expect(rendered).to have_link('Usage Quotas', href: project_usage_quotas_path(project)) - end - end - end - - describe 'Hidden menus' do - it 'has a link to the Activity page' do - render - - expect(rendered).to have_link('Activity', href: activity_project_path(project), class: 'shortcuts-project-activity', visible: false) - end - - it 'has a link to the Graph page' do - render - - expect(rendered).to have_link('Graph', href: project_network_path(project, current_ref), class: 'shortcuts-network', visible: false) - end - - it 'has a link to the New Issue page' do - render - - expect(rendered).to have_link('Create a new issue', href: new_project_issue_path(project), class: 'shortcuts-new-issue', visible: false) - end - - it 'has a link to the Jobs page' do - render - - expect(rendered).to have_link('Jobs', href: project_jobs_path(project), class: 'shortcuts-builds', visible: false) - end - - it 'has a link to the Commits page' do - render - - expect(rendered).to have_link('Commits', href: project_commits_path(project), class: 'shortcuts-commits', visible: false) - end - - it 'has a link to the Issue Boards page' do - render - - expect(rendered).to have_link('Issue Boards', href: project_boards_path(project), class: 'shortcuts-issue-boards', visible: false) - end - end - - it_behaves_like 'sidebar includes snowplow attributes', 'render', 'projects_side_navigation', 'projects_side_navigation' - - describe 'Collapsed menu items' do - it 'does not render the collapsed top menu as a link' do - render - - expect(rendered).not_to have_selector('.sidebar-sub-level-items > li.fly-out-top-item > a') - end - end -end diff --git a/spec/views/projects/pages/_pages_settings.html.haml_spec.rb b/spec/views/projects/pages/_pages_settings.html.haml_spec.rb index e790305da5d..ba276bc6070 100644 --- a/spec/views/projects/pages/_pages_settings.html.haml_spec.rb +++ b/spec/views/projects/pages/_pages_settings.html.haml_spec.rb @@ -42,7 +42,7 @@ RSpec.describe 'projects/pages/_pages_settings', feature_category: :pages do render - expect(rendered).to have_content('Use multiple versions') + expect(rendered).to have_content('Use multiple deployments') end end end diff --git a/spec/views/shared/nav/_sidebar.html.haml_spec.rb b/spec/views/shared/nav/_sidebar.html.haml_spec.rb deleted file mode 100644 index 0eb945f5624..00000000000 --- a/spec/views/shared/nav/_sidebar.html.haml_spec.rb +++ /dev/null @@ -1,43 +0,0 @@ -# frozen_string_literal: true - -require 'spec_helper' - -RSpec.describe 'shared/nav/_sidebar.html.haml' do - let_it_be(:project) { create(:project) } - - let(:context) { Sidebars::Projects::Context.new(current_user: nil, container: project) } - let(:sidebar) { Sidebars::Projects::Panel.new(context) } - - before do - assign(:project, project) - assign(:sidebar, sidebar) - - allow(sidebar).to receive(:renderable_menus).and_return([]) - end - - context 'when sidebar has a scope menu' do - it 'renders the scope menu' do - render - - expect(rendered).to render_template('shared/nav/_scope_menu') - end - end - - context 'when sidebar does not have a scope menu' do - let(:scope_menu_view_path) { 'shared/nav/' } - let(:scope_menu_view_name) { 'scope_menu.html.haml' } - let(:scope_menu_partial) { "#{scope_menu_view_path}_#{scope_menu_view_name}" } - let(:content) { 'Custom test content' } - - context 'when sidebar has a custom scope menu partial defined' do - it 'renders the custom partial' do - allow(view).to receive(:scope_menu).and_return(nil) - stub_template(scope_menu_partial => content) - - render - - expect(rendered).to have_text(content) - end - end - end -end diff --git a/spec/views/shared/runners/_runner_details.html.haml_spec.rb b/spec/views/shared/runners/_runner_details.html.haml_spec.rb index a597c719d87..0612d157ff4 100644 --- a/spec/views/shared/runners/_runner_details.html.haml_spec.rb +++ b/spec/views/shared/runners/_runner_details.html.haml_spec.rb @@ -2,7 +2,7 @@ require 'spec_helper' -RSpec.describe 'shared/runners/_runner_details.html.haml', feature_category: :runner_fleet do +RSpec.describe 'shared/runners/_runner_details.html.haml', feature_category: :fleet_visibility do include PageLayoutHelper let_it_be(:runner) do |