* Add UI to vomit on a user from /mod Co-authored-by: MichaelaHunter <michaela1234@gmail.com> * Add backend tweaks for vomitting on users Co-authored-by: Andy Zhao <17884966+Zhao-Andy@users.noreply.github.com> * Validate that users can't moderate articles * Add system spec for article mod page * Satisfy rubocop after rebase * Fix typo and formatting * Use reactable_id instead of user_id Co-authored-by: Michaela Hunter <60361569+MichaelaHunter@users.noreply.github.com> Co-authored-by: Andy Zhao <17884966+Zhao-Andy@users.noreply.github.com>
148 lines
4.4 KiB
Ruby
148 lines
4.4 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe "Reactions", type: :request do
|
|
let(:user) { create(:user) }
|
|
let(:article) { create(:article, user_id: user.id) }
|
|
let(:comment) { create(:comment, commentable: article) }
|
|
let(:max_age) { FastlyRails.configuration.max_age }
|
|
let(:stale_if_error) { FastlyRails.configuration.stale_if_error }
|
|
|
|
describe "GET /reactions?article_id=:article_id" do
|
|
context "when signed in" do
|
|
before do
|
|
sign_in user
|
|
get "/reactions?article_id=#{article.id}"
|
|
end
|
|
|
|
it "returns reactions count for article" do
|
|
expect(response.body).to include("article_reaction_counts")
|
|
end
|
|
|
|
it "does not set cache control headers" do
|
|
expect(response.headers["Surrogate-Key"]).to eq(nil)
|
|
end
|
|
|
|
it "does not set Fastly cache control and surrogate control headers" do
|
|
expect(response.headers.to_hash).not_to include(
|
|
"Cache-Control" => "public, no-cache",
|
|
"Surrogate-Control" => "max-age=#{max_age}, stale-if-error=#{stale_if_error}",
|
|
)
|
|
end
|
|
end
|
|
|
|
context "when signed out" do
|
|
before { get "/reactions?article_id=#{article.id}" }
|
|
|
|
it "returns reactions count for article" do
|
|
expect(response.body).to include("article_reaction_counts")
|
|
end
|
|
|
|
it "sets the surrogate key header equal to params for article" do
|
|
expect(response.headers["Surrogate-Key"]).to eq(controller.params.to_s)
|
|
end
|
|
|
|
it "sets Fastly cache control and surrogate control headers" do
|
|
expect(response.headers.to_hash).to include(
|
|
"Cache-Control" => "public, no-cache",
|
|
"Surrogate-Control" => "max-age=#{max_age}, stale-if-error=#{stale_if_error}",
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "GET /reactions?commentable_id=:article.id&commentable_type=Comment" do
|
|
context "when signed in" do
|
|
before do
|
|
sign_in user
|
|
get "/reactions?commentable_id=#{article.id}&commentable_type=Comment"
|
|
end
|
|
|
|
it "returns positive reaction counts" do
|
|
expect(response.body).to include("positive_reaction_counts")
|
|
end
|
|
|
|
it "does not set surrogate key headers" do
|
|
expect(response.headers["Surrogate-Key"]).to eq(nil)
|
|
end
|
|
|
|
it "does not set Fastly cache control and surrogate control headers" do
|
|
expect(response.headers.to_hash).not_to include(
|
|
"Cache-Control" => "public, no-cache",
|
|
"Surrogate-Control" => "max-age=#{max_age}, stale-if-error=#{stale_if_error}",
|
|
)
|
|
end
|
|
end
|
|
|
|
context "when signed out" do
|
|
before { get "/reactions?commentable_id=#{article.id}&commentable_type=Comment" }
|
|
|
|
it "returns positive reaction counts" do
|
|
expect(response.body).to include("positive_reaction_counts")
|
|
end
|
|
|
|
it "sets the surrogate key header equal to params" do
|
|
expect(response.headers["Surrogate-Key"]).to eq(controller.params.to_s)
|
|
end
|
|
|
|
it "sets Fastly cache control and surrogate control headers" do
|
|
expect(response.headers.to_hash).to include(
|
|
"Cache-Control" => "public, no-cache",
|
|
"Surrogate-Control" => "max-age=#{max_age}, stale-if-error=#{stale_if_error}",
|
|
)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "POST /reactions" do
|
|
let(:trusted_user) { create(:user, :trusted) }
|
|
let(:article_params) do
|
|
{
|
|
reactable_id: article.id,
|
|
reactable_type: "Article",
|
|
category: "like"
|
|
}
|
|
end
|
|
|
|
let(:user_params) do
|
|
{
|
|
reactable_id: user.id,
|
|
reactable_type: "User",
|
|
category: "vomit"
|
|
}
|
|
end
|
|
|
|
context "when reacting to an article" do
|
|
before do
|
|
sign_in user
|
|
post "/reactions", params: article_params
|
|
end
|
|
|
|
it "creates reaction" do
|
|
expect(Reaction.last.reactable_id).to eq(article.id)
|
|
end
|
|
|
|
it "destroys existing reaction" do
|
|
# same route to destroy, so sending POST request again
|
|
post "/reactions", params: article_params
|
|
expect(Reaction.all.size).to eq(0)
|
|
end
|
|
end
|
|
|
|
context "when vomiting on a user" do
|
|
before do
|
|
sign_in trusted_user
|
|
post "/reactions", params: user_params
|
|
end
|
|
|
|
it "creates reaction" do
|
|
expect(Reaction.last.reactable_id).to eq(user.id)
|
|
end
|
|
|
|
it "destroys existing reaction" do
|
|
# same route to destroy, so sending POST request again
|
|
post "/reactions", params: user_params
|
|
expect(Reaction.all.size).to eq(0)
|
|
end
|
|
end
|
|
end
|
|
end
|