Merge branch 'EN-8090-fix-slack-mention-on-association-account-creation'

ReseauEntourage · Mar 6, 2025 · fa60513 · fa60513
2 parents fd785ae + 088fa25
commit fa60513
Show file tree

Hide file tree

Showing 22 changed files with 13 additions and 94 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -36,6 +36,7 @@ class User < ApplicationRecord
   before_save :slack_id_no_empty
   after_save :clean_up_passwords, if: :saved_change_to_encrypted_password?
   after_save :sync_newsletter, if: :saved_change_to_email?
+  after_save :signal_association
 
   has_many :followings, -> { where active: true }
   has_many :subscriptions, through: :followings, source: :partner
@@ -405,7 +406,14 @@ def newsletter_subscription= newsletter_subscription
 
     email_preference = EmailPreference.find_by(user: self, email_category_id: category_id) || email_preferences.build(email_category_id: category_id)
     email_preference.subscribed = ActiveModel::Type::Boolean.new.cast(newsletter_subscription)
+  end
+
+  def signal_association
+    return unless address.present?
+    return unless saved_change_to_address_id? || saved_change_to_goal?
+    return unless goal_association?
 
+    SlackServices::SignalAssociationCreation.new(user: self).notify
   end
 
   def sync_newsletter

diff --git a/app/services/user_services/public_user_builder.rb b/app/services/user_services/public_user_builder.rb
@@ -44,7 +44,6 @@ def update(user:, platform: nil)
 
       if user.update(params)
         signal_blocked_user(user)
-        signal_association(user)
 
         callback.on_success.try(:call, user)
       else

diff --git a/app/services/user_services/user_builder.rb b/app/services/user_services/user_builder.rb
@@ -26,8 +26,6 @@ def create(send_sms: false, sms_code: nil)
         UserServices::SMSSender.new(user: user).send_welcome_sms(sms_code) if send_sms
         MemberMailer.welcome(user).deliver_later if user.email.present?
 
-        signal_association(user)
-
         callback.on_success.try(:call, user)
       else
         return callback.on_duplicate(user) if User.where(phone: params[:phone]).count>0
@@ -49,13 +47,6 @@ def signal_blocked_user user
       SlackServices::SignalUserCreation.new(user: user, blocked_user_ids: blocked_user_ids).notify
     end
 
-    def signal_association user
-      return unless user.saved_change_to_goal?
-      return unless user.goal_association?
-
-      SlackServices::SignalAssociationCreation.new(user: user).notify
-    end
-
     private
     attr_reader :params, :callback
 

diff --git a/spec/acceptance/api/v1/contributions_controller_spec.rb b/spec/acceptance/api/v1/contributions_controller_spec.rb
@@ -159,12 +159,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report contribution' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/entourages/invitations_controller_spec.rb b/spec/acceptance/api/v1/entourages/invitations_controller_spec.rb
@@ -29,10 +29,6 @@
       }
     }.to_json }
 
-    ENV['SLACK_WEBHOOK_URL'] = "https://url.to.slack.com"
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       let!(:join_request) { FactoryBot.create(:join_request, joinable: entourage, user: user, status: :accepted) }
 

diff --git a/spec/acceptance/api/v1/entourages_controller_spec.rb b/spec/acceptance/api/v1/entourages_controller_spec.rb
@@ -399,12 +399,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report entourage' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/neighborhoods/chat_messages_controller_spec.rb b/spec/acceptance/api/v1/neighborhoods/chat_messages_controller_spec.rb
@@ -140,12 +140,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report neighborhood' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/neighborhoods_controller_spec.rb b/spec/acceptance/api/v1/neighborhoods_controller_spec.rb
@@ -155,12 +155,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report neighborhood' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/outings/chat_messages_controller_spec.rb b/spec/acceptance/api/v1/outings/chat_messages_controller_spec.rb
@@ -170,12 +170,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report outing' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/solicitations_controller_spec.rb b/spec/acceptance/api/v1/solicitations_controller_spec.rb
@@ -159,12 +159,6 @@
       }
     }.to_json }
 
-
-    ENV['ADMIN_HOST'] = 'https://this.is.local'
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context '201' do
       example_request 'Report solicitation' do
         expect(response_status).to eq(201)

diff --git a/spec/acceptance/api/v1/users_controller_spec.rb b/spec/acceptance/api/v1/users_controller_spec.rb
@@ -5,18 +5,6 @@
   explanation "Users"
   header "Content-Type", "application/json"
 
-  ENV['ADMIN_HOST'] = 'https://this.is.local'
-  ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-  before(:each) {
-    ENV['SLACK_WEBHOOK_URL'] = 'https://url.to.slack.com'
-    stub_request(:post, "https://url.to.slack.com").to_return(status: 200)
-  }
-
-  after(:each) {
-    ENV['SLACK_WEBHOOK_URL'] = nil
-  }
-
   post '/api/v1/login' do
     route_summary "Login"
     # route_description "no description"

