2011-12-08 15 views
1

さて、私はその場所全体を検索しましたが、誰もこの '暗号化'方法で問題が発生していると思われますたくさんの他のものは、さらに別れなしに第7章でいくつかの困難があった。ここRoRチュートリアル、第7章 - rspecテストに失敗しました:NoMethodError 'encrypt'

は、ユーザ・モデル・ファイル内のHARTLのChapter 7

私のコードへのリンク、および対応するspecファイルだ、完全に正確なものを彼にのように見えます私はまだテストをパスすることはできません。エラー?

Failures: 
1) User should create a new instance given valid attributes 
Failure/Error: User.Create!(@attr) 
NoMethodError: undefined method 'encrypt' for #<User:asdf> 
#./app/models/user.rb:22:in 'has_password?' 
#./app/models/user.rb:28:in 'encrypt_password' 
#./spec/models/user_spec.rb:15:in 'block (2 levels) in <top (required)>' 

2) User should not allow duplicate email addresses 
Failure/Error: User.Create!(@attr) 
NoMethodError: undefined method 'encrypt' for #<User:asdf> 
#./app/models/user.rb:22:in 'has_password?' 
#./app/models/user.rb:28:in 'encrypt_password' 
#./spec/models/user_spec.rb:15:in 'block (2 levels) in <top (required)>' 

3) User should reject email addresses identical up to case 
Failure/Error: User.Create!(@attr) 
NoMethodError: undefined method 'encrypt' for #<User:asdf> 
#./app/models/user.rb:22:in 'has_password?' 
#./app/models/user.rb:28:in 'encrypt_password' 
#./spec/models/user_spec.rb:15:in 'block (2 levels) in <top (required)>' 

... 

7) User has_password? method should be false if passwords do not match 
Failure/Error: User.Create!(@attr) 
NoMethodError: undefined method 'encrypt' for #<User:asdf> 
#./app/models/user.rb:22:in 'has_password?' 
#./app/models/user.rb:28:in 'encrypt_password' 
#./spec/models/user_spec.rb:15:in 'block (3 levels) in <top (required)>' 

このように、各テストで同じエラーメッセージが表示されています。その理由を調べようとしています。

はここに私のuser.rbです:

require 'digest' 
class User < ActiveRecord::Base 
    attr_accessor :password 
    attr_accessible :name, :email, :password, :password_confirmation 

    email_regex = /\A[\w+\-.][email protected][a-z\d\-.]+\.[a-z]+\z/i 

    validates :name, :presence => true, 
        :length => { :maximum => 50 } 
    validates :email, :presence => true, 
        :format => { :with => email_regex }, 
        :uniqueness => { :case_sensitive => false } 
#automatically create the virtual attribute for 'password_confirmation' 
    validates :password, :presence => true, 
         :confirmation => true, 
         :length => { :within => 6..40 } 

    before_save :encrypt_password 

    #returns true if the users password matches the submitted one 
    def has_password?(submitted_password) 
    encrypted_password == encrypt(submitted_password) 
    end 

    private 

    def encrypt_password 
    self.salt = make_salt unless has_password?(password) 
    self.encrypted_password = encrypt(password) 
    end 

    def encrypt_string 
    secure_hash("#{salt}--#{string}") 
    end 

    def make_salt 
    secure_hash("#{Time.now.utc}--#{password}") 
    end 

    def secure_hash(string) 
    Digest::SHA2.hexdigest(string) 
    end 
end 

と私のuser_spec.rbファイル:

require 'spec_helper' 
require 'digest' 

describe User do 
    before(:each) do 
@attr = { 
    :name => "User Name", 
    :email => "[email protected]", 
    :password => "password", 
    :password_confirmation => "password" 
} 
    end 

    it "should create a new instance given valid attributes" do 
User.create!(@attr) 
    end 

    it "should require a name" do 
no_name_user = User.new(@attr.merge(:name => "")) 
no_name_user.should_not be_valid 
    end 

    it "should require an email" do 
no_email_user = User.new(@attr.merge(:email => "")) 
no_email_user.should_not be_valid 
    end 

    it "should reject names that are too long" do 
long_name = "a" * 51 
long_name_user = User.new(@attr.merge(:name => long_name)) 
long_name_user.should_not be_valid 
    end 

    it "should accept valid email addresses" do 
addresses = %w[[email protected] [email protected] [email protected]] 
addresses.each do |address| 
    valid_email_user = User.new(@attr.merge(:email => address)) 
    valid_email_user.should be_valid 
end 
    end 

    it "should reject invalid email addresses" do 
addresses = %w[[email protected],com user_at_foo.org [email protected]] 
    addresses.each do |address| 
    invalid_email_user = User.new(@attr.merge(:email => address)) 
    invalid_email_user.should_not be_valid 
    end 
    end 

    it "should not allow duplicate email addresses" do 
User.create!(@attr) 
user_with_duplicate_email = User.new(@attr) 
user_with_duplicate_email.should_not be_valid 
    end 

    it "should reject email addresses identical up to case" do 
upcased_email = @attr[:email].upcase 
User.create!(@attr.merge(:email => upcased_email)) 
user_with_duplicate_email = User.new(@attr) 
user_with_duplicate_email.should_not be_valid 
    end 

    describe "password validations" do 
it "should require a password" do 
    User.new(@attr.merge(:password => "", :password_confirmation => "")) 
    should_not be_valid 
end 

it "should require password to match the password confirmation" do 
    User.new(@attr.merge(:password_confirmation => "invalid")) 
    should_not be_valid 
end 

it "should reject short passwords" do 
    short = "a" * 5 
    hash = @attr.merge(:password => short, :password_confirmation => short) 
    User.new(hash).should_not be_valid 
end 

it "should reject long passwords" do 
    long = "a" * 41 
    hash = @attr.merge(:password => long, :password_confirmation => long) 
    User.new(hash).should_not be_valid 
    end 
    end 

    describe "password encryption" do 

before(:each) do 
    @user = User.create!(@attr) 
end 

it "should have an encrypted password attribute" do 
    @user.should respond_to(:encrypted_password) 
end 

it "should not allow a blank encrypted password" do 
    @user.encrypted_password.should_not be_blank 
    end 
    end 

    describe "has_password? method" do 

    before(:each) do 
    @attr = User.create!(@attr) 
    end 

    it "should be true if the passwords match" do 
    @user.has_password?(@attr[:password]).should be_true 
    end 

    it "should be false if the passwords don't match" do 
     @user.has_password?("invalid").should be_false 
     end 
    end 
end 

任意の助けいただければ幸いです。私は他の問題、私のコードをぶち壊し、さまざまな面を変えてテストを動作させてみました。私はそれが私がまだ見ていない本当に馬鹿なものではないことを願っています。

答えて

3

あなたのエラーはここにある:

def encrypt_string 
    secure_hash("#{salt}--#{string}") 
    end 

あなたは、次のencrypt_password方法でencryptを呼び出しているが、上記のあなたの方法をencrypt_string命名されています

def encrypt_password 
    self.salt = make_salt unless has_password?(password) 
    self.encrypted_password = encrypt(password) 
    end 

だけの方法でencryptencrypt_stringを変更あなたは良いことをすべきです。

+0

ありがとうございます。 – user1086832

+1

お世話になりました。これがあなたの質問に答えるなら、私の答えの横にあるチェックマークをチェックして質問を終了してください – iwasrobbed

関連する問題