Блог

Я следил за книгой Хартла в rubystutorial.org и застрял на этой ошибке

Когда я запускаю тесты rspec, он сообщает мне, что метод (save) не существует для пользователя

Вот мой код и тест. Я не могу найти, что с этим не так и почему метод не существует в @user

user_spec.rb

 require 'spec_helper'

describe User do

  before do
    @user = User.new(name: "Example User", email: "user@example.com") 
  end

  subject { @user }

  it { should respond_to(:name) }
  it { should respond_to(:email) }

  it { should be_valid }

  describe "when name is not present" do
    before { @user.name = " " }
    it { should_not be_valid }
  end

  describe "when email is not present" do
    before { @user.email = " " }
    it { should_not be_valid }
  end

  describe "when name is too long" do
    before { @user.name = "a" * 51 }
    it { should_not be_valid } 
  end

  describe "when email format is invalid" do
    it "shoud be invalid" do 
      addresses = %w[user@foo,com user_at_foo.org example.user@foo. foo@bar_baz.com foo@bar baz.com]
      addresses.each do |invalid_address|
        @user.email = invalid_address
        expect(@user).not_to be_valid
      end
    end
  end

  describe "when email format is valid" do 
    it "should be valid" do 
      addresses = %w[user@foo.COM A_US-ER@f.b.org frst.lst@foo.jp a b@baz.cn]
      addresses.each do |valid_address|
        @user.email = valid_address
        expect(@user).to be_valid
      end
    end
  end

  describe "when email address is already taken" do 
    before do
      user_with_same_email = @user.dup
      user_with_same_email = @user.email.upcase
      user_with_same_email.save
    end

    it { should_not be_valid }
  end

end
  

И мой user.rb

 class User < ActiveRecord::Base
    before_save { self.email = email.downcase }
    validates(:name, presence: true, length: { maximum: 50 } )
    VALID_EMAIL_REGEX = /A[w -.] @[a-zd-.] .[a-z] z/i
    validates(:email, presence: true, format: { with: VALID_EMAIL_REGEX },
                      uniqueness: { case_sensitive: false } )
end
  

Точная ошибка

 Failures:

1) User when email address is already taken 

 Failure/Error: user_with_same_email.save
 NoMethodError:
   undefined method `save' for "USER@EXAMPLE.COM":String
 # ./spec/models/user_spec.rb:55:in `block (3 levels) in <top (required)>'

Finished in 0.411 seconds
24 examples, 1 failure

Failed examples:

rspec ./spec/models/user_spec.rb:58 # User when email address is already take
  

Вы определяете пользователя, а затем изменяете его в строке (email, поэтому у email нет метода сохранения) в следующих строках:

 user_with_same_email = @user.dup
user_with_same_email = @user.email.upcase
user_with_same_email.save
  

Вы должны изменить его для этого:

 user_with_same_email = @user.dup
user_with_same_email.email = @user.email.upcase
user_with_same_email.save
  

Объект

Проблему здесь можно увидеть в самой ошибке:

неопределенный метод «сохранить» для «USER@EXAMPLE.COM «:Строка

Проблема не в том, что .save он недоступен, а в том, что объект, к которому вы пытаетесь его вызвать, не может его обработать.

Проблема конкретно связана с этой строкой:

user_with_same_email = @user.email.upcase

—

Исправить

Проблема в том, что вы устанавливаете user_with_same_email переменную как @user.email.upcase . Вы можете сделать то, что zishe предложено (установить email атрибут новой переменной)

Я не совсем понимаю, чего вы пытаетесь достичь — поскольку ваша User модель устанавливает адрес электронной почты для downcase before_save обратного вызова.