mirror of
https://github.com/Growstuff/growstuff.git
synced 2025-12-24 10:07:47 -05:00
* Add comments to photos Extend the photo show page to support comments by logged in users. - Make the Comment model polymorphic. - Update the Photo and Post models to have comments. - Update the comments controller to handle the polymorphic association. - Update the photo show page to display comments and a comment form. - Create a reusable comments partial. * Add migration * Fix tests * Fix tests * Slightly fix tests * Fix variables * Add field * Refactor slightly * Refactor slightly * Refactor slightly * Refactor * Photos respond to this as well * Refactor to polymorphic_url * Rename * Wrong relationship * Refactor and fix tests * Fix relationships * Fix rendering * Fix tests * Fix model tests * Fix test * Fix test * Fix test * Fix test * Fix controller spec * Fix view tests --------- Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: Daniel O'Connor <daniel.oconnor@gmail.com>
130 lines
3.7 KiB
Ruby
130 lines
3.7 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
require 'rails_helper'
|
|
|
|
describe CommentsController do
|
|
subject { response }
|
|
|
|
let(:member) { FactoryBot.create(:member) }
|
|
|
|
before do
|
|
sign_in member
|
|
controller.stub(:current_member) { member }
|
|
end
|
|
|
|
def valid_attributes
|
|
@post = FactoryBot.create(:post)
|
|
{ post_id: @post.id, body: "some text" }
|
|
end
|
|
|
|
describe "GET RSS feed" do
|
|
let!(:first_comment) { FactoryBot.create(:comment, created_at: 10.days.ago) }
|
|
let!(:last_comment) { FactoryBot.create(:comment, created_at: 4.minutes.ago) }
|
|
|
|
describe "returns an RSS feed" do
|
|
before { get :index, format: "rss" }
|
|
|
|
it { is_expected.to be_successful }
|
|
it { is_expected.to render_template("comments/index") }
|
|
it { expect(response.content_type).to eq("application/rss+xml; charset=utf-8") }
|
|
it { expect(assigns(:comments)).to eq([last_comment, first_comment]) }
|
|
end
|
|
end
|
|
|
|
describe "GET new" do
|
|
let(:post) { FactoryBot.create(:post) }
|
|
|
|
describe "with valid params" do
|
|
before do
|
|
get :new, params: {
|
|
comment: { commentable_id: post.id, commentable_type: "Post" }
|
|
}
|
|
end
|
|
|
|
let(:old_comment) { FactoryBot.create(:comment, commentable: post) }
|
|
|
|
it "picks up post from params" do
|
|
expect(assigns(:commentable)).to eq(post)
|
|
end
|
|
|
|
it "assigns the old comments as @comments" do
|
|
expect(assigns(:comments)).to eq [old_comment]
|
|
end
|
|
end
|
|
|
|
it "dies if no post specified" do
|
|
get :new
|
|
expect(response).not_to be_successful
|
|
end
|
|
end
|
|
|
|
describe "GET edit" do
|
|
let(:post) { FactoryBot.create(:post) }
|
|
|
|
before { get :edit, params: { id: comment.to_param } }
|
|
|
|
describe "my comment" do
|
|
let!(:comment) { FactoryBot.create(:comment, author: member, commentable: post) }
|
|
let!(:old_comment) { FactoryBot.create(:comment, commentable: post, created_at: Time.zone.yesterday) }
|
|
|
|
it "assigns previous comments as @comments" do
|
|
expect(assigns(:comments)).to eq([comment, old_comment])
|
|
end
|
|
end
|
|
|
|
describe "not my comment" do
|
|
let(:comment) { FactoryBot.create(:comment, commentable: post) }
|
|
|
|
it { expect(response).not_to be_successful }
|
|
end
|
|
end
|
|
|
|
describe "PUT update" do
|
|
before { put :update, params: { id: comment.to_param, comment: valid_attributes } }
|
|
|
|
describe "my comment" do
|
|
let(:comment) { FactoryBot.create(:comment, author: member) }
|
|
|
|
it "redirects to the comment's post" do
|
|
expect(response).to redirect_to(comment.commentable)
|
|
end
|
|
end
|
|
|
|
describe "not my comment" do
|
|
let(:comment) { FactoryBot.create(:comment) }
|
|
|
|
it { expect(response).not_to be_successful }
|
|
end
|
|
|
|
describe "attempting to change post_id" do
|
|
let(:post) { FactoryBot.create(:post, subject: 'our post') }
|
|
let(:other_post) { FactoryBot.create(:post, subject: 'the other post') }
|
|
let(:valid_attributes) { { commentable_type: "Post", commentable_id: other_post.id, body: "kōrero" } }
|
|
let(:comment) { FactoryBot.create(:comment, author: member, commentable: post) }
|
|
|
|
it "does not change post_id" do
|
|
comment.reload
|
|
expect(comment.commentable_id).to eq(post.id)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "DELETE destroy" do
|
|
before { delete :destroy, params: { id: comment.to_param } }
|
|
|
|
describe "my comment" do
|
|
let(:comment) { FactoryBot.create(:comment, author: member) }
|
|
|
|
it "redirects to the post the comment was on" do
|
|
expect(response).to redirect_to(comment.commentable)
|
|
end
|
|
end
|
|
|
|
describe "not my comment" do
|
|
let(:comment) { FactoryBot.create(:comment) }
|
|
|
|
it { expect(response).not_to be_successful }
|
|
end
|
|
end
|
|
end
|