* schema file undelete description * feat: v1 of the script * Flesh out remaining enums under their categories * complete UsersSettings data update script * complete DUS for relevant attributes in users and profiles tables * complete DUS for users_notification_settings * alphabetize user_settings sql file * safeguard against null values for "null: false" settings * Set up actual UsersSettings DUS and specs files * fix broken DUS script * complete specs for UsersSetting DUS * Address QA of specs * complete specs for users_notification_settings DUS * fix the typos (thanks Julianna!) * begin implementation * still building * add missing attribute "email_membership_newsletter" * complete sync code (except race condition for user profile) * complete implementation, remains tests * Address PR review and fix Travis fails * remove superfluous Profile.new * fix travis fails * feat: update the users_notification_setting attributes from the user model * feat: use the config fonts enums to display the fonts * feat: loop through the keys * fix profile = nil blowing up; add specs for notification_setting model * remove unneeded spec * remove feed validation until after sync code removed; fixes feed_import spec failures * remove spec associated with feed_url validation in user_setting model * fix failing spec 😅 * add TODO * feat: set the user settings in the user controller and use it in the customization form * feat: move some update logic to the users settings controller thats being used from customization * feat: show the updated values form the users_settingd and not the user instance * Generalize redirect back to current tab * still trying to reflect changed theme upon refresh * customizations take effect on refresh * remove 'with_feed' scope from user model Co-authored-by: Jamie Gaskins <jamie@forem.com> * start with takeover for fields previously in profiles table * Takeover code for `publishing_from_rss` section in Settings (#13914) * implement takeover code part 1 * implement takeover code * fix feed fetch * need rhymes help * complete implementation; specs pending * fix STUPID omission that caused so many headaches 😫 * implement profile fields pointing to users_settings 🎉 * run migrations * implement inbox type & guidelines takeover code; specs pending (#13911) * Point changes in notification settings to `users_notification_settings` table (#13910) * implement takeover code; remains specs * address PR feedback; remove related sync code * address PR review feedback * need help with routing and specs * address pr review * addressing pr review * Treat implementation edge cases and omissions 😅 * fix uncommented comment * fixing implementation cases * address more PR review feedback * fixing notifications use-cases * refactor settings controller * more pr review changes * solving bugs * fix broken onboarding * handle eperience_level calls * more fixes * remove unneeded mappings * add To-dos for quety updates * remove done TODO * purge done TODOs * update notification_settings-related queries * start fixing specs * fixing specs * fix notification and lrg_forem specs * fixing broken specs * still fixing * fix line dif and remove reloads from user.rb * run specs * silence bullet and other fixes * remove setting migration scripts and specs, fix more settings for specs * handle missing user for article builder and fix notification specs * fix some final controller specs and re-add incorrectly removed specs * remove deprecated data update scripts and related workers, put travis back * refactor admin tags mods controller, write/move specs for users notifications settings controller * schema cleanup and other small refactors for consistency * set field we can invalidate in spec via active record instead of at the db level * remove I think an uneccessary hook call from subscribe_to_mailchimp_newsletter * use bnefore_create to setup settings, please dont blow up the test suite * mailchimp bot fix * remove decorator in favor of single model method Co-authored-by: Arit Amana <msarit@gmail.com> Co-authored-by: Ridhwana <ridhwana.khan16@gmail.com> Co-authored-by: Arit Amana <32520970+msarit@users.noreply.github.com> Co-authored-by: Jamie Gaskins <jamie@forem.com>
330 lines
10 KiB
Ruby
330 lines
10 KiB
Ruby
require "rails_helper"
|
|
|
|
RSpec.describe "Reactions", type: :request do
|
|
let(:user) { create(:user) }
|
|
let(:article) { create(:article, user: user) }
|
|
let(:comment) { create(:comment, commentable: article) }
|
|
|
|
let(:max_age) { 1.day.to_i }
|
|
let(:stale_if_error) { 26_400 }
|
|
|
|
describe "GET /reactions?article_id=:article_id" do
|
|
before do
|
|
create(:reaction, reactable: article, user: user, points: 1)
|
|
end
|
|
|
|
context "when signed in" do
|
|
before do
|
|
sign_in user
|
|
|
|
get reactions_path(article_id: article.id)
|
|
end
|
|
|
|
it "returns the correct json response" do
|
|
result = response.parsed_body
|
|
|
|
expect(result["current_user"]).to eq("id" => user.id)
|
|
expected_reactions_counts = [
|
|
{ "category" => "like", "count" => 1 },
|
|
{ "category" => "readinglist", "count" => 0 },
|
|
{ "category" => "unicorn", "count" => 0 },
|
|
]
|
|
expect(result["article_reaction_counts"]).to eq(expected_reactions_counts)
|
|
expect(result["reactions"].to_json).to eq(user.reactions.where(reactable: article).to_json)
|
|
end
|
|
|
|
it "does not set Surrogate-Key cache control headers" do
|
|
expect(response.headers["Surrogate-Key"]).to eq(nil)
|
|
end
|
|
|
|
it "does not set X-Accel-Expires headers" do
|
|
expect(response.headers["X-Accel-Expires"]).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_path(article_id: article.id) }
|
|
|
|
it "returns the correct json response" do
|
|
result = response.parsed_body
|
|
|
|
expect(result["current_user"]).to eq("id" => nil)
|
|
expected_reactions = [
|
|
{ "category" => "like", "count" => 1 },
|
|
{ "category" => "readinglist", "count" => 0 },
|
|
{ "category" => "unicorn", "count" => 0 },
|
|
]
|
|
expect(result["article_reaction_counts"]).to eq(expected_reactions)
|
|
expect(result["reactions"]).to be_empty
|
|
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 the x-accel-expires header equal to max-age for article" do
|
|
expect(response.headers["X-Accel-Expires"]).to eq(max_age.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=Article" do
|
|
before do
|
|
create(:reaction, reactable: comment, user: user, points: 1)
|
|
end
|
|
|
|
context "when signed in" do
|
|
before do
|
|
sign_in user
|
|
|
|
get reactions_path(commentable_id: article.id, commentable_type: "Article")
|
|
end
|
|
|
|
it "returns the correct json response" do
|
|
expect(response.parsed_body).to eq(
|
|
"current_user" => { "id" => user.id },
|
|
"public_reaction_counts" => [{ "id" => article.comments.last.id, "count" => 1 }],
|
|
"reactions" => JSON.parse(user.reactions.where(reactable: comment).to_json),
|
|
)
|
|
end
|
|
|
|
it "does not set surrogate key headers" do
|
|
expect(response.headers["surrogate-key"]).to be_nil
|
|
end
|
|
|
|
it "does not set x-accel-expires headers" do
|
|
expect(response.headers["x-accel-expires"]).to be_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_path(commentable_id: article.id, commentable_type: "Article") }
|
|
|
|
it "returns the correct json response" do
|
|
result = response.parsed_body
|
|
|
|
expect(result["current_user"]).to eq("id" => nil)
|
|
expect(result["public_reaction_counts"]).to eq([{ "id" => article.comments.last.id, "count" => 1 }])
|
|
expect(result["reactions"]).to be_empty
|
|
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 the x-accel-expires header equal to max-age for article" do
|
|
expect(response.headers["X-Accel-Expires"]).to eq(max_age.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 rate limiting" do
|
|
let(:rate_limiter) { RateLimitChecker.new(user) }
|
|
|
|
before do
|
|
allow(RateLimitChecker).to receive(:new).and_return(rate_limiter)
|
|
sign_in user
|
|
end
|
|
|
|
it "increments rate limit for reaction_creation" do
|
|
allow(rate_limiter).to receive(:track_limit_by_action)
|
|
post "/reactions", params: article_params
|
|
|
|
expect(rate_limiter).to have_received(:track_limit_by_action).with(:reaction_creation)
|
|
end
|
|
|
|
it "returns a 429 status when rate limit is reached" do
|
|
allow(rate_limiter).to receive(:limit_by_action).and_return(true)
|
|
post "/reactions", params: article_params
|
|
|
|
expect(response.status).to eq(429)
|
|
end
|
|
end
|
|
|
|
context "when reacting to an article" do
|
|
before do
|
|
sign_in user
|
|
end
|
|
|
|
it "creates reaction" do
|
|
expect do
|
|
post "/reactions", params: article_params
|
|
end.to change(Reaction, :count).by(1)
|
|
end
|
|
|
|
it "destroys existing reaction" do
|
|
post "/reactions", params: article_params
|
|
expect do
|
|
# same route to destroy, so sending POST request again
|
|
post "/reactions", params: article_params
|
|
end.to change(Reaction, :count).by(-1)
|
|
end
|
|
end
|
|
|
|
context "when creating readinglist" do
|
|
before do
|
|
user.setting.update_column(:experience_level, 8)
|
|
sign_in user
|
|
post "/reactions", params: {
|
|
reactable_id: article.id,
|
|
reactable_type: "Article",
|
|
category: "readinglist"
|
|
}
|
|
end
|
|
|
|
it "creates reaction" do
|
|
expect(Reaction.last.reactable_id).to eq(article.id)
|
|
end
|
|
|
|
it "creates rating vote" do
|
|
expect(RatingVote.last.context).to eq("readinglist_reaction")
|
|
expect(RatingVote.last.rating).to be(8.0)
|
|
end
|
|
end
|
|
|
|
context "when creating thumbsup" do
|
|
before do
|
|
user.add_role(:trusted)
|
|
sign_in user
|
|
end
|
|
|
|
it "clears thumbsdown comments but not like" do
|
|
create(:reaction, reactable: article, user: user, points: 1, category: "like")
|
|
create(:reaction, reactable: article, user: user, points: 1, category: "thumbsdown")
|
|
post "/reactions", params: {
|
|
reactable_id: article.id,
|
|
reactable_type: "Article",
|
|
category: "thumbsup"
|
|
}
|
|
expect(Reaction.where(category: "thumbsup").size).to be 1
|
|
expect(Reaction.where(category: "thumbsdown").size).to be 0
|
|
expect(Reaction.where(category: "like").size).to be 1
|
|
end
|
|
end
|
|
|
|
context "when creating thumbsdown" do
|
|
before do
|
|
user.add_role(:trusted)
|
|
sign_in user
|
|
end
|
|
|
|
it "clears thumbsup comments but not vomit or like" do
|
|
create(:reaction, reactable: article, user: user, points: 1, category: "vomit")
|
|
create(:reaction, reactable: article, user: user, points: 1, category: "thumbsup")
|
|
create(:reaction, reactable: article, user: user, points: 1, category: "like")
|
|
post "/reactions", params: {
|
|
reactable_id: article.id,
|
|
reactable_type: "Article",
|
|
category: "thumbsdown"
|
|
}
|
|
expect(Reaction.where(category: "thumbsdown").size).to be 1
|
|
expect(Reaction.where(category: "thumbsup").size).to be 0
|
|
expect(Reaction.where(category: "like").size).to be 1
|
|
expect(Reaction.where(category: "vomit").size).to be 1
|
|
end
|
|
end
|
|
|
|
context "when vomiting on a user" do
|
|
before do
|
|
sign_in trusted_user
|
|
end
|
|
|
|
it "creates reaction" do
|
|
expect do
|
|
post "/reactions", params: user_params
|
|
end.to change(Reaction, :count).by(1)
|
|
end
|
|
|
|
it "destroys existing reaction" do
|
|
post "/reactions", params: user_params
|
|
expect do
|
|
post "/reactions", params: user_params
|
|
end.to change(Reaction, :count).by(-1)
|
|
end
|
|
end
|
|
|
|
context "when signed in as admin" do
|
|
let(:admin) { create(:user, :admin) }
|
|
|
|
before do
|
|
sign_in admin
|
|
end
|
|
|
|
it "automatically confirms vomits on users" do
|
|
post "/reactions", params: user_params
|
|
|
|
reaction = Reaction.find_by(reactable_id: user.id)
|
|
expect(reaction.category).to eq("vomit")
|
|
expect(reaction.status).to eq("confirmed")
|
|
end
|
|
|
|
it "automatically confirms vomits on articles" do
|
|
post "/reactions", params: article_params.merge(category: "vomit")
|
|
|
|
reaction = Reaction.find_by(reactable_id: article.id)
|
|
expect(reaction.category).to eq("vomit")
|
|
expect(reaction.status).to eq("confirmed")
|
|
end
|
|
|
|
it "does not automatically confirm positive reactions" do
|
|
post "/reactions", params: article_params
|
|
|
|
reaction = Reaction.find_by(reactable_id: article.id)
|
|
expect(reaction.category).to eq("like")
|
|
expect(reaction.status).to eq("valid")
|
|
end
|
|
end
|
|
|
|
context "when signed out" do
|
|
it "returns an unauthorized error" do
|
|
expect { post "/reactions", params: article_params }.to raise_error(Pundit::NotAuthorizedError)
|
|
end
|
|
end
|
|
end
|
|
end
|