Return Error messages for all integration settings

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

We changed the JiraService testing behavior to include the error message (gitlab-ce#32483) in the test response and we include this response in the error message displayed to users. But we have also other services that need to be checked and should return error messages as well. The services that have enabled testing are:

Service Method Test using
AsanaService execute asana gem
AssemblaService execute HTTParty
BambooService execute HTTParty
BugzillaService IssueTrackerService#execute HTTParty (exception rescued)
BuildkiteService execute service hook
CampfireService execute HTTParty
ChatNotificationService execute slack-notifier gem
CustomIssueTrackerService IssueTrackerService#execute HTTParty (exception rescued)
DroneCiService execute service hook
EmailsOnPushService execute send email
ExternalWikiService execute HTTParty
FlowdockService execute flowdock gem
GemnasiumService execute gemnasium-gitlab-service gem
GitlabIssueTrackerService IssueTrackerService#execute HTTParty (exception rescued)
HipchatService test hipchat gem
IrkerService execute TCPSocket
KubernetesService test kubeclient gem
MattermostService ChatNotificationService#execute slack-notifier gem
MicrosoftTeamsService ChatNotificationService#execute slack-notifier gem
PipelinesEmailService execute Notify mailer
PivotaltrackerService execute HTTParty
PrometheusService test Gitlab::PrometheusClient
PushoverService execute HTTParty
RedmineService IssueTrackerService#execute HTTParty (exception rescued)
SlackService ChatNotificationService#execute slack-notifier gem
TeamcityService execute HTTParty

The services that have testing disabled

  • DeploymentService
  • MattermostSlashCommandsService
  • MockCiService
  • MockDeploymentService
  • MockMonitoringService
  • SlackSlashCommandsService, SlashCommandsService

/cc @smcgivern @victorwu

Edited by 🤖 GitLab Bot 🤖