Skip to content
This repository has been archived by the owner on Mar 31, 2023. It is now read-only.

Removed exception handling (test smell) #188

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

eas5
Copy link

@eas5 eas5 commented Jul 28, 2020

This is a test refactoring.

Problem:
The exception handling test smell occurs when a test method explicitly a passing or failing of a test method is dependent on the production method throwing an exception. The improved test manually handled and declared exceptions that are never thrown.

Solution:
Utilize JUnit's exception handling to automatically pass/fail the test instead of writing custom exception handling code or throwing an exception.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant