如何在Ruby中删除整个数组并使用RSpec进行测试

时间:2015-06-19 23:04:20

标签: ruby-on-rails ruby rspec

我对Ruby很新,目前正在进行全栈课程。对于我的一个项目,我们正在构建一个地址簿。我已经设置了如何在地址簿中添加一个条目,但是,我似乎无法弄清楚如何删除一个条目(我尝试使用下面的AddressBook类中的remove_entry方法但是没有任何运气)。我们也应该首先使用RSpec进行测试,测试失败,然后编写一些代码以使其通过。如果我没有包含此问题所需的所有信息,请告诉我(菜鸟在这里)。无论如何,这是我到目前为止:

RSpec的

context ".remove_entry" do
    it "removes only one entry from the address book" do
      book = AddressBook.new
      entry = book.add_entry('Ada Lovelace', '010.012.1815', 'augusta.king@lovelace.com')
      book.remove_entry(entry)

      expect(entry).to eq nil
    end
  end

AddressBook类

require_relative "entry.rb"

class AddressBook
  attr_accessor :entries

  def initialize
    @entries = []
  end

  def add_entry(name, phone, email)
    index = 0
    @entries.each do |entry|
      if name < entry.name
        break
      end
      index += 1
    end

    @entries.insert(index, Entry.new(name, phone, email))
  end

  def remove_entry(entry)
    @entries.delete(entry)
  end
end

入门级

class Entry
  attr_accessor :name, :phone_number, :email

  def initialize(name, phone_number, email)
    @name = name
    @phone_number = phone_number
    @email = email
  end

  def to_s
    "Name: #{@name}\nPhone Number: #{@phone_number}\nEmail: #{@email}"
  end
end

使用RSpec测试我的代码时,收到以下错误消息:

.....F

Failures:

  1) AddressBook.remove_entry removes only one entry from the address book
     Failure/Error: expect(entry).to eq nil

       expected: nil
            got: [#<Entry:0x00000101bc82f0 @name="Ada Lovelace", @phone_number="010.012.1815", @email="augusta.king@lovelace.com">]

       (compared using ==)
     # ./spec/address_book_spec.rb:49:in `block (3 levels) in <top (required)>'

Finished in 0.02075 seconds (files took 0.14221 seconds to load)
6 examples, 1 failure

Failed examples:

rspec ./spec/address_book_spec.rb:44 # AddressBook.remove_entry removes only one entry from the address book

2 个答案:

答案 0 :(得分:1)

我认为你的期望有问题。 entry变量未设置为nil,但book内的条目将为nil

我认为这样的事情会更好:

expect(book.entries.find { |e| e.name == "Ada Lovelace" }).to eq nil

更好的是,您的AddressBook可以拥有自己的find方法,这样可以使expect参数更好,例如book.find(:name => "Ada Lovelace")

最后,我还会在 remove_entry来电之前预约电话,以确保其结果等于entry

答案 1 :(得分:1)

只测试book.entries关联为空:

expect(book.entries).to be_empty

由于book是测试中的局部变量,如果保持测试的原子性,则不会得到错误的否定结果。关于rspec的一些best practices

修改: 您还可以检查条目中没有的条目:

expect(book.entries.index(entry)).to be_nil

或用以下方法测试数组长度的变化:

expect { book.remove_entry(entry) }.to change{book.entries.count}.by(-1)

如果您想知道be_xxx语法糖,如果对象回复xxx?,那么您可以在测试中使用be_xxxpredicate matchers