63

I'm trying out test driven development in a toy project. I can get the tests working for the public interface to my classes (although I'm still on the fence because I'm writing more testing code than there is in the methods being tested).

I tend to use a lot of private methods becuase I like to keep the public interfaces clean; however, I'd still like to use tests on these methods.

Since Cocoa is a dynamic language, I can still call these private methods, but i get warnings in my tests that my class may not respond to these methods (although it clearly does). Since I like to compile with no warnings here are my questions:

  1. How do i turn off these warnings in Xcode?
  2. Is there something else I could do to turn off these warnings?
  3. Am I doing something wrong in trying 'white box' testing?
3
  • So far I have not exposed my private methods in the header file. Why should I, I find it more elegant to have only public methods in the header. However it seems I have to change this in objective-c.
    – Nils
    Commented Jan 10, 2012 at 13:01
  • 1
    @Nils You don't have private methods in a header file. Once there, they are public methods.
    – Abizern
    Commented Jan 10, 2012 at 13:35
  • Yes @Abizern so far I followed the advice of Peter Hosey below.
    – Nils
    Commented Jan 10, 2012 at 13:43

7 Answers 7

126

Remember that there's actually no such thing as "private methods" in Objective-C, and it's not just because it's a dynamic language. By design, Objective-C has visibility modifiers for ivars, but not for methods — it's not by accident that you can call any method you like.

@Peter's suggestion is a great one. To complement his answer, an alternative I've used (when I don't want/need a header just for private methods) is to declare a category in the unit test file itself. (I use @interface MyClass (Test) as the name.) This is a great way to add methods that would be unnecessary bloat in the release code, such as for accessing ivars that the class under test has access to. (This is obviously less of an issue when properties are used.)

I've found this approach makes it easy to expose and verify internal state, as well as adding test-only methods. For example, in this unit test file, I wrote an -isValid method for verifying correctness of a binary heap. In production, this method would be a waste of space, since I assume a heap is valid — I only care about it when testing for unit test regressions if I modify the code.

4
  • 4
    Great suggestion! I like this idea better, keeps the exposure of the methods in context.
    – Bach
    Commented Feb 3, 2011 at 23:54
  • 1
    Wow, Peter Hosey should fix his answer to refer to yours. Great work, trying this now +1 Commented Oct 15, 2012 at 0:10
  • +1 for a helpful post, however I would like to point out that the unit test file you linked to is a broken link.
    – Stunner
    Commented Jan 9, 2014 at 22:08
  • A quick google search links to his git repo, Commented Jan 16, 2014 at 15:26
90

How do i turn off these warnings in Xcode?

Don't.

Is there something else I could do to turn off these warnings?

Don't.

Am I doing something wrong in trying 'white box' testing?

No.

The solution is to move your private methods to a category in its own header. Import this header into both the real class and test-case class implementation files.

5
  • 4
    +1 — Great suggestion! For a little further detail, check out this question/answer: stackoverflow.com/questions/1020070/#1020330 The private header containing the category with private methods can be named MyClass_Private.h, for example. Commented Jul 8, 2009 at 16:36
  • 2
    Thanks for this answer. I'm still not sure about it, but I must admit that there is a 'fun' element of writing a test and then making it pass.
    – Abizern
    Commented Jul 9, 2009 at 10:58
  • I've declared these private methods in an extension header file and for some reason, in XCode5 the compiler complains when calling these private methods declared in the extension header. Any thoughts?
    – ArdenDev
    Commented Oct 7, 2013 at 22:53
  • @IphoneDeveloper: You should ask a separate question with more details. Commented Oct 8, 2013 at 0:22
  • @QuinnTaylor Actually the convention is really helpful with the shortcut to switch between <class_name>.h & <class_name>.m (CMD+CTRL+UP_ARROW) cos' it also includes the <class_name>_Private.h :) At least in XCode5.
    – atxe
    Commented Jun 17, 2014 at 22:18
4

Looks like another question has the answer: Is there a way to suppress warnings in Xcode?

2
  • 1
    That has part of the answer, but the best (or at least a more complete) solution for a particular case is probably related to either Peter's answer or my answer. Commented Jul 8, 2009 at 17:15
  • This is also not the place to put duplicate links.
    – Rambatino
    Commented Oct 8, 2014 at 17:37
4

I was dealing with the same issue when I started with TDD few days ago. I've found this very interesting point of view in Test-Driven iOS Development book:

I have often been asked, “Should I test my private methods?” or the related question “How should I test my private methods?” People asking the second question have assumed that the answer to the first is “Yes” and are now looking for a way to expose their classes’ private interfaces in their test suites.

My answer relies on observation of a subtle fact: You already have tested your private methods. By following the red–green–refactor approach common in test-driven development, you designed your objects’ public APIs to do the work those objects need to do. With that work specified by the tests—and the continued execution of the tests assuring you that you haven’t broken anything—you are free to organize the internal plumbing of your classes as you see fit.

Your private methods are already tested because all you’re doing is refactoring behavior that you already have tests for. You should never end up in a situation where a private method is untested or incompletely tested, because you create them only when you see an opportunity to clean up the implementation of public methods. This ensures that the private methods exist only to support the class’s that they must be invoked during testing because they are definitely being called from public methods.

2
  • I read this book and although he is right at some point, he doesn't answer the question. Why shall I move an IBAction to the interface file (it shall be private to other classes)? This is not a model object, we are talking about a controller here, a view controller in particular (this book doesn't talk enough about UI testing on iOS) Commented Feb 12, 2014 at 0:02
  • My post is a reaction to the original question. I thought that the citation contained interesting point of view, because when you need to test private methods, it probably means that you should move those methods to the separate object. And this "leading" of the programmer to the smaller objects with shorter methods is the main benefit of the TDD. Of course, unit testing is not about UI testing. There are other tools (in instruments) and other tests for that. Commented Feb 12, 2014 at 8:54
3

While having a private header or defining your own category are probably more correct solutions there is also another very simple solution: cast the object to (id) before calling the method.

2
  • I would say this is preferable. No duplication, no fragmentation of source files and takes advantage of the dynamic nature of Objective-C.
    – Michael
    Commented Sep 18, 2013 at 4:54
  • 2
    @Michael a problem with this approach in xCode5 is that using an unknown selector will cause a warning. The compiler has to see the selector somehow when compiling a particular unit test. For this reason I like Quinn Taylor's solution. Commented Sep 23, 2013 at 9:55
3

If you don't want to distribute your private method implementations across multiple source files, a refinement to the Category solution is to define an Extension (essentially an anonymous Category - refer to Apple's documentation) in a header file that's imported by both your existing class' implementation and the relevant unit test source files.

Using an Extension allows the compiler to warn you if the implementation of the private method is not present in the main @implementation block. This link illustrates it nicely.

1
  • Please do not post links here, rather keywords. The link to apple has been moved. Commented Oct 11, 2013 at 19:47
1

Easy Job. Steps: 1. You have -(NSString*)getTestString; in your target m file for interface Foo

  1. Add a category in your unit test file:

    @interface DemoHomeViewController() -(NSString*)getTestString; @end

Then, do anything you want now.

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.