Merge branch 'fix/25843' into 'master'

Rename SlackNotificationService back to SlackService

## What does this MR do?

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8191#note_20310845

## What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/25843

/cc @yorickpeterse @stanhu @smcgivern @felipe_artur @ayufan

See merge request !8208

app/models/project.rb

@@ -94,9 +94,9 @@ class Project < ActiveRecord::Base
   has_one :asana_service, dependent: :destroy
   has_one :gemnasium_service, dependent: :destroy
   has_one :mattermost_slash_commands_service, dependent: :destroy
-  has_one :mattermost_notification_service, dependent: :destroy
+  has_one :mattermost_service, dependent: :destroy
   has_one :slack_slash_commands_service, dependent: :destroy
-  has_one :slack_notification_service, dependent: :destroy
+  has_one :slack_service, dependent: :destroy
   has_one :buildkite_service, dependent: :destroy
   has_one :bamboo_service, dependent: :destroy
   has_one :teamcity_service, dependent: :destroy

app/models/project_services/mattermost_notification_service.rb → app/models/project_services/mattermost_service.rb

-class MattermostNotificationService < ChatNotificationService
+class MattermostService < ChatNotificationService
   def title
     'Mattermost notifications'
   end
@@ -8,7 +8,7 @@ class MattermostNotificationService < ChatNotificationService
   end
 
   def to_param
-    'mattermost_notification'
+    'mattermost'
   end
 
   def help

app/models/project_services/slack_notification_service.rb → app/models/project_services/slack_service.rb

-class SlackNotificationService < ChatNotificationService
+class SlackService < ChatNotificationService
   def title
     'Slack notifications'
   end
@@ -8,7 +8,7 @@ class SlackNotificationService < ChatNotificationService
   end
 
   def to_param
-    'slack_notification'
+    'slack'
   end
 
   def help

app/models/service.rb

@@ -216,13 +216,13 @@ class Service < ActiveRecord::Base
       jira
       kubernetes
       mattermost_slash_commands
-      mattermost_notification
+      mattermost
       pipelines_email
       pivotaltracker
       pushover
       redmine
       slack_slash_commands
-      slack_notification
+      slack
       teamcity
     ]
   end

db/migrate/20141006143943_move_slack_service_to_webhook.rb

@@ -5,7 +5,7 @@ class MoveSlackServiceToWebhook < ActiveRecord::Migration
   DOWNTIME_REASON = 'Move old fields "token" and "subdomain" to one single field "webhook"'
 
   def change
-    SlackNotificationService.all.each do |slack_service|
+    SlackService.all.each do |slack_service|
       if ["token", "subdomain"].all? { |property| slack_service.properties.key? property }
         token = slack_service.properties['token']
         subdomain = slack_service.properties['subdomain']

db/migrate/20161213172958_change_slack_service_to_slack_notification_service.rb

 class ChangeSlackServiceToSlackNotificationService < ActiveRecord::Migration
   include Gitlab::Database::MigrationHelpers
 
-  DOWNTIME = true
-  DOWNTIME_REASON = 'Rename SlackService to SlackNotificationService'
+  DOWNTIME = false
 
-  def up
-    execute("UPDATE services SET type = 'SlackNotificationService' WHERE type = 'SlackService'")
-  end
-
-  def down
-    execute("UPDATE services SET type = 'SlackService' WHERE type = 'SlackNotificationService'")
+  # This migration is a no-op, as it existed in an RC but we renamed
+  # SlackNotificationService back to SlackService:
+  #   https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8191#note_20310845
+  def change
   end
 end

lib/api/services.rb

@@ -480,7 +480,7 @@ module API
           desc: 'The description of the tracker'
         }
       ],
-      'slack-notification' => [
+      'slack' => [
         {
           required: true,
           name: :webhook,
@@ -500,7 +500,7 @@ module API
           desc: 'The channel name'
         }
       ],
-      'mattermost-notification' => [
+      'mattermost' => [
         {
           required: true,
           name: :webhook,

spec/features/projects/services/slack_service_spec.rb

@@ -2,8 +2,8 @@ require 'spec_helper'
 
 feature 'Projects > Slack service > Setup events', feature: true do
   let(:user) { create(:user) }
-  let(:service) { SlackNotificationService.new }
-  let(:project) { create(:project, slack_notification_service: service) }
+  let(:service) { SlackService.new }
+  let(:project) { create(:project, slack_service: service) }
 
   background do
     service.fields

spec/lib/gitlab/import_export/all_models.yml

@@ -137,8 +137,8 @@ project:
 - assembla_service
 - asana_service
 - gemnasium_service
-- slack_notification_service
-- mattermost_notification_service
+- slack_service
+- mattermost_service
 - buildkite_service
 - bamboo_service
 - teamcity_service

spec/models/project_services/slack_notification_service_spec.rb → spec/models/project_services/mattermost_service_spec.rb

 require 'spec_helper'
 
-describe SlackNotificationService, models: true do
+describe MattermostService, models: true do
   it_behaves_like "slack or mattermost notifications"
 end

spec/models/project_services/mattermost_notification_service_spec.rb → spec/models/project_services/slack_service_spec.rb

 require 'spec_helper'
 
-describe MattermostNotificationService, models: true do
+describe SlackService, models: true do
   it_behaves_like "slack or mattermost notifications"
 end

spec/models/project_spec.rb

@@ -21,8 +21,8 @@ describe Project, models: true do
     it { is_expected.to have_many(:hooks).dependent(:destroy) }
     it { is_expected.to have_many(:protected_branches).dependent(:destroy) }
     it { is_expected.to have_one(:forked_project_link).dependent(:destroy) }
-    it { is_expected.to have_one(:slack_notification_service).dependent(:destroy) }
-    it { is_expected.to have_one(:mattermost_notification_service).dependent(:destroy) }
+    it { is_expected.to have_one(:slack_service).dependent(:destroy) }
+    it { is_expected.to have_one(:mattermost_service).dependent(:destroy) }
     it { is_expected.to have_one(:pushover_service).dependent(:destroy) }
     it { is_expected.to have_one(:asana_service).dependent(:destroy) }
     it { is_expected.to have_many(:boards).dependent(:destroy) }