let’s say there is a Shape interface.

interface IShape { double Area(); }

A Rectangle class and a Triangle class implement it. Now should i write tests for:

  1. IShape interface and test both implementations in a single test file?
  2. Write tests for Rectangle and Triangle class separately, testing their implementation of Area() ?
  3. Do something else?

From what I see I am testing implementations either ways. How do you even test an interface without testing the implementation? Can someone please help clarify my doubts? Thanks!

  • hallettj@beehaw.org
    link
    fedilink
    arrow-up
    52
    ·
    11 months ago

    That advice does not literally refer to interface the programming language feature. It means to test the observable behavior of a component, not internal implementation details.

    In your example, write tests for both Rectangle and Triangle that call area, and assert the result is correct. But do not test, for example, the order of mathematical operations that were run to calculate the result. The details of the math are an internal detail, not part of the “interface”.

    • pizzahoe@lemm.eeOP
      link
      fedilink
      arrow-up
      18
      ·
      11 months ago

      Fuck! Now it makes sense to me. I thought it literally meant an interface. Thanks a lot for clarifying!

      • hallettj@beehaw.org
        link
        fedilink
        arrow-up
        1
        ·
        11 months ago

        No problem! I thought there was a good chance you already know the concept, just not in the exact, unfortunately-overloaded words of your post title.

  • Alex@programming.dev
    link
    fedilink
    arrow-up
    15
    ·
    edit-2
    11 months ago

    Testing interface is not about exactly interface as language’s feature. It’s about testing outer public interface (public API) of you things, looking at it as a black-box and doesn’t matter what happens inside. That’s all it means.

  • Aurenkin@sh.itjust.works
    link
    fedilink
    arrow-up
    9
    ·
    11 months ago

    From my understanding the interface being referred to in the original guideline doesn’t necessarily refer to an interface as in the language concept but the more general concept of the interfaces to a given part of the system. In your example, I would interpret it as suggesting that you should test the public interface of the shape ie, when I call the Area method with some known cases, does it return the expected value.

    Testing the implementation details in this case could be something like testing that the Area method for a circle called the Math.Pi function which would be an implementation detail. I don’t know if it’s the best or most helpful example but that’s my interpretation at least.

  • asyncrosaurus@programming.dev
    link
    fedilink
    arrow-up
    7
    ·
    11 months ago

    It’s a form of Black-Box Testing, essentially you want to validate expected behavior. Implementation can change, but your outcome should remain the same.

    This is a big target for Test Driven Development, since your first step is to write the test with the expected outcome, then you write the most basic implementation, and when you can verify the behavior, then you are free to re-factor to improve implementation knowing your test will tell you if the behavior changes with each internal change.

  • abhibeckert@lemmy.world
    link
    fedilink
    arrow-up
    6
    ·
    edit-2
    11 months ago

    Is that a real example, or a contrived one?

    I don’t think it’s normal to write a test for IShape.Area() in your example. You’d only write tests for Rectangle.Area() and Triangle.Area().

    Wether or not all of that is in the same file should, in my opinion, depend how many tests there are. If there are “too many” lines of code, split it into multiple files.

  • pelya@lemmy.world
    link
    fedilink
    arrow-up
    2
    ·
    11 months ago

    Testing the interface is more like validating the architecture of your module.

    Say, you have your interface IShape, which represents a geometric shape.

    Now you decide to add some shape editing capabilities to your qpplication.

    Can you rotate your IShape? Can you flip it? So your IShape interface needs methods rotate(double angle) and flip(enum direction). So you add a test that checks that your module exports methods rotate(double angle) and flip(enum direction).

    Does your application requires rotating shapes by a fractional angle? Maybe rotating by 90⁰ is enough? So you replace rotate(double angle) with rotate90degrees(bool clockwise) in your test and you can simplify your implementation.

  • hoodlem@hoodlem.me
    link
    fedilink
    English
    arrow-up
    2
    ·
    11 months ago

    There is no functionality in the interface so there is nothing to test. You test the logic of the implemented methods.

    With that said, with reflection you can at least test that an interface has the right method names and annotations present, but I’ve extremely rarely found it useful to do so.

  • kakes@sh.itjust.works
    link
    fedilink
    arrow-up
    2
    arrow-down
    1
    ·
    11 months ago

    Not an “expert” in testing per se, but I’m not sure I would ever attempt to “test” an interface. As you said, there is no implementation. You can only test implementation, so #2 in your list.