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
    ·
    1 year 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
      ·
      1 year 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
        ·
        1 year 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
    1 year 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.

  • asyncrosaurus@programming.dev
    link
    fedilink
    arrow-up
    7
    ·
    1 year 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
    1 year 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.

  • hoodlem@hoodlem.me
    link
    fedilink
    English
    arrow-up
    2
    ·
    1 year 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.

  • pelya@lemmy.world
    link
    fedilink
    arrow-up
    2
    ·
    1 year 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.