具有多个复杂私有方法的单个公共方法的TDD流程

Question

注意 ：问题“我应该测试私有方法还是仅测试公共方法？” 是我要问的很好的参考。

我的问题是：用单一的，防弹的，可靠的公共方法与复杂的私有方法建立起来的最实用的TDD 过程是什么？

我最好通过示例来学习，所以这里是：

---

第1章）测试范围

假设我有一个仅做一件事的红宝石课程，它给了我培根。

它可能看起来像这样：

class Servant
  def gimme_bacon
    # a bunch of complicated private methods go here
  end

  private

  # all of the private methods required to make the bacon
end  

现在我可以servant = Servant.new ; servant.gimme_bacon 。 太棒了，这就是我所关心的。 我想要的只是我的培根。

但是说我的仆人很烂。 那是因为他还没有任何私有方法，所以gimme_bacon只返回nil 。 好的，没问题，我是一名开发人员，我将为Servant类提供所有正确的私有方法，以便他最终可以gimme_bacon 。

在寻求可靠的仆人时，我想对他的所有方法进行TDD。 但是等等，我只关心他要去gimme_bacon 。 我真的不在乎他要采取的所有步骤，只要在一天结束时得到培根就可以了。 毕竟， gimme_bacon是唯一的公共方法。

因此，我这样编写测试：

RSpec.describe Servant do
  let(:servant) { Servant.new }

  it "should give me bacon when I tell it to!" do
    expect(servant.gimme_bacon).to_not be_nil
  end
end

尼斯。 我只测试了公共方法。 完美的100％测试覆盖率。 我继续进一步开发gimme_bacon功能，并完全有信心对其进行了测试。

---

第2章）编写摩尔专用方法

经过一些开发（不幸的是，不是TDD，因为我要添加私有方法），我可能会有类似的东西（用伪代码）：

class Servant
  attr_reader :bacon

  def initialize(whats_in_the_fridge)
    @bacon = whats_in_the_fridge[:bacon]
  end

  def gimme_bacon(specifications)
    write_down_specifications(specifications)
    google_awesome_recipes
    go_grocery_shopping if bacon.nil?
    cook_bacon
    serve
  end

  private

  attr_reader :specifications, :grocery_list

  def write_down_specifications(specifications)
    @specifications = specifications
  end

  def google_awesome_recipes
    specifications.each do |x|
      search_result = google_it(x)
      add_to_grocery_list if looks_yummy?(search_result)
    end
  end

  def google_it(item)
    HTTParty.get "http://google.com/#q=#{item}"
  end

  def looks_yummy?(search_result)
    search_result.match(/yummy/)
  end

  def add_to_grocery_list
    @grocery_list ||= []
    search_result.each do |tasty_item|
      @grocery_list << tasty_item
    end
  end

  def go_grocery_shopping
    grocery_list.each { |item| buy_item(item) }
  end

  def buy_item
    1_000_000 - item.cost
  end

  def cook_bacon
    puts "#{bacon} slices #{bacon_size_in_inches} inch thick on skillet"
    bacon.cooked = true
  end

  def bacon_size_in_inches
    case specifications
    when "chunky" then 2
    when "kinda chunky" then 1
    when "tiny" then 0.1
    else
      raise "wtf"
    end
  end

  def serve
    bacon + plate
  end

  def plate
    "---"
  end
end

---

结论：

事后看来，这是很多私有方法。

可能会有多个故障点，因为我没有真正尝试任何TDD。 上面是一个简单的示例，但是如果服务生不得不做出决定，例如根据我的要求去哪家杂货店，该怎么办？ 如果互联网不畅而他无法使用Google等，该怎么办？

是的，您可以说我应该创建一个子类，但是我不确定。 我想要的只是一个具有一种公共方法的类。

为了将来参考，我在TDD流程中可以做得更好吗？

Answer 1

我不确定您为什么会这样想，因为它们是私有方法，因此无法TDD。 它们是私有方法（或50个不同的类）的事实是测试培根仆人所需行为的实现细节。

为了在私有方法中完成所有工作，您的类必须具有

• 依赖
• 输入

否则，它将像第一个示例一样返回一些培根。

这些输入和相关性是在TDD上进行测试的关键，即使这些输入导致私有方法也是如此。 您仍然只能通过公共界面进行测试

因此，在第二个示例中，您有一些规范要通过gimme_bacon方法传递给您的类（红宝石不是我的事，因此请谅解任何误解）。 您的测试可能如下所示：

When I ask for chunky bacon I should get bacon that's 2" thick
When I ask for kinda chunky bacon I should get bacon that's 1" thick
When I ask for tiny bacon I should get bacon thats 0.1" thick
When I ask for an unsupported bacon chunkyness I should get an error telling me 'wtf'

您可以在添加测试定义您的培根提供者所需行为的测试时逐步实现此功能

当您必须接触Google东西时，便会与依赖项进行交互。 您的班级应允许切换这些依赖项（我相信这在ruby中很简单），因此您可以轻松测试在班级边界发生的情况。 因此，在您的示例中，您可能有一个食谱查找器。 您将其传递给班级，并在测试中将其给予

• 一个找到食谱
• 一无所获
• 错误之一
• 等等
• 等等

每次编写测试说明您的类的依赖关系以某种方式运行时，您期望类的行为如何。 然后，您将创建一个以这种方式运行的依赖项，并在您的类中实现所需的行为。

所有TDD，无论这些方法是否私有。

Answer 2

当一个类变得非常复杂时，可能是时候通过将片段委派给一些下级类来分解它了。 考虑一下单一责任原则。 主类负责协调培根过程，有一个类负责查找食谱，等等。每个下级类都可以通过公共方法进行TDD'ed，该方法包括其行为的所有不同变体。 对于主类，我将做一些集成测试以确保所有内容都正确地连接在一起。