Remove user update policy (#983)

* Remove constraint for user update * remove spec
hostolab · Jun 14, 2021 · 38062c3 · 38062c3
1 parent 9fdbf42
commit 38062c3
Show file tree

Hide file tree

Showing 2 changed files with 0 additions and 59 deletions.
diff --git a/app/policies/user_policy.rb b/app/policies/user_policy.rb
@@ -15,17 +15,6 @@ def create?
 
   def update?
     return false unless user.confirmed?
-
-    if user.matches.confirmed.any?
-      raise Pundit::NotAuthorizedError, "Vous ne pouvez pas modifier vos informations actuellement car vous avez confirmé un rendez-vous de vaccination. Votre profil sera anonymisé quelques jours après le RDV."
-    elsif user.matches.pending.any?
-      user.matches.pending.each do |match|
-        if match.confirmable? && !match.expired?
-          raise Pundit::NotAuthorizedError, "Vous ne pouvez pas modifier vos informations actuellement car vous avez une proposition rendez vous de vaccination en cours."
-        end
-      end
-    end
-
     user == record
   end
 

diff --git a/spec/system/user_spec.rb b/spec/system/user_spec.rb
@@ -146,54 +146,6 @@ def signup_submit
       end
     end
 
-    context "with a confirmed match" do
-      let(:campaign) { build(:campaign) }
-      let!(:match) { create(:match, campaign: campaign, confirmed_at: Time.now, user: user) }
-
-      it "it does not allow me to edit personal information" do
-        click_on "Je modifie mes informations"
-        expect(page).not_to have_text("Modifications enregistrées.")
-        expect(page).to have_text("Vous ne pouvez pas modifier vos informations actuellement car vous avez confirmé un rendez-vous de vaccination.")
-      end
-
-      it "it warns about match" do
-        visit profile_url
-        expect(page).to have_text("Vous avez un confirmé un RDV de vaccination")
-      end
-    end
-
-    context "with a pending match" do
-      let(:campaign) { build(:campaign) }
-      let!(:match) { create(:match, campaign: campaign, confirmed_at: nil, expires_at: 10.minutes.since, user: user) }
-
-      it "it does not allow me to edit personal information" do
-        click_on "Je modifie mes informations"
-        expect(page).not_to have_text("Modifications enregistrées.")
-        expect(page).to have_text("Vous ne pouvez pas modifier vos informations actuellement car vous avez une proposition rendez vous de vaccination en cours.")
-      end
-
-      it "it warns about match" do
-        visit profile_url
-        expect(page).to have_text("Nous avons trouvé une dose de vaccin pour vous !")
-      end
-    end
-
-    context "with a pending match" do
-      let(:campaign) { build(:campaign) }
-      let!(:match) { create(:match, campaign: campaign, confirmed_at: nil, expires_at: 10.minutes.since, user: user) }
-
-      it "it does not allow me to edit personal information" do
-        click_on "Je modifie mes informations"
-        expect(page).not_to have_text("Modifications enregistrées.")
-        expect(page).to have_text("Vous ne pouvez pas modifier vos informations actuellement car vous avez une proposition rendez vous de vaccination en cours.")
-      end
-
-      it "it warns about match" do
-        visit profile_url
-        expect(page).to have_text("Nous avons trouvé une dose de vaccin pour vous !")
-      end
-    end
-
     context "with a new campaign" do
       let!(:center) { create(:vaccination_center, :from_paris) }
       before do