search

nzt4567 / gh_issue_agent

Created for the MI-PYT course @ FIT CTU
Creative Commons Zero v1.0 Universal
0 stars 0 forks source link

Test for main web app purpose is missing #4

Open hroncok opened 7 years ago

hroncok commented 7 years ago

For the web app, a crucial test is missing. The test would POST some json tot he hook and you would test if the hook reacted as required.

nzt4567 commented 7 years ago

As always, you're right. I must have missed that one, sorry about that. I am in the middle of something right now (MI-PAA task) - is fixing this tomorrow or over the weekend all right?

hroncok commented 7 years ago

Whatever works for you.

Just be clear, I already gave you some points and fixing this will not give you more. It is juts feedback and you don't need to fix it if you don't want to.

nzt4567 commented 7 years ago

Sure, no prob, I want to have it complete. Will do it over the weekend :)