diff options
author | Grzegorz Bizon <grzegorz@gitlab.com> | 2017-09-13 10:28:58 +0300 |
---|---|---|
committer | Jarka Kadlecova <jarka@gitlab.com> | 2017-09-19 10:18:03 +0300 |
commit | 9e687e61759c0fdcc4f09a386f6a7b186c454442 (patch) | |
tree | 356664c6c2e1c4098b4b4a4b46d622816af04162 | |
parent | 827d4d2b6e0c746f0975ab03a511eb78ffc4b510 (diff) |
Merge branch '36907-new-issue-from-failed-build-no-longer-links-to-failed-build' into 'master'
Resolve "New issue from failed build no longer links to failed build"
Closes #36907
See merge request !13790
-rw-r--r-- | app/helpers/builds_helper.rb | 2 | ||||
-rw-r--r-- | app/serializers/build_details_entity.rb | 4 | ||||
-rw-r--r-- | spec/features/projects/jobs_spec.rb | 6 |
3 files changed, 6 insertions, 6 deletions
diff --git a/app/helpers/builds_helper.rb b/app/helpers/builds_helper.rb index 85bc784d53c..aa3a9a055a0 100644 --- a/app/helpers/builds_helper.rb +++ b/app/helpers/builds_helper.rb @@ -30,7 +30,7 @@ module BuildsHelper def build_failed_issue_options { - title: "Build Failed ##{@build.id}", + title: "Job Failed ##{@build.id}", description: project_job_url(@project, @build) } end diff --git a/app/serializers/build_details_entity.rb b/app/serializers/build_details_entity.rb index 743a08acefe..8c89eea607f 100644 --- a/app/serializers/build_details_entity.rb +++ b/app/serializers/build_details_entity.rb @@ -32,8 +32,8 @@ class BuildDetailsEntity < JobEntity private def build_failed_issue_options - { title: "Build Failed ##{build.id}", - description: project_job_path(project, build) } + { title: "Job Failed ##{build.id}", + description: "Job [##{build.id}](#{project_job_path(project, build)}) failed for #{build.sha}:\n" } end def current_user diff --git a/spec/features/projects/jobs_spec.rb b/spec/features/projects/jobs_spec.rb index 3b5c6966287..a4ed589f3de 100644 --- a/spec/features/projects/jobs_spec.rb +++ b/spec/features/projects/jobs_spec.rb @@ -164,9 +164,9 @@ feature 'Jobs' do end it 'links to issues/new with the title and description filled in' do - button_title = "Build Failed ##{job.id}" - job_path = project_job_path(project, job) - options = { issue: { title: button_title, description: job_path } } + button_title = "Job Failed ##{job.id}" + job_url = project_job_path(project, job) + options = { issue: { title: button_title, description: "Job [##{job.id}](#{job_url}) failed for #{job.sha}:\n" } } href = new_project_issue_path(project, options) |