diff --git a/spec/controllers/api/v1/contributions_controller_spec.rb b/spec/controllers/api/v1/contributions_controller_spec.rb
@@ -458,8 +458,6 @@
 
     ENV['SLACK_SIGNAL_NEIGHBORHOOD_WEBHOOK'] = '{"url":"https://url.to.slack.com","channel":"channel","username":"signal-contribution"}'
 
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalContribution).to receive(:notify)

diff --git a/spec/controllers/api/v1/conversations_controller_spec.rb b/spec/controllers/api/v1/conversations_controller_spec.rb
@@ -464,10 +464,6 @@
 
     let!(:join_request) { FactoryBot.create(:join_request, joinable: conversation, user: user, status: :accepted) }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalConversation).to receive(:notify)

diff --git a/spec/controllers/api/v1/entourages_controller_spec.rb b/spec/controllers/api/v1/entourages_controller_spec.rb
@@ -965,9 +965,6 @@
 
     ENV['SLACK_SIGNAL_GROUP_WEBHOOK'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
 
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalGroup).to receive(:notify)

diff --git a/spec/controllers/api/v1/neighborhoods/chat_messages_controller_spec.rb b/spec/controllers/api/v1/neighborhoods/chat_messages_controller_spec.rb
@@ -470,10 +470,6 @@
     let(:chat_message) { create :chat_message, messageable: neighborhood }
     let!(:join_request) { FactoryBot.create(:join_request, joinable: neighborhood, user: user, status: "accepted") }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "correct messageable" do
       before {
         expect_any_instance_of(SlackServices::SignalNeighborhoodChatMessage).to receive(:notify)

diff --git a/spec/controllers/api/v1/neighborhoods_controller_spec.rb b/spec/controllers/api/v1/neighborhoods_controller_spec.rb
@@ -755,10 +755,6 @@
   describe 'POST #report' do
     let(:neighborhood) { create :neighborhood }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalNeighborhood).to receive(:notify)

diff --git a/spec/controllers/api/v1/outings/chat_messages_controller_spec.rb b/spec/controllers/api/v1/outings/chat_messages_controller_spec.rb
@@ -493,10 +493,6 @@
     let(:chat_message) { create :chat_message, messageable: outing }
     let!(:join_request) { FactoryBot.create(:join_request, joinable: outing, user: user, status: "accepted") }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "correct messageable" do
       before {
         expect_any_instance_of(SlackServices::SignalOutingChatMessage).to receive(:notify)

diff --git a/spec/controllers/api/v1/outings_controller_spec.rb b/spec/controllers/api/v1/outings_controller_spec.rb
@@ -864,10 +864,6 @@
   describe 'POST #report' do
     let(:outing) { create :outing }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalOuting).to receive(:notify)

diff --git a/spec/controllers/api/v1/solicitations_controller_spec.rb b/spec/controllers/api/v1/solicitations_controller_spec.rb
@@ -459,8 +459,6 @@
 
     ENV['SLACK_SIGNAL_NEIGHBORHOOD_WEBHOOK'] = '{"url":"https://url.to.slack.com","channel":"channel","username":"signal-solicitation"}'
 
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalSolicitation).to receive(:notify)

diff --git a/spec/controllers/api/v1/users_controller_spec.rb b/spec/controllers/api/v1/users_controller_spec.rb
@@ -1110,10 +1110,6 @@ def event
     let(:reported_user)  { create :public_user }
     let(:result) { JSON.parse(response.body) }
 
-    ENV['SLACK_SIGNAL'] = '{"url":"https://url.to.slack.com","channel":"channel"}'
-
-    before { stub_request(:post, "https://url.to.slack.com").to_return(status: 200) }
-
     context "valid params" do
       before {
         expect_any_instance_of(SlackServices::SignalUser).to receive(:notify)

diff --git a/spec/rails_helper.rb b/spec/rails_helper.rb
@@ -20,6 +20,7 @@
 
     stub_request(:any, /.*api.mailjet.com.*/).to_return(status: 200, body: { id: 1 }.to_json, headers: {})
     stub_request(:any, /.*api.openai.*/).to_return(status: 200, body: "{}", headers: {})
+    stub_request(:any, "https://url.to.slack.com").to_return(status: 200)
 
     # deactivate slack_trace notifications
     SlackServices::StackTrace.any_instance.stub(:notify).and_return(nil)

diff --git a/spec/spec_helper.rb b/spec/spec_helper.rb
@@ -21,8 +21,11 @@
   end
 end
 
-ENV["DISABLE_CRYPT"]="TRUE"
+ENV['ADMIN_HOST'] = 'https://this.is.local'
+ENV["DISABLE_CRYPT"] = "TRUE"
 ENV['ENTOURAGE_SECRET'] = 'test_entourage_secret'
 ENV["SLACK_DEFAULT_INTERLOCUTOR"] = "laure"
 ENV["MODERATOR_PHONE"] = "+33768037348"
+ENV['SLACK_WEBHOOK_URL'] = "https://url.to.slack.com"
+ENV["SLACK_SIGNAL"] = '{"url":"https://url.to.slack.com","channel":"channel","channel-associations":"channel-associations"}'
 API_HOST = 'api.entourage.test'