測試突變和存取方法時避免重複

Question

比方說，你有一個簡單的類，如：

class Box 
    def initialize 
    @widgets = [] 
    end 

    def add_widget(widget) 
    @widgets << widget 
    end 

    def widgets 
    @widgets 
    end 
end 

我會寫一個測試，看起來是這樣的：

describe Box do 
    describe "#initialize" do 
    it "creates an empty box" 
     Box.new.widgets.should == [] 
    end 
    end 

    describe "#add_widget" do 
    it "adds a widget to the box" 
     widget = stub('widget') 
     box = Box.new 
     box.add_widget(widget) 
     box.widgets.should == [widget] 
    end 
    end 

    describe "#widgets" do 
    it "returns the list of widgets" 
     widget = stub('widget') 
     box = Box.new 
     box.add_widget(widget) 
     box.widgets.should == [widget] 
    end 
    end 
end 

注意如何最後兩個測試最終是相同的。我正在努力避免這些重疊的情況。我在前兩種情況下隱式測試#widgets，但我覺得應該有一個明確的測試。但是，代碼與第二種情況相同。

如果一個類有3個公共方法，那麼我會期望至少有一個測試對應每個這些方法。我錯了嗎？

UPDATE

我通過羅恩傑弗里斯其建議不要測試簡單的存取發現this article。

Answer 1

我不確定你，至少在這種情況下，你可以避免代碼重複。在沒有構造函數的情況下，您必須使用一種方法來測試另一種方法。一個辦法可以改變你的測試，在這種情況下可能是以下幾點：

describe Box do 
    describe "#initialize" do 
    it "creates an empty box" 
     Box.new.widgets.should == [] 
    end 
    end 

    describe "#add_widget" do 
    before do 
     @widget = stub('widget') 
     @box = Box.new 
     @box.add_widget(@widget) 
    end 

    it "adds a widget to the box, which gets returned by #widgets" 
     @box.widgets.should == [@widget] 
    end 
    end 
end 

Answer 2

這是非常簡單的例子，當你可能說你不應該這樣的存取。但是，如果情況稍微複雜一點，或者訪問者不是真正的訪問者，但它內部有一些邏輯，並且您確實想要測試它，那麼您可以使用instance_variable_get和instance_variable_set的Object。

describe Box do 
    describe "#initialize" do 
    it "creates an empty box" do 
     Box.new.widgets.should == [] 
    end 
    end 

    describe "#add_widget" do 
    it "adds a widget to the box" do 
     widget = stub('widget') 
     box = Box.new 
     box.add_widget(widget) 
     box.instance_variable_get(:@widgets).should == [widget] 
    end 
    end 

    describe "#widgets" do 
    it "returns the list of widgets" do 
     widget = stub('widget') 
     box = Box.new 
     box.instance_variable_set(:@widgets, [widget]) 
     box.widgets.should == [widget] 
    end 
    end 
end 

但是，我想這不是很好的測試技術，因爲你是一個對象的內部狀態，所以每當內部實現的改變，你必須確保測試正在改變，即使干擾的公共接口班級沒有變化。

Answer 3

也許這是測試正在發聲嗎？如果是這樣，而且我在聽，我會聽到這樣的話：「夥計，停止測試方法，這是重要的行爲。」。對此我想有以下回應：

describe Box do 
    it "contains widgets" do 
    widget = stub('widget') 
    box = Box.new 

    box.add_widget(widget) 
    box.widgets.should == [widget] 
    end 

    it "has no widgets by default" do 
    Box.new.widgets.should == [] 
    end 
end 

由於#widgets是唯一的（公衆）的方式來訪問箱部件和#add_widget是增加他們的唯一辦法，沒有其他的方式來測試Box行爲，但要同時鍛鍊它們。