应该失败的 RSPEC 测试通过答案

【问题标题】：RSPEC Test Passing that should fail应该失败的 RSPEC 测试通过
【发布时间】：2011-09-13 20:02:17
【问题描述】：

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

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

    require 'spec_helper'

    describe User do
     before(:each) do
       @attr = { :email => "testuser@example.com", :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，它会变成红色。提前致谢！

【问题讨论】：

听起来您的应用程序的行为完全符合您的预期。如果您的测试是 long_email.should_not be_valid，并且您的测试通过了，那似乎就是您想要的。如果没有，您能否更详细地解释您的问题？
您的电子邮件不是太长，它是 60 个字符，并且您接受

标签： ruby-on-rails rspec

【解决方案1】：

正如@apneadiving 所指出的，测试的格式很重要。它不仅使阅读更容易，而且还显示了你没有测试过的地方：

require 'spec_helper'

describe User do
  let(:attr){ 
    {:email => "testuser@example.com", :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）
向您展示了您只指定了一个失败案例，而没有针对阳性案例（或其他案例）进行测试

如果你写了肯定的情况并且也通过了，那么你真的知道有问题了！

【讨论】：