Replace deprecated name_with_namespace with full_name in app and spec
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
This commit is contained in:
parent
1b54a8cdda
commit
8fe9995f11
|
@ -62,7 +62,7 @@ class InvitesController < ApplicationController
|
|||
case source
|
||||
when Project
|
||||
project = member.source
|
||||
label = "project #{project.name_with_namespace}"
|
||||
label = "project #{project.full_name}"
|
||||
path = project_path(project)
|
||||
when Group
|
||||
group = member.source
|
||||
|
|
|
@ -38,7 +38,7 @@ class Projects::BlobController < Projects::ApplicationController
|
|||
end
|
||||
|
||||
format.json do
|
||||
page_title @blob.path, @ref, @project.name_with_namespace
|
||||
page_title @blob.path, @ref, @project.full_name
|
||||
|
||||
show_json
|
||||
end
|
||||
|
|
|
@ -36,7 +36,7 @@ class Projects::TreeController < Projects::ApplicationController
|
|||
end
|
||||
|
||||
format.json do
|
||||
page_title @path.presence || _("Files"), @ref, @project.name_with_namespace
|
||||
page_title @path.presence || _("Files"), @ref, @project.full_name
|
||||
|
||||
# n+1: https://gitlab.com/gitlab-org/gitlab-ce/issues/38261
|
||||
Gitlab::GitalyClient.allow_n_plus_1_calls do
|
||||
|
|
|
@ -130,7 +130,7 @@ class ProjectsController < Projects::ApplicationController
|
|||
return access_denied! unless can?(current_user, :remove_project, @project)
|
||||
|
||||
::Projects::DestroyService.new(@project, current_user, {}).async_execute
|
||||
flash[:notice] = _("Project '%{project_name}' is in the process of being deleted.") % { project_name: @project.name_with_namespace }
|
||||
flash[:notice] = _("Project '%{project_name}' is in the process of being deleted.") % { project_name: @project.full_name }
|
||||
|
||||
redirect_to dashboard_projects_path, status: 302
|
||||
rescue Projects::DestroyService::DestroyError => ex
|
||||
|
|
|
@ -99,7 +99,7 @@ module IssuablesHelper
|
|||
project = Project.find_by(id: project_id)
|
||||
|
||||
if project
|
||||
project.name_with_namespace
|
||||
project.full_name
|
||||
else
|
||||
default_label
|
||||
end
|
||||
|
|
|
@ -97,13 +97,13 @@ module ProjectsHelper
|
|||
end
|
||||
|
||||
def remove_project_message(project)
|
||||
_("You are going to remove %{project_name_with_namespace}. Removed project CANNOT be restored! Are you ABSOLUTELY sure?") %
|
||||
{ project_name_with_namespace: project.name_with_namespace }
|
||||
_("You are going to remove %{project_full_name}. Removed project CANNOT be restored! Are you ABSOLUTELY sure?") %
|
||||
{ project_full_name: project.full_name }
|
||||
end
|
||||
|
||||
def transfer_project_message(project)
|
||||
_("You are going to transfer %{project_name_with_namespace} to another owner. Are you ABSOLUTELY sure?") %
|
||||
{ project_name_with_namespace: project.name_with_namespace }
|
||||
_("You are going to transfer %{project_full_name} to another owner. Are you ABSOLUTELY sure?") %
|
||||
{ project_full_name: project.full_name }
|
||||
end
|
||||
|
||||
def remove_fork_project_message(project)
|
||||
|
|
|
@ -110,7 +110,7 @@ module SearchHelper
|
|||
category: "Projects",
|
||||
id: p.id,
|
||||
value: "#{search_result_sanitize(p.name)}",
|
||||
label: "#{search_result_sanitize(p.name_with_namespace)}",
|
||||
label: "#{search_result_sanitize(p.full_name)}",
|
||||
url: project_path(p)
|
||||
}
|
||||
end
|
||||
|
|
|
@ -114,7 +114,7 @@ module TodosHelper
|
|||
projects = current_user.authorized_projects.sorted_by_activity.non_archived.with_route
|
||||
|
||||
projects = projects.map do |project|
|
||||
{ id: project.id, text: project.name_with_namespace }
|
||||
{ id: project.id, text: project.full_name }
|
||||
end
|
||||
|
||||
projects.unshift({ id: '', text: 'Any Project' }).to_json
|
||||
|
|
|
@ -158,7 +158,7 @@ class Event < ActiveRecord::Base
|
|||
|
||||
def project_name
|
||||
if project
|
||||
project.name_with_namespace
|
||||
project.full_name
|
||||
else
|
||||
"(deleted project)"
|
||||
end
|
||||
|
|
|
@ -68,7 +68,7 @@ http://app.asana.com/-/account_api'
|
|||
end
|
||||
|
||||
user = data[:user_name]
|
||||
project_name = project.name_with_namespace
|
||||
project_name = project.full_name
|
||||
|
||||
data[:commits].each do |commit|
|
||||
push_msg = "#{user} pushed to branch #{branch} of #{project_name} ( #{commit[:url]} ):"
|
||||
|
|
|
@ -86,7 +86,7 @@ class CampfireService < Service
|
|||
after = push[:after]
|
||||
|
||||
message = ""
|
||||
message << "[#{project.name_with_namespace}] "
|
||||
message << "[#{project.full_name}] "
|
||||
message << "#{push[:user_name]} "
|
||||
|
||||
if Gitlab::Git.blank_ref?(before)
|
||||
|
|
|
@ -129,7 +129,7 @@ class ChatNotificationService < Service
|
|||
end
|
||||
|
||||
def project_name
|
||||
project.name_with_namespace.gsub(/\s/, '')
|
||||
project.full_name.gsub(/\s/, '')
|
||||
end
|
||||
|
||||
def project_url
|
||||
|
|
|
@ -120,7 +120,7 @@ class HipchatService < Service
|
|||
else
|
||||
message << "pushed to #{ref_type} <a href=\""\
|
||||
"#{project.web_url}/commits/#{CGI.escape(ref)}\">#{ref}</a> "
|
||||
message << "of <a href=\"#{project.web_url}\">#{project.name_with_namespace.gsub!(/\s/, '')}</a> "
|
||||
message << "of <a href=\"#{project.web_url}\">#{project.full_name.gsub!(/\s/, '')}</a> "
|
||||
message << "(<a href=\"#{project.web_url}/compare/#{before}...#{after}\">Compare changes</a>)"
|
||||
|
||||
push[:commits].take(MAX_COMMITS).each do |commit|
|
||||
|
@ -274,7 +274,7 @@ class HipchatService < Service
|
|||
end
|
||||
|
||||
def project_name
|
||||
project.name_with_namespace.gsub(/\s/, '')
|
||||
project.full_name.gsub(/\s/, '')
|
||||
end
|
||||
|
||||
def project_url
|
||||
|
|
|
@ -37,7 +37,7 @@ class MattermostSlashCommandsService < SlashCommandsService
|
|||
private
|
||||
|
||||
def command(params)
|
||||
pretty_project_name = project.name_with_namespace
|
||||
pretty_project_name = project.full_name
|
||||
|
||||
params.merge(
|
||||
auto_complete: true,
|
||||
|
|
|
@ -88,10 +88,10 @@ class PushoverService < Service
|
|||
user: user_key,
|
||||
device: device,
|
||||
priority: priority,
|
||||
title: "#{project.name_with_namespace}",
|
||||
title: "#{project.full_name}",
|
||||
message: message,
|
||||
url: data[:project][:web_url],
|
||||
url_title: "See project #{project.name_with_namespace}"
|
||||
url_title: "See project #{project.full_name}"
|
||||
}
|
||||
|
||||
# Sound parameter MUST NOT be sent to API if not selected
|
||||
|
|
|
@ -246,7 +246,7 @@ describe AutocompleteController do
|
|||
expect(json_response.size).to eq(1)
|
||||
|
||||
expect(json_response.first['id']).to eq authorized_project.id
|
||||
expect(json_response.first['name_with_namespace']).to eq authorized_project.name_with_namespace
|
||||
expect(json_response.first['full_name']).to eq authorized_project.full_name
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -267,7 +267,7 @@ describe AutocompleteController do
|
|||
expect(json_response.size).to eq(1)
|
||||
|
||||
expect(json_response.first['id']).to eq authorized_search_project.id
|
||||
expect(json_response.first['name_with_namespace']).to eq authorized_search_project.name_with_namespace
|
||||
expect(json_response.first['full_name']).to eq authorized_search_project.full_name
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -173,7 +173,7 @@ feature 'Admin Groups' do
|
|||
|
||||
visit admin_group_path(group)
|
||||
|
||||
expect(page).to have_content(empty_project.name_with_namespace)
|
||||
expect(page).to have_content(empty_project.full_name)
|
||||
expect(page).to have_content('Projects shared with')
|
||||
end
|
||||
end
|
||||
|
|
|
@ -58,7 +58,7 @@ describe "Admin::Projects" do
|
|||
expect(current_path).to eq admin_project_path(project)
|
||||
expect(page).to have_content(project.path)
|
||||
expect(page).to have_content(project.name)
|
||||
expect(page).to have_content(project.name_with_namespace)
|
||||
expect(page).to have_content(project.full_name)
|
||||
expect(page).to have_content(project.creator.name)
|
||||
end
|
||||
end
|
||||
|
|
|
@ -76,8 +76,8 @@ describe "Admin Runners" do
|
|||
|
||||
describe 'projects' do
|
||||
it 'contains project names' do
|
||||
expect(page).to have_content(@project1.name_with_namespace)
|
||||
expect(page).to have_content(@project2.name_with_namespace)
|
||||
expect(page).to have_content(@project1.full_name)
|
||||
expect(page).to have_content(@project2.full_name)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -89,8 +89,8 @@ describe "Admin Runners" do
|
|||
end
|
||||
|
||||
it 'contains name of correct project' do
|
||||
expect(page).to have_content(@project1.name_with_namespace)
|
||||
expect(page).not_to have_content(@project2.name_with_namespace)
|
||||
expect(page).to have_content(@project1.full_name)
|
||||
expect(page).not_to have_content(@project2.full_name)
|
||||
end
|
||||
end
|
||||
|
||||
|
|
|
@ -74,8 +74,8 @@ RSpec.describe 'Dashboard Issues' do
|
|||
find('.new-project-item-select-button').click
|
||||
|
||||
page.within('.select2-results') do
|
||||
expect(page).to have_content(project.name_with_namespace)
|
||||
expect(page).not_to have_content(project_with_issues_disabled.name_with_namespace)
|
||||
expect(page).to have_content(project.full_name)
|
||||
expect(page).not_to have_content(project_with_issues_disabled.full_name)
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -85,7 +85,7 @@ RSpec.describe 'Dashboard Issues' do
|
|||
wait_for_requests
|
||||
|
||||
project_path = "/#{project.full_path}"
|
||||
project_json = { name: project.name_with_namespace, url: project_path }.to_json
|
||||
project_json = { name: project.full_name, url: project_path }.to_json
|
||||
|
||||
# simulate selection, and prevent overlap by dropdown menu
|
||||
first('.project-item-select', visible: false)
|
||||
|
|
|
@ -28,8 +28,8 @@ feature 'Dashboard Merge Requests' do
|
|||
find('.new-project-item-select-button').click
|
||||
|
||||
page.within('.select2-results') do
|
||||
expect(page).to have_content(project.name_with_namespace)
|
||||
expect(page).not_to have_content(project_with_disabled_merge_requests.name_with_namespace)
|
||||
expect(page).to have_content(project.full_name)
|
||||
expect(page).not_to have_content(project_with_disabled_merge_requests.full_name)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -24,14 +24,14 @@ feature 'Dashboard > User filters todos', :js do
|
|||
it 'filters by project' do
|
||||
click_button 'Project'
|
||||
within '.dropdown-menu-project' do
|
||||
fill_in 'Search projects', with: project_1.name_with_namespace
|
||||
click_link project_1.name_with_namespace
|
||||
fill_in 'Search projects', with: project_1.full_name
|
||||
click_link project_1.full_name
|
||||
end
|
||||
|
||||
wait_for_requests
|
||||
|
||||
expect(page).to have_content project_1.name_with_namespace
|
||||
expect(page).not_to have_content project_2.name_with_namespace
|
||||
expect(page).to have_content project_1.full_name
|
||||
expect(page).not_to have_content project_2.full_name
|
||||
end
|
||||
|
||||
context 'Author filter' do
|
||||
|
|
|
@ -73,7 +73,7 @@ feature 'issue move to another project' do
|
|||
wait_for_requests
|
||||
|
||||
page.within '.js-sidebar-move-issue-block' do
|
||||
expect(page).to have_content new_project.name_with_namespace
|
||||
expect(page).to have_content new_project.full_name
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -25,7 +25,7 @@ feature 'Projects > Members > Master manages access requests' do
|
|||
perform_enqueued_jobs { click_on 'Grant access' }
|
||||
|
||||
expect(ActionMailer::Base.deliveries.last.to).to eq [user.notification_email]
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.name_with_namespace} project was granted"
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.full_name} project was granted"
|
||||
end
|
||||
|
||||
scenario 'master can deny access' do
|
||||
|
@ -36,7 +36,7 @@ feature 'Projects > Members > Master manages access requests' do
|
|||
perform_enqueued_jobs { click_on 'Deny access' }
|
||||
|
||||
expect(ActionMailer::Base.deliveries.last.to).to eq [user.notification_email]
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.name_with_namespace} project was denied"
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.full_name} project was denied"
|
||||
end
|
||||
|
||||
def expect_visible_access_request(project, user)
|
||||
|
|
|
@ -21,7 +21,7 @@ feature 'Projects > Members > User requests access', :js do
|
|||
perform_enqueued_jobs { click_link 'Request Access' }
|
||||
|
||||
expect(ActionMailer::Base.deliveries.last.to).to eq [master.notification_email]
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to eq "Request to join the #{project.name_with_namespace} project"
|
||||
expect(ActionMailer::Base.deliveries.last.subject).to eq "Request to join the #{project.full_name} project"
|
||||
|
||||
expect(project.requesters.exists?(user_id: user)).to be_truthy
|
||||
expect(page).to have_content 'Your request for access has been queued for review.'
|
||||
|
|
|
@ -39,7 +39,7 @@ describe 'User manages project members' do
|
|||
click_link('Import')
|
||||
end
|
||||
|
||||
select(project2.name_with_namespace, from: 'source_project_id')
|
||||
select(project2.full_name, from: 'source_project_id')
|
||||
click_button('Import')
|
||||
|
||||
project_member = project.project_members.find_by(user_id: user_mike.id)
|
||||
|
|
|
@ -35,7 +35,7 @@ describe 'User searches for code' do
|
|||
find('.js-search-project-dropdown').click
|
||||
|
||||
page.within('.project-filter') do
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
fill_in('dashboard_search', with: 'rspec')
|
||||
|
|
|
@ -34,7 +34,7 @@ describe 'User searches for issues', :js do
|
|||
find('.js-search-project-dropdown').click
|
||||
|
||||
page.within('.project-filter') do
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
fill_in('dashboard_search', with: issue1.title)
|
||||
|
|
|
@ -33,7 +33,7 @@ describe 'User searches for merge requests', :js do
|
|||
find('.js-search-project-dropdown').click
|
||||
|
||||
page.within('.project-filter') do
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
fill_in('dashboard_search', with: merge_request1.title)
|
||||
|
|
|
@ -33,7 +33,7 @@ describe 'User searches for milestones', :js do
|
|||
find('.js-search-project-dropdown').click
|
||||
|
||||
page.within('.project-filter') do
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
fill_in('dashboard_search', with: milestone1.title)
|
||||
|
|
|
@ -18,7 +18,7 @@ describe 'User searches for wiki pages', :js do
|
|||
find('.js-search-project-dropdown').click
|
||||
|
||||
page.within('.project-filter') do
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
fill_in('dashboard_search', with: 'content')
|
||||
|
|
|
@ -31,7 +31,7 @@ describe 'User uses search filters', :js do
|
|||
|
||||
wait_for_requests
|
||||
|
||||
expect(page).to have_link(group_project.name_with_namespace)
|
||||
expect(page).to have_link(group_project.full_name)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -43,10 +43,10 @@ describe 'User uses search filters', :js do
|
|||
|
||||
wait_for_requests
|
||||
|
||||
click_link(project.name_with_namespace)
|
||||
click_link(project.full_name)
|
||||
end
|
||||
|
||||
expect(find('.js-search-project-dropdown')).to have_content(project.name_with_namespace)
|
||||
expect(find('.js-search-project-dropdown')).to have_content(project.full_name)
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -12,10 +12,10 @@ describe MembersHelper do
|
|||
let(:group_member_invite) { build(:group_member, group: group).tap { |m| m.generate_invite_token! } }
|
||||
let(:group_member_request) { group.request_access(requester) }
|
||||
|
||||
it { expect(remove_member_message(project_member)).to eq "Are you sure you want to remove #{project_member.user.name} from the #{project.name_with_namespace} project?" }
|
||||
it { expect(remove_member_message(project_member_invite)).to eq "Are you sure you want to revoke the invitation for #{project_member_invite.invite_email} to join the #{project.name_with_namespace} project?" }
|
||||
it { expect(remove_member_message(project_member_request)).to eq "Are you sure you want to deny #{requester.name}'s request to join the #{project.name_with_namespace} project?" }
|
||||
it { expect(remove_member_message(project_member_request, user: requester)).to eq "Are you sure you want to withdraw your access request for the #{project.name_with_namespace} project?" }
|
||||
it { expect(remove_member_message(project_member)).to eq "Are you sure you want to remove #{project_member.user.name} from the #{project.full_name} project?" }
|
||||
it { expect(remove_member_message(project_member_invite)).to eq "Are you sure you want to revoke the invitation for #{project_member_invite.invite_email} to join the #{project.full_name} project?" }
|
||||
it { expect(remove_member_message(project_member_request)).to eq "Are you sure you want to deny #{requester.name}'s request to join the #{project.full_name} project?" }
|
||||
it { expect(remove_member_message(project_member_request, user: requester)).to eq "Are you sure you want to withdraw your access request for the #{project.full_name} project?" }
|
||||
it { expect(remove_member_message(group_member)).to eq "Are you sure you want to remove #{group_member.user.name} from the #{group.name} group?" }
|
||||
it { expect(remove_member_message(group_member_invite)).to eq "Are you sure you want to revoke the invitation for #{group_member_invite.invite_email} to join the #{group.name} group?" }
|
||||
it { expect(remove_member_message(group_member_request)).to eq "Are you sure you want to deny #{requester.name}'s request to join the #{group.name} group?" }
|
||||
|
@ -42,7 +42,7 @@ describe MembersHelper do
|
|||
let(:group) { build_stubbed(:group) }
|
||||
let(:user) { build_stubbed(:user) }
|
||||
|
||||
it { expect(leave_confirmation_message(project)).to eq "Are you sure you want to leave the \"#{project.name_with_namespace}\" project?" }
|
||||
it { expect(leave_confirmation_message(project)).to eq "Are you sure you want to leave the \"#{project.full_name}\" project?" }
|
||||
it { expect(leave_confirmation_message(group)).to eq "Are you sure you want to leave the \"#{group.name}\" group?" }
|
||||
end
|
||||
end
|
||||
|
|
|
@ -26,8 +26,8 @@ describe TodosHelper do
|
|||
|
||||
expected_results = [
|
||||
{ 'id' => '', 'text' => 'Any Project' },
|
||||
{ 'id' => projects.second.id, 'text' => projects.second.name_with_namespace },
|
||||
{ 'id' => projects.first.id, 'text' => projects.first.name_with_namespace }
|
||||
{ 'id' => projects.second.id, 'text' => projects.second.full_name },
|
||||
{ 'id' => projects.first.id, 'text' => projects.first.full_name }
|
||||
]
|
||||
|
||||
expect(JSON.parse(helper.todo_projects_options)).to match_array(expected_results)
|
||||
|
|
|
@ -50,7 +50,7 @@ describe('glDropdown', function describeDropdown() {
|
|||
search: {
|
||||
fields: ['name']
|
||||
},
|
||||
text: project => (project.name_with_namespace || project.name),
|
||||
text: project => (project.full_name || project.name),
|
||||
id: project => project.id,
|
||||
}, extraOpts);
|
||||
this.dropdownButtonElement = $('#js-project-dropdown', this.dropdownContainerElement).glDropdown(options);
|
||||
|
@ -76,7 +76,7 @@ describe('glDropdown', function describeDropdown() {
|
|||
});
|
||||
|
||||
it('escapes HTML as text', () => {
|
||||
this.projectsData[0].name_with_namespace = '<script>alert("testing");</script>';
|
||||
this.projectsData[0].full_name = '<script>alert("testing");</script>';
|
||||
|
||||
initDropDown.call(this, false);
|
||||
|
||||
|
@ -88,7 +88,7 @@ describe('glDropdown', function describeDropdown() {
|
|||
});
|
||||
|
||||
it('should output HTML when highlighting', () => {
|
||||
this.projectsData[0].name_with_namespace = 'testing';
|
||||
this.projectsData[0].full_name = 'testing';
|
||||
$('.dropdown-input .dropdown-input-field').val('test');
|
||||
|
||||
initDropDown.call(this, false, true, {
|
||||
|
|
|
@ -24,7 +24,7 @@ describe('ProjectsStore', () => {
|
|||
const processedProjects = store.getSearchedProjects();
|
||||
expect(processedProjects.length).toBe(1);
|
||||
expect(processedProjects[0].id).toBe(mockRawProject.id);
|
||||
expect(processedProjects[0].namespace).toBe(mockRawProject.name_with_namespace);
|
||||
expect(processedProjects[0].namespace).toBe(mockRawProject.full_name);
|
||||
expect(processedProjects[0].webUrl).toBe(mockRawProject.web_url);
|
||||
expect(processedProjects[0].avatarUrl).toBe(mockRawProject.avatar_url);
|
||||
});
|
||||
|
|
|
@ -381,11 +381,11 @@ describe Banzai::Filter::LabelReferenceFilter do
|
|||
end
|
||||
|
||||
it 'has valid link text' do
|
||||
expect(result.css('a').first.text).to eq "#{label.name} in #{project2.name_with_namespace}"
|
||||
expect(result.css('a').first.text).to eq "#{label.name} in #{project2.full_name}"
|
||||
end
|
||||
|
||||
it 'has valid text' do
|
||||
expect(result.text).to eq "See #{label.name} in #{project2.name_with_namespace}"
|
||||
expect(result.text).to eq "See #{label.name} in #{project2.full_name}"
|
||||
end
|
||||
|
||||
it 'ignores invalid IDs on the referenced label' do
|
||||
|
@ -481,12 +481,12 @@ describe Banzai::Filter::LabelReferenceFilter do
|
|||
|
||||
it 'has valid link text' do
|
||||
expect(result.css('a').first.text)
|
||||
.to eq "#{group_label.name} in #{another_project.name_with_namespace}"
|
||||
.to eq "#{group_label.name} in #{another_project.full_name}"
|
||||
end
|
||||
|
||||
it 'has valid text' do
|
||||
expect(result.text)
|
||||
.to eq "See #{group_label.name} in #{another_project.name_with_namespace}"
|
||||
.to eq "See #{group_label.name} in #{another_project.full_name}"
|
||||
end
|
||||
|
||||
it 'ignores invalid IDs on the referenced label' do
|
||||
|
|
|
@ -16,7 +16,7 @@ describe Gitlab::DataBuilder::Build do
|
|||
it { expect(data[:build_status]).to eq(build.status) }
|
||||
it { expect(data[:build_allow_failure]).to eq(false) }
|
||||
it { expect(data[:project_id]).to eq(build.project.id) }
|
||||
it { expect(data[:project_name]).to eq(build.project.name_with_namespace) }
|
||||
it { expect(data[:project_name]).to eq(build.project.full_name) }
|
||||
|
||||
context 'commit author_url' do
|
||||
context 'when no commit present' do
|
||||
|
|
|
@ -457,7 +457,7 @@ describe Notify do
|
|||
|
||||
it 'has the correct subject and body' do
|
||||
is_expected.to have_subject("#{project.name} | Project was moved")
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text(project.ssh_url_to_repo)
|
||||
end
|
||||
end
|
||||
|
@ -483,8 +483,8 @@ describe Notify do
|
|||
to_emails = subject.header[:to].addrs.map(&:address)
|
||||
expect(to_emails).to eq([recipient.notification_email])
|
||||
|
||||
is_expected.to have_subject "Request to join the #{project.name_with_namespace} project"
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_subject "Request to join the #{project.full_name} project"
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project_project_members_url(project)
|
||||
is_expected.to have_body_text project_member.human_access
|
||||
end
|
||||
|
@ -503,8 +503,8 @@ describe Notify do
|
|||
it_behaves_like "a user cannot unsubscribe through footer link"
|
||||
|
||||
it 'contains all the useful information' do
|
||||
is_expected.to have_subject "Access to the #{project.name_with_namespace} project was denied"
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_subject "Access to the #{project.full_name} project was denied"
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project.web_url
|
||||
end
|
||||
end
|
||||
|
@ -520,8 +520,8 @@ describe Notify do
|
|||
it_behaves_like "a user cannot unsubscribe through footer link"
|
||||
|
||||
it 'contains all the useful information' do
|
||||
is_expected.to have_subject "Access to the #{project.name_with_namespace} project was granted"
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_subject "Access to the #{project.full_name} project was granted"
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project.web_url
|
||||
is_expected.to have_body_text project_member.human_access
|
||||
end
|
||||
|
@ -550,8 +550,8 @@ describe Notify do
|
|||
it_behaves_like "a user cannot unsubscribe through footer link"
|
||||
|
||||
it 'contains all the useful information' do
|
||||
is_expected.to have_subject "Invitation to join the #{project.name_with_namespace} project"
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_subject "Invitation to join the #{project.full_name} project"
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project.web_url
|
||||
is_expected.to have_body_text project_member.human_access
|
||||
is_expected.to have_body_text project_member.invite_token
|
||||
|
@ -575,7 +575,7 @@ describe Notify do
|
|||
|
||||
it 'contains all the useful information' do
|
||||
is_expected.to have_subject 'Invitation accepted'
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project.web_url
|
||||
is_expected.to have_body_text project_member.invite_email
|
||||
is_expected.to have_html_escaped_body_text invited_user.name
|
||||
|
@ -598,7 +598,7 @@ describe Notify do
|
|||
|
||||
it 'contains all the useful information' do
|
||||
is_expected.to have_subject 'Invitation declined'
|
||||
is_expected.to have_html_escaped_body_text project.name_with_namespace
|
||||
is_expected.to have_html_escaped_body_text project.full_name
|
||||
is_expected.to have_body_text project.web_url
|
||||
is_expected.to have_body_text project_member.invite_email
|
||||
end
|
||||
|
|
|
@ -47,7 +47,7 @@ describe AsanaService do
|
|||
|
||||
it 'calls Asana service to create a story' do
|
||||
data = create_data_for_commits('Message from commit. related to #123456')
|
||||
expected_message = "#{data[:user_name]} pushed to branch #{data[:ref]} of #{project.name_with_namespace} ( #{data[:commits][0][:url]} ): #{data[:commits][0][:message]}"
|
||||
expected_message = "#{data[:user_name]} pushed to branch #{data[:ref]} of #{project.full_name} ( #{data[:commits][0][:url]} ): #{data[:commits][0][:message]}"
|
||||
|
||||
d1 = double('Asana::Task')
|
||||
expect(d1).to receive(:add_comment).with(text: expected_message)
|
||||
|
|
|
@ -29,7 +29,7 @@ describe HipchatService do
|
|||
let(:user) { create(:user) }
|
||||
let(:project) { create(:project, :repository) }
|
||||
let(:api_url) { 'https://hipchat.example.com/v2/room/123456/notification?auth_token=verySecret' }
|
||||
let(:project_name) { project.name_with_namespace.gsub(/\s/, '') }
|
||||
let(:project_name) { project.full_name.gsub(/\s/, '') }
|
||||
let(:token) { 'verySecret' }
|
||||
let(:server_url) { 'https://hipchat.example.com'}
|
||||
let(:push_sample_data) do
|
||||
|
@ -303,7 +303,7 @@ describe HipchatService do
|
|||
message = hipchat.__send__(:create_pipeline_message, data)
|
||||
|
||||
project_url = project.web_url
|
||||
project_name = project.name_with_namespace.gsub(/\s/, '')
|
||||
project_name = project.full_name.gsub(/\s/, '')
|
||||
pipeline_attributes = data[:object_attributes]
|
||||
ref = pipeline_attributes[:ref]
|
||||
ref_type = pipeline_attributes[:tag] ? 'tag' : 'branch'
|
||||
|
|
|
@ -31,10 +31,10 @@ describe MattermostSlashCommandsService do
|
|||
url: 'http://trigger.url',
|
||||
icon_url: 'http://icon.url/icon.png',
|
||||
auto_complete: true,
|
||||
auto_complete_desc: "Perform common operations on: #{project.name_with_namespace}",
|
||||
auto_complete_desc: "Perform common operations on: #{project.full_name}",
|
||||
auto_complete_hint: '[help]',
|
||||
description: "Perform common operations on: #{project.name_with_namespace}",
|
||||
display_name: "GitLab / #{project.name_with_namespace}",
|
||||
description: "Perform common operations on: #{project.full_name}",
|
||||
display_name: "GitLab / #{project.full_name}",
|
||||
method: 'P',
|
||||
username: 'GitLab'
|
||||
}.to_json)
|
||||
|
|
Loading…
Reference in New Issue