Commit 9a286f3c authored by Arturo Herrero's avatar Arturo Herrero

Clarify default message on double object

parent e4b997a5
...@@ -74,6 +74,8 @@ module Integrations ...@@ -74,6 +74,8 @@ module Integrations
def author_web_url(jira_issue) def author_web_url(jira_issue)
# There are differences between Jira Cloud and Jira Server URLs and responses. # There are differences between Jira Cloud and Jira Server URLs and responses.
# accountId is only available on Jira Cloud.
# https://community.atlassian.com/t5/Jira-Questions/How-to-find-account-id-on-jira-on-premise/qaq-p/1168652
if jira_issue.reporter.try(:accountId) if jira_issue.reporter.try(:accountId)
"#{jira_issue.client.options[:site]}people/#{jira_issue.reporter.accountId}" "#{jira_issue.client.options[:site]}people/#{jira_issue.reporter.accountId}"
else else
......
...@@ -8,7 +8,7 @@ RSpec.describe Integrations::Jira::IssueEntity do ...@@ -8,7 +8,7 @@ RSpec.describe Integrations::Jira::IssueEntity do
let(:reporter) do let(:reporter) do
double( double(
'displayName' => 'reporter', 'displayName' => 'reporter',
'name' => double 'name' => double # Default to Jira Server issue response, Jira Cloud replaces name with accountId
) )
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment