diff options
Diffstat (limited to 'spec/features/projects_spec.rb')
-rw-r--r-- | spec/features/projects_spec.rb | 214 |
1 files changed, 107 insertions, 107 deletions
diff --git a/spec/features/projects_spec.rb b/spec/features/projects_spec.rb index dbf0d427976..a8c3d956ad6 100644 --- a/spec/features/projects_spec.rb +++ b/spec/features/projects_spec.rb @@ -1,6 +1,6 @@ -require 'spec_helper' +require "spec_helper" -describe 'Project' do +describe "Project" do include ProjectForksHelper include MobileHelpers @@ -8,7 +8,7 @@ describe 'Project' do stub_feature_flags(approval_rules: false) end - describe 'creating from template' do + describe "creating from template" do let(:user) { create(:user) } let(:template) { Gitlab::ProjectTemplate.find(:rails) } @@ -18,11 +18,11 @@ describe 'Project' do end it "allows creation from templates", :js do - find('#create-from-template-tab').click + find("#create-from-template-tab").click find("label[for=#{template.name}]").click fill_in("project_name", with: template.name) - page.within '#content-body' do + page.within "#content-body" do click_button "Create project" end @@ -30,7 +30,7 @@ describe 'Project' do end end - describe 'shows tip about push to create git command' do + describe "shows tip about push to create git command" do let(:user) { create(:user) } before do @@ -38,17 +38,17 @@ describe 'Project' do visit new_project_path end - it 'shows the command in a popover', :js do - page.within '.profile-settings-sidebar' do - click_link 'Show command' + it "shows the command in a popover", :js do + page.within ".profile-settings-sidebar" do + click_link "Show command" end - expect(page).to have_css('.popover .push-to-create-popover #push_to_create_tip') - expect(page).to have_content 'Private projects can be created in your personal namespace with:' + expect(page).to have_css(".popover .push-to-create-popover #push_to_create_tip") + expect(page).to have_content "Private projects can be created in your personal namespace with:" end end - describe 'description' do + describe "description" do let(:project) { create(:project, :repository) } let(:path) { project_path(project) } @@ -56,54 +56,54 @@ describe 'Project' do sign_in(create(:admin)) end - it 'parses Markdown' do - project.update_attribute(:description, 'This is **my** project') + it "parses Markdown" do + project.update_attribute(:description, "This is **my** project") visit path - expect(page).to have_css('.home-panel-description > .home-panel-description-markdown > p > strong') + expect(page).to have_css(".home-panel-description > .home-panel-description-markdown > p > strong") end - it 'passes through html-pipeline' do - project.update_attribute(:description, 'This project is the :poop:') + it "passes through html-pipeline" do + project.update_attribute(:description, "This project is the :poop:") visit path - expect(page).to have_css('.home-panel-description > .home-panel-description-markdown > p > gl-emoji') + expect(page).to have_css(".home-panel-description > .home-panel-description-markdown > p > gl-emoji") end - it 'sanitizes unwanted tags' do + it "sanitizes unwanted tags" do project.update_attribute(:description, "```\ncode\n```") visit path - expect(page).not_to have_css('.home-panel-description code') + expect(page).not_to have_css(".home-panel-description code") end - it 'permits `rel` attribute on links' do - project.update_attribute(:description, 'https://google.com/') + it "permits `rel` attribute on links" do + project.update_attribute(:description, "https://google.com/") visit path - expect(page).to have_css('.home-panel-description a[rel]') + expect(page).to have_css(".home-panel-description a[rel]") end - context 'read more', :js do - let(:read_more_selector) { '.read-more-container' } - let(:read_more_trigger_selector) { '.home-panel-home-desc .js-read-more-trigger' } + context "read more", :js do + let(:read_more_selector) { ".read-more-container" } + let(:read_more_trigger_selector) { ".home-panel-home-desc .js-read-more-trigger" } it 'does not display "read more" link on desktop breakpoint' do - project.update_attribute(:description, 'This is **my** project') + project.update_attribute(:description, "This is **my** project") visit path expect(find(read_more_trigger_selector, visible: false)).not_to be_visible end it 'displays "read more" link on mobile breakpoint' do - project.update_attribute(:description, 'This is **my** project') + project.update_attribute(:description, "This is **my** project") visit path resize_screen_xs find(read_more_trigger_selector).click - expect(page).to have_css('.home-panel-description .is-expanded') + expect(page).to have_css(".home-panel-description .is-expanded") end end end - describe 'project topics' do + describe "project topics" do let(:project) { create(:project, :repository) } let(:path) { project_path(project) } @@ -112,29 +112,29 @@ describe 'Project' do visit path end - it 'shows project topics' do - project.update_attribute(:tag_list, 'topic1') + it "shows project topics" do + project.update_attribute(:tag_list, "topic1") visit path - expect(page).to have_css('.home-panel-topic-list') - expect(page).to have_link('Topic1', href: explore_projects_path(tag: 'topic1')) + expect(page).to have_css(".home-panel-topic-list") + expect(page).to have_link("Topic1", href: explore_projects_path(tag: "topic1")) end - it 'shows up to 3 project tags' do - project.update_attribute(:tag_list, 'topic1, topic2, topic3, topic4') + it "shows up to 3 project tags" do + project.update_attribute(:tag_list, "topic1, topic2, topic3, topic4") visit path - expect(page).to have_css('.home-panel-topic-list') - expect(page).to have_link('Topic1', href: explore_projects_path(tag: 'topic1')) - expect(page).to have_link('Topic2', href: explore_projects_path(tag: 'topic2')) - expect(page).to have_link('Topic3', href: explore_projects_path(tag: 'topic3')) - expect(page).to have_content('+ 1 more') + expect(page).to have_css(".home-panel-topic-list") + expect(page).to have_link("Topic1", href: explore_projects_path(tag: "topic1")) + expect(page).to have_link("Topic2", href: explore_projects_path(tag: "topic2")) + expect(page).to have_link("Topic3", href: explore_projects_path(tag: "topic3")) + expect(page).to have_content("+ 1 more") end end - describe 'copy clone URL to clipboard', :js do + describe "copy clone URL to clipboard", :js do let(:project) { create(:project, :repository) } let(:path) { project_path(project) } @@ -143,23 +143,23 @@ describe 'Project' do visit path end - context 'desktop component' do - it 'shows on md and larger breakpoints' do - expect(find('.git-clone-holder')).to be_visible - expect(find('.mobile-git-clone', visible: false)).not_to be_visible + context "desktop component" do + it "shows on md and larger breakpoints" do + expect(find(".git-clone-holder")).to be_visible + expect(find(".mobile-git-clone", visible: false)).not_to be_visible end end - context 'mobile component' do - it 'shows mobile component on sm and smaller breakpoints' do + context "mobile component" do + it "shows mobile component on sm and smaller breakpoints" do resize_screen_xs - expect(find('.mobile-git-clone')).to be_visible - expect(find('.git-clone-holder', visible: false)).not_to be_visible + expect(find(".mobile-git-clone")).to be_visible + expect(find(".git-clone-holder", visible: false)).not_to be_visible end end end - describe 'remove forked relationship', :js do + describe "remove forked relationship", :js do let(:user) { create(:user) } let(:project) { fork_project(create(:project, :public), user, namespace_id: user.namespace) } @@ -168,18 +168,18 @@ describe 'Project' do visit edit_project_path(project) end - it 'removes fork' do - expect(page).to have_content 'Remove fork relationship' + it "removes fork" do + expect(page).to have_content "Remove fork relationship" - remove_with_confirm('Remove fork relationship', project.path) + remove_with_confirm("Remove fork relationship", project.path) - expect(page).to have_content 'The fork relationship has been removed.' + expect(page).to have_content "The fork relationship has been removed." expect(project.reload.forked?).to be_falsey - expect(page).not_to have_content 'Remove fork relationship' + expect(page).not_to have_content "Remove fork relationship" end end - describe 'showing information about source of a project fork' do + describe "showing information about source of a project fork" do let(:user) { create(:user) } let(:base_project) { create(:project, :public, :repository) } let(:forked_project) { fork_project(base_project, user, repository: true) } @@ -188,23 +188,23 @@ describe 'Project' do sign_in user end - it 'shows a link to the source project when it is available' do + it "shows a link to the source project when it is available" do visit project_path(forked_project) - expect(page).to have_content('Forked from') + expect(page).to have_content("Forked from") expect(page).to have_link(base_project.full_name) end - it 'does not contain fork network information for the root project' do + it "does not contain fork network information for the root project" do forked_project visit project_path(base_project) - expect(page).not_to have_content('In fork network of') - expect(page).not_to have_content('Forked from') + expect(page).not_to have_content("In fork network of") + expect(page).not_to have_content("Forked from") end - it 'shows the name of the deleted project when the source was deleted' do + it "shows the name of the deleted project when the source was deleted" do forked_project Projects::DestroyService.new(base_project, base_project.owner).execute @@ -213,10 +213,10 @@ describe 'Project' do expect(page).to have_content("Forked from #{base_project.full_name} (deleted)") end - context 'a fork of a fork' do + context "a fork of a fork" do let(:fork_of_fork) { fork_project(forked_project, user, repository: true) } - it 'links to the base project if the source project is removed' do + it "links to the base project if the source project is removed" do fork_of_fork Projects::DestroyService.new(forked_project, user).execute @@ -228,7 +228,7 @@ describe 'Project' do end end - describe 'when the project repository is disabled', :js do + describe "when the project repository is disabled", :js do let(:user) { create(:user) } let(:project) { create(:project, :repository_disabled, :repository, namespace: user.namespace) } @@ -238,14 +238,14 @@ describe 'Project' do visit project_path(project) end - it 'does not show an error' do + it "does not show an error" do wait_for_requests - expect(page).not_to have_selector('.flash-alert') + expect(page).not_to have_selector(".flash-alert") end end - describe 'removal', :js do + describe "removal", :js do let(:user) { create(:user) } let(:project) { create(:project, namespace: user.namespace) } @@ -255,14 +255,14 @@ describe 'Project' do visit edit_project_path(project) end - it 'focuses on the confirmation field' do - click_button 'Remove project' + it "focuses on the confirmation field" do + click_button "Remove project" - expect(page).to have_selector '#confirm_name_input:focus' + expect(page).to have_selector "#confirm_name_input:focus" end - it 'removes a project' do - expect { remove_with_confirm('Remove project', project.path) }.to change { Project.count }.by(-1) + it "removes a project" do + expect { remove_with_confirm("Remove project", project.path) }.to change { Project.count }.by(-1) expect(page).to have_content "Project '#{project.full_name}' is in the process of being deleted." expect(Project.all.count).to be_zero expect(project.issues).to be_empty @@ -270,8 +270,8 @@ describe 'Project' do end end - describe 'tree view (default view is set to Files)' do - let(:user) { create(:user, project_view: 'files') } + describe "tree view (default view is set to Files)" do + let(:user) { create(:user, project_view: "files") } let(:project) { create(:forked_project_with_submodules) } before do @@ -280,70 +280,70 @@ describe 'Project' do visit project_path(project) end - it 'has working links to files' do - click_link('PROCESS.md') + it "has working links to files" do + click_link("PROCESS.md") expect(page.status_code).to eq(200) end - it 'has working links to directories' do - click_link('encoding') + it "has working links to directories" do + click_link("encoding") expect(page.status_code).to eq(200) end - it 'has working links to submodules' do - click_link('645f6c4c') + it "has working links to submodules" do + click_link("645f6c4c") expect(page.status_code).to eq(200) end - context 'for signed commit on default branch', :js do + context "for signed commit on default branch", :js do before do - project.change_head('33f3729a45c02fc67d00adb1b8bca394b0e761d9') + project.change_head("33f3729a45c02fc67d00adb1b8bca394b0e761d9") end - it 'displays a GPG badge' do + it "displays a GPG badge" do visit project_path(project) wait_for_requests - expect(page).not_to have_selector '.gpg-status-box.js-loading-gpg-badge' - expect(page).to have_selector '.gpg-status-box.invalid' + expect(page).not_to have_selector ".gpg-status-box.js-loading-gpg-badge" + expect(page).to have_selector ".gpg-status-box.invalid" end end - context 'for subgroups', :js, :nested_groups do + context "for subgroups", :js, :nested_groups do let(:group) { create(:group) } let(:subgroup) { create(:group, parent: group) } let(:project) { create(:project, :repository, group: subgroup) } - it 'renders tree table without errors' do + it "renders tree table without errors" do wait_for_requests - expect(page).to have_selector('.tree-item') - expect(page).not_to have_selector('.flash-alert') + expect(page).to have_selector(".tree-item") + expect(page).not_to have_selector(".flash-alert") end - context 'for signed commit' do + context "for signed commit" do before do repository = project.repository - repository.write_ref("refs/heads/#{project.default_branch}", '33f3729a45c02fc67d00adb1b8bca394b0e761d9') + repository.write_ref("refs/heads/#{project.default_branch}", "33f3729a45c02fc67d00adb1b8bca394b0e761d9") repository.expire_branches_cache end - it 'displays a GPG badge' do + it "displays a GPG badge" do visit project_path(project) wait_for_requests - expect(page).not_to have_selector '.gpg-status-box.js-loading-gpg-badge' - expect(page).to have_selector '.gpg-status-box.invalid' + expect(page).not_to have_selector ".gpg-status-box.js-loading-gpg-badge" + expect(page).to have_selector ".gpg-status-box.invalid" end end end end - describe 'activity view' do - let(:user) { create(:user, project_view: 'activity') } + describe "activity view" do + let(:user) { create(:user, project_view: "activity") } let(:project) { create(:project, :repository) } before do @@ -352,30 +352,30 @@ describe 'Project' do visit project_path(project) end - it 'loads activity', :js do - expect(page).to have_selector('.event-item') + it "loads activity", :js do + expect(page).to have_selector(".event-item") end end - context 'content is not cached after signing out', :js do - let(:user) { create(:user, project_view: 'activity') } + context "content is not cached after signing out", :js do + let(:user) { create(:user, project_view: "activity") } let(:project) { create(:project, :repository) } - it 'does not load activity', :js do + it "does not load activity", :js do project.add_maintainer(user) sign_in(user) visit project_path(project) sign_out(user) - page.evaluate_script('window.history.back()') + page.evaluate_script("window.history.back()") - expect(page).not_to have_selector('.event-item') + expect(page).not_to have_selector(".event-item") end end def remove_with_confirm(button_text, confirm_with) click_button button_text - fill_in 'confirm_name_input', with: confirm_with - click_button 'Confirm' + fill_in "confirm_name_input", with: confirm_with + click_button "Confirm" end end |