Rename SlackNotificationService back to SlackService

parent 1139da27
...@@ -96,7 +96,7 @@ class Project < ActiveRecord::Base ...@@ -96,7 +96,7 @@ class Project < ActiveRecord::Base
has_one :mattermost_slash_commands_service, dependent: :destroy has_one :mattermost_slash_commands_service, dependent: :destroy
has_one :mattermost_notification_service, dependent: :destroy has_one :mattermost_notification_service, dependent: :destroy
has_one :slack_slash_commands_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 :buildkite_service, dependent: :destroy
has_one :bamboo_service, dependent: :destroy has_one :bamboo_service, dependent: :destroy
has_one :teamcity_service, dependent: :destroy has_one :teamcity_service, dependent: :destroy
......
class SlackNotificationService < ChatNotificationService class SlackService < ChatNotificationService
def title def title
'Slack notifications' 'Slack notifications'
end end
...@@ -8,7 +8,7 @@ class SlackNotificationService < ChatNotificationService ...@@ -8,7 +8,7 @@ class SlackNotificationService < ChatNotificationService
end end
def to_param def to_param
'slack_notification' 'slack'
end end
def help def help
......
...@@ -222,7 +222,7 @@ class Service < ActiveRecord::Base ...@@ -222,7 +222,7 @@ class Service < ActiveRecord::Base
pushover pushover
redmine redmine
slack_slash_commands slack_slash_commands
slack_notification slack
teamcity teamcity
] ]
end end
......
...@@ -5,7 +5,7 @@ class MoveSlackServiceToWebhook < ActiveRecord::Migration ...@@ -5,7 +5,7 @@ class MoveSlackServiceToWebhook < ActiveRecord::Migration
DOWNTIME_REASON = 'Move old fields "token" and "subdomain" to one single field "webhook"' DOWNTIME_REASON = 'Move old fields "token" and "subdomain" to one single field "webhook"'
def change def change
SlackNotificationService.all.each do |slack_service| SlackService.all.each do |slack_service|
if ["token", "subdomain"].all? { |property| slack_service.properties.key? property } if ["token", "subdomain"].all? { |property| slack_service.properties.key? property }
token = slack_service.properties['token'] token = slack_service.properties['token']
subdomain = slack_service.properties['subdomain'] subdomain = slack_service.properties['subdomain']
......
class ChangeSlackServiceToSlackNotificationService < ActiveRecord::Migration class ChangeSlackServiceToSlackNotificationService < ActiveRecord::Migration
include Gitlab::Database::MigrationHelpers include Gitlab::Database::MigrationHelpers
DOWNTIME = true DOWNTIME = false
DOWNTIME_REASON = 'Rename SlackService to SlackNotificationService'
def up # This migration is a no-op, as it existed in an RC but we renamed
execute("UPDATE services SET type = 'SlackNotificationService' WHERE type = 'SlackService'") # SlackNotificationService back to SlackService:
end # https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8191#note_20310845
def change
def down
execute("UPDATE services SET type = 'SlackService' WHERE type = 'SlackNotificationService'")
end end
end end
...@@ -480,7 +480,7 @@ module API ...@@ -480,7 +480,7 @@ module API
desc: 'The description of the tracker' desc: 'The description of the tracker'
} }
], ],
'slack-notification' => [ 'slack' => [
{ {
required: true, required: true,
name: :webhook, name: :webhook,
......
...@@ -2,8 +2,8 @@ require 'spec_helper' ...@@ -2,8 +2,8 @@ require 'spec_helper'
feature 'Projects > Slack service > Setup events', feature: true do feature 'Projects > Slack service > Setup events', feature: true do
let(:user) { create(:user) } let(:user) { create(:user) }
let(:service) { SlackNotificationService.new } let(:service) { SlackService.new }
let(:project) { create(:project, slack_notification_service: service) } let(:project) { create(:project, slack_service: service) }
background do background do
service.fields service.fields
......
...@@ -137,7 +137,7 @@ project: ...@@ -137,7 +137,7 @@ project:
- assembla_service - assembla_service
- asana_service - asana_service
- gemnasium_service - gemnasium_service
- slack_notification_service - slack_service
- mattermost_notification_service - mattermost_notification_service
- buildkite_service - buildkite_service
- bamboo_service - bamboo_service
......
require 'spec_helper' require 'spec_helper'
describe SlackNotificationService, models: true do describe SlackService, models: true do
it_behaves_like "slack or mattermost notifications" it_behaves_like "slack or mattermost notifications"
end end
...@@ -21,7 +21,7 @@ describe Project, models: true do ...@@ -21,7 +21,7 @@ describe Project, models: true do
it { is_expected.to have_many(:hooks).dependent(:destroy) } it { is_expected.to have_many(:hooks).dependent(:destroy) }
it { is_expected.to have_many(:protected_branches).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(:forked_project_link).dependent(:destroy) }
it { is_expected.to have_one(:slack_notification_service).dependent(:destroy) } it { is_expected.to have_one(:slack_service).dependent(:destroy) }
it { is_expected.to have_one(:mattermost_notification_service).dependent(:destroy) } it { is_expected.to have_one(:mattermost_notification_service).dependent(:destroy) }
it { is_expected.to have_one(:pushover_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_one(:asana_service).dependent(:destroy) }
......
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