exercism / ruby

Exercism exercises in Ruby.
https://exercism.org/tracks/ruby
MIT License
543 stars 512 forks source link

Insufficient test cases in `house` #1694

Closed armahillo closed 2 months ago

armahillo commented 2 months ago

Currently there is a single example in house that only requires a class method, :recite and that it dumps the full text of the rhyme. But this can technically be satisfied by doing a HEREDOC that outputs the prescribed text.

I was looking at the practice module and considered adding more tests, but I was unclear on what the intention was for this module.

Some possible outcomes I think could be tested for (based on what I believe this exercise is meant to teach):

Most of the exercises don't tend to coerce the creation of support objects, that's just something the user is meant to figure out on their own, but maybe this exercise could be a little more heavy-handed about that, because it seems like a good opportunity to do that?

All that is to say -- I think this is a potentially useful exercise that can be fun to work with and teach some cool stuff, but currently the tests do not provide the typical useful guidance found in most other exercises.

github-actions[bot] commented 2 months ago

Hello. Thanks for opening an issue on Exercism 🙂

At Exercism we use our Community Forum, not GitHub issues, as the primary place for discussion. That allows maintainers and contributors from across Exercism's ecosystem to discuss your problems/ideas/suggestions without them having to subscribe to hundreds of repositories.

This issue will be automatically closed. Please use this link to copy your GitHub Issue into a new topic on the forum, where we look forward to chatting with you!

If you're interested in learning more about this auto-responder, please read this blog post.