Skip to content

word-count: Removed test case #501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
behrtam opened this issue Mar 8, 2017 · 4 comments
Closed

word-count: Removed test case #501

behrtam opened this issue Mar 8, 2017 · 4 comments

Comments

@behrtam
Copy link

behrtam commented Mar 8, 2017

After some discussion we adopted one of your test cases symbols are separators (exercism/python#177) because only splitting on white space was seen as to easy. Now I saw that you removed that test case in #232 when you switched from HUnit to hspec.

Any specific reason why you dropped that test case or did you just take what was part of the x-common json at that time?

@rbasso
Copy link
Contributor

rbasso commented Mar 8, 2017

I guess I just used what was in x-common, because it was easier and also to comply with it (the Haskell track was too "unique" at the time), but I also think that tests should not be too prescriptive.

It this particular case, I'm not sure if it makes the exercise more interesting. The core of the problem is counting things, while the accessory problem is extracting the words from a string.

With it the exercise is a little harder. Without it people have a little more freedom.

@petertseng
Copy link
Member

exercism/rust#196 also did not like the test.

Neither in that track did we have a specific reason for removing it, other than x-common.

@petertseng
Copy link
Member

The test was added in exercism/exercism#650.

@behrtam
Copy link
Author

behrtam commented Mar 8, 2017

Okay, than I will probably remove that test from the python track as well to be consistent.

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

No branches or pull requests

3 participants