summaryrefslogtreecommitdiff
path: root/spec/features/projects_spec.rb
diff options
context:
space:
mode:
Diffstat (limited to 'spec/features/projects_spec.rb')
-rw-r--r--spec/features/projects_spec.rb214
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