diff --git a/app/controllers/reviewable_claimed_topics_controller.rb b/app/controllers/reviewable_claimed_topics_controller.rb index 37f66cd9b5..01f2b8e37b 100644 --- a/app/controllers/reviewable_claimed_topics_controller.rb +++ b/app/controllers/reviewable_claimed_topics_controller.rb @@ -7,6 +7,9 @@ class ReviewableClaimedTopicsController < ApplicationController topic = Topic.find_by(id: params[:reviewable_claimed_topic][:topic_id]) guardian.ensure_can_claim_reviewable_topic!(topic) ReviewableClaimedTopic.create(user_id: current_user.id, topic_id: topic.id) + topic.reviewables.find_each do |reviewable| + reviewable.log_history(:claimed, current_user) + end render json: success_json rescue ActiveRecord::RecordNotUnique # This is just in case the validation fails under concurrency @@ -19,6 +22,9 @@ class ReviewableClaimedTopicsController < ApplicationController guardian.ensure_can_claim_reviewable_topic!(topic) ReviewableClaimedTopic.where(topic_id: topic.id).delete_all + topic.reviewables.find_each do |reviewable| + reviewable.log_history(:unclaimed, current_user) + end render json: success_json end diff --git a/app/models/reviewable_history.rb b/app/models/reviewable_history.rb index 1c0eebc5f7..98b6636500 100644 --- a/app/models/reviewable_history.rb +++ b/app/models/reviewable_history.rb @@ -8,7 +8,9 @@ class ReviewableHistory < ActiveRecord::Base @types ||= Enum.new( created: 0, transitioned: 1, - edited: 2 + edited: 2, + claimed: 3, + unclaimed: 4 ) end diff --git a/spec/requests/reviewable_claimed_topics_controller_spec.rb b/spec/requests/reviewable_claimed_topics_controller_spec.rb index 81dc06e3c7..3d3a29cbb8 100644 --- a/spec/requests/reviewable_claimed_topics_controller_spec.rb +++ b/spec/requests/reviewable_claimed_topics_controller_spec.rb @@ -7,6 +7,7 @@ describe ReviewableClaimedTopicsController do describe '#create' do fab!(:topic) { Fabricate(:topic) } + fab!(:reviewable) { Fabricate(:reviewable_flagged_post, topic: topic) } let(:params) do { reviewable_claimed_topic: { topic_id: topic.id } } end @@ -32,6 +33,7 @@ describe ReviewableClaimedTopicsController do post "/reviewable_claimed_topics.json", params: params expect(response.code).to eq("200") expect(ReviewableClaimedTopic.where(user_id: moderator.id, topic_id: topic.id).exists?).to eq(true) + expect(topic.reviewables.first.history.where(reviewable_history_type: ReviewableHistory.types[:claimed]).size).to eq(1) end it "won't an error if you claim twice" do @@ -45,7 +47,9 @@ describe ReviewableClaimedTopicsController do end describe '#destroy' do - fab!(:claimed) { Fabricate(:reviewable_claimed_topic) } + fab!(:topic) { Fabricate(:topic) } + fab!(:reviewable) { Fabricate(:reviewable_flagged_post, topic: topic) } + fab!(:claimed) { Fabricate(:reviewable_claimed_topic, topic: topic) } before do sign_in(moderator) @@ -66,6 +70,7 @@ describe ReviewableClaimedTopicsController do delete "/reviewable_claimed_topics/#{claimed.topic_id}.json" expect(response.code).to eq("200") expect(ReviewableClaimedTopic.where(topic_id: claimed.topic_id).exists?).to eq(false) + expect(topic.reviewables.first.history.where(reviewable_history_type: ReviewableHistory.types[:unclaimed]).size).to eq(1) end end end