alreadybored / basic-testing Goto Github PK

Current Behavior:
The default case in the switch statement of the simpleCalculator function throws an error, even though it is unreachable due to the preceding validation logic.

Expected Behavior:
The default case and the associated error throw statement are unnecessary and can be safely removed.
Since the isInputValid function already checks the validity of the input and returns false if it's invalid, the control flow will never reach the default case.

Additional Context:
The purpose of the isInputValid function is to validate the input before performing calculations. When the control flow reaches the switch statement, it indicates that the input is valid, and there is no need for the default case.
Removing the default case helps achieve 100% coverage:

Please address this issue by removing the default case and the associated error throw. Thank you.