Skip to main content
Software Engineering

You are not logged in. Your edit will be placed in a queue until it is peer reviewed.

We welcome edits that make the post easier to understand and more valuable for readers. Because community members review edits, please try to make the post substantially better than how you found it, for example, by fixing grammar or adding additional resources and hyperlinks.

Required fields*

9
  • This is interesting. Our test suite devotes a lot of energy to testing things I think of as "plumbing", and not valuable to test - e.g. making sure the MCV controller method invokes a mediator with a certain request. Perhaps that's what you'd call "fascia"? (I would agree if you're saying it's not worth testing.) My reading of Michael Feathers' article however is that the method signature and the language-specific method delimiters (e.g. curly braces in C#) are the fascia. Commented Sep 2, 2022 at 10:58
  • 3
    That's exactly what I'm saying. Coverage tools are good at showing you what you missed but poor at telling you what to care about. My reading of Feathers' article is that "extracting methods" adds code. But the code it adds isn't interesting. That method A uses helper method B isn't interesting to test. That method A calculates tax based on state or province, now that's interesting. Commented Sep 2, 2022 at 13:11
  • Ah, makes sense. The boundary between the controller and the mediator's handler is kinda arbitrarily-imposed, thus it's a fascia. We've squeezed the juice into the handler and we're left with something we don't want to eat (or test) in the controller. Commented Sep 3, 2022 at 0:32
  • 1
    Tests are useful even when they pass. it's curious. I look at it the other way around. To me, tests are useful when they fail. The safety net worked and they caught an issue I missed when analysing or implementing the change. Commented Sep 6, 2022 at 10:33
  • 1
    @Laiv that’s certainly true. Just saying well designed passing tests against an api are also good examples of how to use the api. Commented Sep 6, 2022 at 12:52