Skip to content

Feature2/eli singer#81

Open
YeetMaster467 wants to merge 6 commits intodevelopmentfrom
feature2/EliSinger
Open

Feature2/eli singer#81
YeetMaster467 wants to merge 6 commits intodevelopmentfrom
feature2/EliSinger

Conversation

@YeetMaster467
Copy link
Collaborator

ChillGame is basically just hangman. Firstly, say !chillGame in the chat to start it, and then guess letters by typing !chillGame e, or any other letter. The bot will have a display of a dash representing every letter in the randomly selected word, and the dashed letter will be revealed if the user guesses the letter correctly. If the user guesses every letter correctly, or if they guess the word. It will tell them they won and the game will reset.

@opie10
Copy link

opie10 commented Aug 27, 2025

Code is good, everything seems to work, however with tests FeatureOne has 100% method coverage, but only 52% line coverage. There are certain cases within your handle method which are never checked (ranInt = 1, 2, 3, 4 or invalid (default). I would suggest moving the switch cases into their own function which takes an int as an input, so you can keep the randomness, but also test it easily within a loop.

Copy link

@opie10 opie10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I put this in the wrong place so:

Code is good, everything seems to work, however with tests FeatureOne has 100% method coverage, but only 52% line coverage. There are certain cases within your handle method which are never checked (ranInt = 1, 2, 3, 4 or invalid (default). I would suggest moving the switch cases into their own function which takes an int as an input, so you can keep the randomness, but also test it easily within a loop.

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

Successfully merging this pull request may close these issues.

2 participants