Rename MattermostNotificationService back to MattermostService

app/models/project.rb

@@ -89,7 +89,7 @@ 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_service, dependent: :destroy
   has_one :jenkins_service, dependent: :destroy
   has_one :jenkins_deprecated_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/service.rb

@@ -222,7 +222,7 @@ class Service < ActiveRecord::Base
       pivotaltracker
       pushover
       redmine
-      mattermost_notification
+      mattermost
       slack
       teamcity
     ]

lib/api/services.rb

@@ -492,7 +492,7 @@ module API
           desc: 'The channel name'
         }
       ],
-      'mattermost-notification' => [
+      'mattermost' => [
         {
           required: true,
           name: :webhook,

spec/lib/gitlab/import_export/all_models.yml

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

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

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

spec/models/project_spec.rb

@@ -22,8 +22,8 @@ describe Project, models: true do
     it { is_expected.to have_many(:protected_branches).dependent(:destroy) }
     it { is_expected.to have_many(:chat_services) }
     it { is_expected.to have_one(:forked_project_link).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) }