Merge pull request #4678 from Growstuff/fix-photo-reindex-error-4629481995101950332

Fix NoMethodError: undefined method 'reindex' for class Photo in specs
This commit is contained in:
Daniel O'Connor
2026-06-24 01:45:40 +09:30
committed by GitHub
4 changed files with 3 additions and 9 deletions

View File

@@ -11,7 +11,7 @@ class LikesController < ApplicationController
likeable_id: params[:id]
)
if @like.likeable && @like.save
@like.likeable.reindex(refresh: true)
@like.likeable.reindex(refresh: true) if @like.likeable.respond_to?(:reindex)
success(@like, liked_by_member: true, status_code: :created)
else
failed(@like, message: t('messages.unable_to_like'))
@@ -26,7 +26,7 @@ class LikesController < ApplicationController
)
if @like&.destroy
@like.likeable.reindex(refresh: true)
@like.likeable.reindex(refresh: true) if @like.likeable.respond_to?(:reindex)
success(@like, liked_by_member: false, status_code: :ok)
else
failed(@like, message: t('messages.unable_to_unlike'))

View File

@@ -28,7 +28,6 @@ describe "home page", :search do
Planting.reindex
Seed.reindex
Harvest.reindex
Photo.reindex
visit root_path
end

View File

@@ -6,14 +6,10 @@ describe 'Likeable', :js, :search do
let(:another_member) { create(:london_member) }
let!(:post) { create(:post, :reindex, author: member) }
let!(:activity) { create(:activity, :reindex, owner: member) }
let!(:photo) { create(:photo, :reindex, owner: member) }
let!(:photo) { create(:photo, owner: member) }
let!(:harvest) { create(:harvest, :reindex, owner: member) }
let!(:planting) { create(:planting, :reindex, owner: member) }
before do
Photo.reindex
end
include_context 'signed in member'
shared_examples 'object can be liked' do

View File

@@ -47,7 +47,6 @@ RSpec.configure do |config|
# reindex models
Crop.reindex
Harvest.reindex
Photo.reindex
Planting.reindex
Seed.reindex
end