Ranter
Join devRant
Do all the things like
++ or -- rants, post your own rants, comment on others' rants and build your customized dev avatar
Sign Up
Pipeless API
From the creators of devRant, Pipeless lets you power real-time personalized recommendations and activity feeds using a simple API
Learn More
Comments
-
Tobyvw6986yIn test driven development you're supposed to change/write your codebase in such a way that it IS possible to test.
For example by passing parameters to the class and mocking these in your tests -
@Tobyvw but doesn't this introduce params that are not required, for example why would my production code have a param that says: isMock (or whatever that helper param is called)
-
cannonau1966yIn a unit test you would do this by making sure the actual function f is never called from the calling function, by using inversion of control (for example, dependency injection), and swapping the function f with a function that always returns the value (or throws the exception) for the situation you are testing. This is called a mock function (other relevant terms: stub, spy).
You basically say:
If the function f returns "failed"
..(by replacing the function f with an actual function { return "failed"; })
...then I expect my program (the calling function) to behave in some way.
This way, it does not matter the functionality hasn't been implemented yet in your actual dependency; when it does (presumably because you will be coding that next), your function will be ready for it. -
Tobyvw6986y@cannonau This is what I meant.
Mocking is not the art of creating an extra variable, it's "faking" an object in your test. -
willol13896yIf you doing unit tests, you probably shouldn't have IO anyway, you use mocks (and get your mock to throw in your case).
I usually prefer mocks rather than using dependency injection, as the latter leads to using factories or this kind of pattern that adds indirection. Plus, with DI you still have to mock the dependencies you want to inject (even if they aren't used, as they are required by constructor). -
Generally, testing for something that will only go wrong under very rare circumstances might just be too much effort (considering the actual benefit of that particular test).
So instead, maybe simply review the error handling code once more and call it a day.
There will almost certainly be more critical tests to write ;) -
@Tobyvw @cannonau @losdanielos @willol
I still don't understand, how do I test for this:
try
{
sendEmail()
}catch(ex){
// Email not sent
}
How do I test code in catch block? When my function takes only title and body and to as params -
Tobyvw6986yIf sendMail is a function in a different class, mock the Object of that class.
If sendMail is a function in the same class, it's a tad more difficult, and to fully comply to tdd, you probably need to refactor. While you could try to mock sendMail()
(Using PowerMock in Java iirc), this is considered bad practice. -
@Tobyvw Ok point is clear now, thanks man I'll read more about it. Happy new year :)
-
felixandrea2185dSome ways to test for unreachable code or extreme cases:
Fault simulation: To simulate I/O errors or lack of disk space, you can use fault simulation tools such as fault injection frameworks or hardware emulators.
Create test scenarios: Create test cases to catch exceptions and unreachable conditions. For example, you can write test cases to raise exceptions and ensure that the code handles them properly.
Code coverage: Use code coverage tools to identify untested parts of code and focus on them.
Check for branches that cannot be reached: Check for branches of code that cannot be accessed to ensure that they do not affect the main logic.
Check for extreme cases: Check for extreme cases like invalid input values, system errors, etc. to ensure that the application handles them appropriately. -
whoopigoldberg0137dCheck for extreme instances: Make that the application handles severe cases such invalid input values, system faults, etc.
Related Rants
How do you test unreachable code or part that is considered an edge case?
For example I catch exceptions in case IO failed and data was not written on database, but that only happens if hardware failure, or no disk space left, how do I mimic that?
I also have unreachable code for example, in one layer I fetch data (lets call it function x) and always return success result unless item not found I throw KeyNotFound exception. But in the calling function I handle the case of Status == Failed
Just in-case in the future I change function x and start returning failed status, so my logic already written but never reachable
question
unit tests
how to handle this situation?