質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.03%

rails tutorial 12章 テンプレートエラー に苦しめられております。

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 177

kntarohh

score 5

前提・実現したいこと

初質問です。言葉足らずなところがあるかも知れませんがご協力いただけますと幸いです。
現在rails tutorialの12章に取り組んでおります。

パスワードを再設定する為のメールを送るべく、パスワードリセットのリソース作成、Userモデルにパスワード再設定用のトークンとそれに対応するダイジェストを追加し、再設定メールのテンプレートを作成などをしました。

順調に進んでるかと思いきや、以下のエラーメッセージが解消できずに半日戦っております。
前章ではエラーが出なかったので恐らく本章でやらかしたかと思われます。

発生している問題・エラーメッセージ

Error:
UserMailerTest#test_password_reset:
ActionView::Template::Error: No route matches {:action=>"edit", :controller=>"account_activations", :email=>"michael@example.com", :id=>nil}, possible unmatched constraints: [:id]
    app/views/layouts/mailer.html.erb:9:in `_app_views_layouts_mailer_html_erb___4506527162881960628_72659080'
    app/mailers/user_mailer.rb:13:in `password_reset'
    test/mailers/user_mailer_test.rb:21:in `block in <class:UserMailerTest>'

実施したテスト("password_reset"のテストを追加したところからエラーが出始めました。コメントアウトするとパスします。)

require 'test_helper'

class UserMailerTest < ActionMailer::TestCase

  test "account_activation" do
    user = users(:michael)
    user.activation_token = User.new_token
    mail = UserMailer.account_activation(user)
    assert_equal "Account activation", mail.subject
    assert_equal ["michael@example.com"], mail.to
    assert_equal ["noreply@example.com"], mail.from
    assert_match user.name,               mail.body.encoded
    assert_match user.activation_token,   mail.body.encoded
    assert_match CGI.escape(user.email),  mail.body.encoded
  end

  test "password_reset" do
    user = users(:michael)
    user.reset_token = User.new_token
    mail = UserMailer.password_reset(user)
    assert_equal "Password reset", mail.subject
    assert_equal [user.email], mail.to
    assert_equal ["noreply@example.com"], mail.from
    assert_match user.reset_token,        mail.body.encoded
    assert_match CGI.escape(user.email),  mail.body.encoded
  end
end

★app/views/layouts/mailer.html.erb

<h1>Sample App</h1>

<p>Hi <%= @user.name %>,</p>

<p>
Welcome to the Sample App! Click on the link below to activate your account:
</p>

<%= link_to "Activate", edit_account_activation_url(@user.activation_token,
                                                    email: @user.email) %>

★app/mailers/user_mailer.rb

class UserMailer < ApplicationMailer
  def account_activation(user)
    @user = user
    mail to: user.email
  end

  def password_reset(user)
    @user = user
    mail to: user.email, subject: "Password reset"
  end
end

★app/models/user.rb

class User < ApplicationRecord
  attr_accessor :remember_token, :activation_token, :reset_token
  before_save   :downcase_email
  before_create :create_activation_digest

  validates :name,  presence: true, length: { maximum: 50 }
  VALID_EMAIL_REGEX = /\A[\w+\-.]+@[a-z\d\-]+(\.[a-z\d\-]+)*\.[a-z]+\z/i
  validates :email, presence: true, length: { maximum: 255 },
                    format: { with: VALID_EMAIL_REGEX },
                    uniqueness: { case_sensitive: false }
  has_secure_password
  validates :password, presence: true, length: { minimum: 6 }, allow_nil: true

  # 渡された文字列のハッシュ値を返す
  def self.digest(string)
    cost = ActiveModel::SecurePassword.min_cost ? BCrypt::Engine::MIN_COST :
                                                  BCrypt::Engine.cost
    BCrypt::Password.create(string, cost: cost)
  end

  # ランダムなトークンを返す
  def self.new_token
    SecureRandom.urlsafe_base64
  end

  <略>

   # 渡されたトークンがダイジェストと一致したらtrueを返す
  def authenticated?(attribute, token)
    digest = send("#{attribute}_digest")
    return false if digest.nil?
    BCrypt::Password.new(digest).is_password?(token)
  end

  # アカウントを有効にする
  def activate
    update_columns(activated: true, activated_at: Time.zone.now)
  end

  # 有効化用のメールを送信する
  def send_activation_email
    UserMailer.account_activation(self).deliver_now
  end

  # パスワード再設定の属性を設定する
  def create_reset_digest
    self.reset_token = User.new_token
    update_attribute(:reset_digest,  User.digest(reset_token))
    update_attribute(:reset_sent_at, Time.zone.now)
  end

  # パスワード再設定のメールを送信する
  def send_password_reset_email
    UserMailer.password_reset(self).deliver_now
  end

  private

    # メールアドレスをすべて小文字にする
    def downcase_email
      email.downcase!
    end

    # 有効化トークンとダイジェストを作成および代入する
    def create_activation_digest
      self.activation_token  = User.new_token
      self.activation_digest = User.digest(self.activation_token)
    end
end

★config/routes.rb

Rails.application.routes.draw do

  get 'password_resets/new'

  get 'password_resets/edit'

  get 'sessions/new'

  root 'static_pages#home'
  get  '/help',    to: 'static_pages#help'
  get  '/about',   to: 'static_pages#about'
  get  '/contact', to: 'static_pages#contact'
  get  '/signup',  to: 'users#new'
  post '/signup',  to: 'users#create'
  get    '/login',   to: 'sessions#new'
  post   '/login',   to: 'sessions#create'
  delete '/logout',  to: 'sessions#destroy'
  resources :users
  resources :account_activations, only: [:edit]
  resources :password_resets,     only: [:new, :create, :edit, :update]
end

試したこと

activation_token が nil であることが原因であると思うのですが、"account_activation"のテストはパスするので、user.rb の create_activation_digest が実行されていないとしたら原因がわかりません。。

また、メールのプレビューを見ようとブラウザで http://<ホスト名>/rails/mailers/user_mailer/password_reset にアクセスすると本件と同様のエラーが確認できました。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • winterboum

    2019/07/21 05:18

    routees.rbも載せて下さい

    キャンセル

  • winterboum

    2019/07/21 20:40

    app/mailers/user_mailer.rb が有るかと思ったら、これ中身違いますね、viewのでは。
    app/mailers/user_mailer.rbを。

    キャンセル

回答 1

checkベストアンサー

0

1) resources :account_activations, only: [:edit]
に対応して
2) edit_account_activation_url(@user.activation_token,         email: @user.email)
がつかわれています。
が、
1)の様に定義した場合定義されるurlは /account_activations/:id/edit と :id というパラメータを必要とするものです。
しかし2)にはidが定義されていません。
AccountActibationsController#edit が提示されていないんで予想ですが、Userから該当するレコードを取り出すのではないでしょうか。
すると
edit_account_activation_url(@user)としてください。
email、activation_token は #editの方で、user=User.find(parms[:id])してそのuserから採ります。
ただ、その場合 user.reset_token = User.new_token のあとsaveが必要です。実際の運用では、tokenをsaveしておく必要が有るはずですからそれで良いかと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/07/22 21:44

    ご回答いただき誠に有難うございます。
    app/views/user_mailer/account_acctivation.html.erb に書くべき内容を
    app/views/layouts/mailer.html.erb に書いてしまっていたようです。
    上記ファイルの修正で無事テストはパスしましたが、チュートリアル完走後、
    ご回答いただいた内容につきましてもトライしてみようと思います。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.03%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる