本应失败的 RSPEC 测试已通过

Question

我使用 rspec、rails、guard 和 sorcery 进行身份验证和测试。

我有一个测试正在测试电子邮件的长度。我想拒绝太长的电子邮件。这是我为 spec/models/user_spec.rb 编写的测试

    require 'spec_helper'

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


     it "should reject emails that are too long" do
       long_email = "a" * 101 + "gmail.com"
       long_email = User.new (@attr.merge(:email => long_email))
       long_email.should_not be_valid
     end

这是我已有的模型验证：

class User < ActiveRecord::Base
  authenticates_with_sorcery!

  attr_accessible :email, :password, :password_confirmation

  validates_presence_of :password, :on => :create
  validates :password, :confirmation => true,
                       :length       => { :within => 6..100 }

  email_regex = /\A[\w+\-.]+@[a-z\d\-.]+\.[a-z]+\z/i
  validates :email, :presence        => true,
                    :format          => { :with => email_regex },
                    :uniqueness      => {:case_sensitive => false},
                    :length          => { :within => 5..100 }
end

我是这方面的菜鸟，所以任何帮助将不胜感激。测试现在是绿色的，如果我将行更改为 long_email.should be_valid，它会变成红色。提前致谢！

Answer 1

正如@apneadiving 指出的，测试的格式很重要。它不仅使其更易于阅读，而且还显示您尚未测试的地方：

require 'spec_helper'

describe User do
  let(:attr){ 
    {:email => "[email protected]", :password => "password", :password_confirmation => "password" }
  }

  context "When given an email address" do
    context "That is too long" do
      let(:long_email){ "a" * 101 + "gmail.com" }
      subject{ User.new attr.merge(:email => long_email) }

      specify{ subject.should_not be_valid }
    end
    context "That is too short" do
      pending
    end
    context "That is between 5 and 100 characters long"
      pending
    end
  end

end

如您所见，重写它做了几件事：

• 使其更易于阅读，
• 更容易添加更多测试，而不会影响早期的测试那些（通过使用 let 而不是 before）
• 表明您只指定了失败的情况，并且没有测试积极的情况（或其他情况）

如果您写积极的情况也过去了，那么你就真的知道出了问题！