gbowne1 / TwitchBot

A MERN full stack TwitchBot
MIT License
7 stars 10 forks source link

tests done for Help.js #56

Closed Sky-De closed 8 months ago

Sky-De commented 9 months ago

Screenshot 2024-02-22 050827

Also i used helpContent obj in Help.js

Screenshot 2024-02-22 050901

jarmentor commented 9 months ago

Greetings -- just want to note that not all tests pass. The ones that are relevant to the PR do.

(unsure if this is the place to note this; I'm happy to raise issue)

Screenshot 2024-02-22 at 2 14 08 PM

Sky-De commented 9 months ago

Greetings -- just want to note that not all tests pass. The ones that are relevant to the PR do.

(unsure if this is the place to note this; I'm happy to raise issue)

Screenshot 2024-02-22 at 2 14 08 PM

@jarmentor Yes, I understand. The best thing about testing is that it helps to identify any issues or changes that may cause test failures. I ran a test on the About.js file and it seems that there have been some changes that caused the test to fail. Therefore, I will rewrite About.test.js to test the updated version of About.js and push new commit , review and test it, thanks for informing

Sky-De commented 9 months ago

@jarmentor it is done

Screenshot 2024-02-22 150025