退会時に分報チャンネルを自動で削除する

app/controllers/admin/users_controller.rb

@@ -22,7 +22,7 @@ def edit; end
   def update
     if @user.update(user_params)
       destroy_subscription(@user)
-      Newspaper.publish(:retirement_create, @user)
+      Newspaper.publish(:retirement_create, @user) if @user.saved_change_to_retired_on?
       redirect_to admin_users_url, notice: 'ユーザー情報を更新しました。'
     else
       render :edit
@@ -61,7 +61,7 @@ def user_params
       :auto_retire,
       :profile_image, :profile_name, :profile_job, :mentor,
       :profile_text, { authored_books_attributes: %i[id title url cover _destroy] },
-      :country_code, :subdivision_code, discord_profile_attributes: %i[account_name times_url]
+      :country_code, :subdivision_code, discord_profile_attributes: %i[account_name times_url times_id]
     )
   end
 

app/models/discord/server.rb

@@ -17,6 +17,14 @@ def create_text_channel(name:, parent: nil)
         nil
       end
 
+      def delete_text_channel(channel_id)
+        response = Discordrb::API::Channel.delete(authorize_token, channel_id)
+        response.code == 200
+      rescue Discordrb::Errors::CodeError => e
+        log_error(e)
+        nil
+      end
+
       def find_by(id:, token:)
         return nil unless enabled?
 

app/models/times_channel_destroyer.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+class TimesChannelDestroyer
+  def call(user)
+    return unless user.discord_profile.times_id
+
+    if Discord::Server.delete_text_channel(user.discord_profile.times_id)
+      user.discord_profile.update!(times_id: nil)
+    else
+      Rails.logger.warn "[Discord API] #{user.login_name}の分報チャンネルが削除できませんでした。"
+    end
+  end
+end

app/models/unfinished_data_destroyer.rb

@@ -2,8 +2,6 @@
 
 class UnfinishedDataDestroyer
   def call(user)
-    return unless user.saved_change_to_retired_on?
-
     Product.where(user: user).unchecked.destroy_all
     Report.where(user: user).wip.destroy_all
     user.update(job_seeking: false)

app/views/users/form/_sns.html.slim

@@ -26,6 +26,7 @@
             | こちら
           span.a-help
             i.fa-solid.fa-question
+  = discord_profile_fields.hidden_field :times_id
 
 .form-item#form-github-account
   = f.label :github_account, class: 'a-form-label'

config/initializers/newspaper.rb

@@ -59,6 +59,7 @@
   Newspaper.subscribe(:question_update, ai_answer_creator)
 
   Newspaper.subscribe(:retirement_create, UnfinishedDataDestroyer.new)
+  Newspaper.subscribe(:retirement_create, TimesChannelDestroyer.new)
 
   question_notifier = QuestionNotifier.new
   Newspaper.subscribe(:question_create, question_notifier)

test/models/discord/server_test.rb

@@ -96,6 +96,28 @@ class ServerTest < ActiveSupport::TestCase
       end
     end
 
+    test '.delete_text_channel' do
+      VCR.use_cassette 'discord/server/delete_text_channel' do
+        assert Discord::Server.delete_text_channel('987654321987654321')
+      end
+    end
+
+    test '.delete_text_channel with error' do
+      logs = []
+      Rails.logger.stub(:error, ->(message) { logs << message }) do
+        VCR.use_cassette 'discord/server/delete_text_channel_with_unknown_channel_id' do
+          assert_nil Discord::Server.delete_text_channel('12345')
+          assert_equal '[Discord API] Unknown Channel', logs.pop
+        end
+
+        VCR.use_cassette 'discord/server/delete_text_channel_with_unauthorized' do
+          Discord::Server.authorize_token = 'Bot invalid token'
+          assert_nil Discord::Server.delete_text_channel('987654321987654321')
+          assert_equal '[Discord API] 401: Unauthorized', logs.pop
+        end
+      end
+    end
+
     test '.enabled?' do
       Discord::Server.guild_id = '1234567890123456789'
       Discord::Server.authorize_token = 'Bot valid token'

test/models/times_channel_destroyer_test.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+require 'test_helper'
+
+class TimesChannelCreatorTest < ActiveSupport::TestCase
+  test '#call' do
+    logs = []
+    user = users(:hajime)
+    user.discord_profile.update!(times_id: '987654321987654321')
+    Rails.logger.stub(:warn, ->(message) { logs << message }) do
+      Discord::Server.stub(:delete_text_channel, true) do
+        TimesChannelDestroyer.new.call(user)
+      end
+      assert_nil user.discord_profile.times_id
+      assert_nil logs.last
+    end
+  end
+
+  test '#call with failure' do
+    logs = []
+    user = users(:hajime)
+    user.discord_profile.update!(times_id: '987654321987654321')
+    Rails.logger.stub(:warn, ->(message) { logs << message }) do
+      Discord::Server.stub(:delete_text_channel, nil) do
+        TimesChannelDestroyer.new.call(user)
+      end
+      assert_equal '987654321987654321', user.discord_profile.times_id
+      assert_equal "[Discord API] #{user.login_name}の分報チャンネルが削除できませんでした。", logs.last
+    end
+  end
+end

test/system/admin/users_test.rb

@@ -132,15 +132,19 @@ class Admin::UsersTest < ApplicationSystemTestCase
 
   test 'make user retired' do
     user = users(:hatsuno)
+    user.discord_profile.update!(times_id: '987654321987654321')
     date = Date.current
-    VCR.use_cassette 'subscription/update' do
-      visit_with_auth edit_admin_user_path(user.id), 'komagata'
-      check '退会済', allow_label_click: true
-      fill_in 'user_retired_on', with: date
-      click_on '更新する'
+    Discord::Server.stub(:delete_text_channel, true) do
+      VCR.use_cassette 'subscription/update' do
+        visit_with_auth edit_admin_user_path(user.id), 'komagata'
+        check '退会済', allow_label_click: true
+        fill_in 'user_retired_on', with: date
+        click_on '更新する'
+      end
     end
     assert_text 'ユーザー情報を更新しました。'
     assert_equal date, user.reload.retired_on
+    assert_nil user.discord_profile.times_id
 
     assert_requested(:post, "https://api.stripe.com/v1/subscriptions/#{user.subscription_id}") do |req|
       req.body.include?('cancel_at_period_end=true')