Sollte ich private Methoden mit RSpec testen?

Question

Ist es eine gute Übung, Tests für private Methoden zu schreiben?

Betrachten Sie das folgende einfache Beispiel:

class Group 
    has_many :members 

    private 

    def release_members 
    members.each { |member| member.update_attributes group_id: nil } 
    end 
end 

Wäre es eine gute Übung einen Test für die release_members Methode in RSpec zu schreiben? Ich glaube, du müsstest den Test schreiben, indem du die Methode mit sendest anrufst. group.send(:release_members), die manchmal verpönt ist.

Answer 1

Eine ausführliche Diskussion dieses Themas finden Sie in diesen Folien aus einem Gespräch von Sandi Metz.

https://speakerdeck.com/skmetz/magic-tricks-of-testing-railsconf

Sie sagt, dass Sie Ihre private Methoden Testfahrt können, wenn Sie mögen, aber dass der einzige Test, der Sie sind diejenigen, die Prüfung der öffentlichen Schnittstelle sorgen sollte. Andernfalls könnten Sie zu eng mit der Implementierung verbunden sein.

Ich denke, der Punkt von toch, über das Austeilen von Service- und Wertobjekt und das Unterstellen von Tests ist auch ein gutes, wenn Sie nervös werden über komplexe private Methoden, die nicht getestet werden.

Answer 2

Sie sollten keine privaten Methoden testen, da sie zum internen Mechanismus der Klasse gehören. Der Zweck von Komponententests besteht darin, zu überprüfen, ob sich Ihre Klasse bei der Interaktion mit ihrer Schnittstelle, d. H. Ihren öffentlichen Methoden, wie erwartet verhält.

Wenn Sie an einem bestimmten Punkt nicht mit langen privaten Methoden vertraut sind, liegt es wahrscheinlich daran, dass Sie hier die Möglichkeit haben, diese Logik außerhalb der Klasse zu verwenden und ein anderes Modul oder eine andere Klasse zu erstellen. Dann können Sie es testen, wiederum nur seine Schnittstelle, d.h. seine öffentlichen Methoden.

In einigen seltenen Fällen ist es notwendig, die privaten Methoden zu testen, da die gesamte interne Logik sehr komplex ist und Sie das Problem teilen möchten. Aber in 99,9% ist das Testen privater Methoden eine schlechte Idee.