railsのupdateでバリデーションが効かない

Question

前提・実現したいこと

rails 6.0.0 ruby 2.6.5
モデルで設定しているpresence: trueがupdateで効かない。

該当のソースコード

class Post < ApplicationRecord
  has_one_attached :image

  with_options presence: true do
    validates :text
    validates :image
  end
end

class PostsController < ApplicationController
  before_action :set_post, only: [:show, :destroy, :edit, :update]

  ~~~省略~~~

  def edit
  end

  def update
    if @post.valid?
      @post.update(post_params)
      flash[:notice] = "編集が完了しました"
      redirect_to post_path(@post.id)
    else
      flash.now[:alert] = "編集に失敗しました"
      render :edit
    end
  end

  private

  def post_params
    params.require(:post).permit(:text, :image)
  end

  def set_post
    @post = Post.find(params[:id])
  end
end

試したこと

createアクションのsaveメソッドでも効いていない可能性があったので確認したが、空では登録できなくなっていてバリデーションが効いていた。

updateアクションでbinding.pryを実行して、paramsの中身を見たが空になっているのに@post.valid? => true
になっていた。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

@postには既存のpostが入っているわけで
validにならない方が問題かと思います

修正後の内容で正しいかと思いますが、
修正の結果バリデーションが効くようになったわけでなく
invalidのケースでも

      flash[:notice] = "編集が完了しました"
      redirect_to post_path(@post.id)

の成功処理が行われていたことが問題です

Answer 2

if @post.valid?
      @post.update(post_params)

このvalid時点で@postには値が入っていませんか？
順番を変更すれば、バリデーションが効くような気がします。
よければお試しください。

@post.update(post_params)
if @post.valid?

Answer 3

updateアクションの、@post.valid?の部分を削除して、以下のように編集したら上手くバリデーションが効くようになりました。

def update
    if @post.update(post_params)
      flash[:notice] = "編集が完了しました"
      redirect_to post_path(@post.id)
    else
      flash.now[:alert] = "編集に失敗しました"
      render :edit
    end
  end