exercism / exercism/python

[Error-Handling]: instructions are unclear, and wrong programs pass the tests

Open
#1,780 11 comments 0 reactions 1 assignee Claimed by @BethanyG View on GitHub
bug 🐛 smolder🍲
Dominant language
Python
Stars
2.5k
Forks
1.5k
PR merge metrics
No merged PRs in 30d

Description

I just did the error-handling exercise, and had several issues:

- It was unclear at first what to do with inputs. Looking at the tests, it seemed that trying to convert the input to `int` was the right thing to try, but this is not mentioned in the readme or the stubs.
- The test for whether a method throws an exception will pass any error that might occur, not just if it's intentionally raised:
```python
def handle_error_by_throwing_exception():
raise "(╯°□°)╯︵ ┻━┻" # Raising a string is a TypeError, but passes the test
```
- When returning a tuple, which element should be the result and which one the status? What should the result element be when it fails? No suggestion
- No mention of advantages or disadvantages of the different kinds of error handling, or what is idiomatic Python.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.