diff options
author | Nathan Friend <nathan@gitlab.com> | 2019-08-13 22:04:56 +0300 |
---|---|---|
committer | Nathan Friend <nathan@gitlab.com> | 2019-08-13 22:04:56 +0300 |
commit | 5112d92c9bbd2c5ba270e033b5eb0252f574a814 (patch) | |
tree | 3f80ed1ca3d0d53c1999d8ae53eacf69823a332b /spec/models/project_services/chat_message | |
parent | df35d772c655587eecbe7b3e387c8b8bc287b23c (diff) |
Fix project image in Slack pipeline notifications
This commit fixes the project avatar images that are rendered in the
footer of Slack pipeline notifications. Previously, the image URLs
provided to Slack were relative URLs; now they are absolute.
Diffstat (limited to 'spec/models/project_services/chat_message')
-rw-r--r-- | spec/models/project_services/chat_message/pipeline_message_spec.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/spec/models/project_services/chat_message/pipeline_message_spec.rb b/spec/models/project_services/chat_message/pipeline_message_spec.rb index 619ab96af94..cf7c7bf7e61 100644 --- a/spec/models/project_services/chat_message/pipeline_message_spec.rb +++ b/spec/models/project_services/chat_message/pipeline_message_spec.rb @@ -42,9 +42,9 @@ describe ChatMessage::PipelineMessage do before do test_commit = double("A test commit", committer: args[:user], title: "A test commit message") - test_project = double("A test project", - commit_by: test_commit, name: args[:project][:name], - web_url: args[:project][:web_url], avatar_url: args[:project][:avatar_url]) + test_project = double("A test project", commit_by: test_commit, name: args[:project][:name], web_url: args[:project][:web_url]) + allow(test_project).to receive(:avatar_url).with(no_args).and_return("/avatar") + allow(test_project).to receive(:avatar_url).with(only_path: false).and_return(args[:project][:avatar_url]) allow(Project).to receive(:find) { test_project } test_pipeline = double("A test pipeline", has_yaml_errors?: has_yaml_errors, |