search

go-telegram / bot

Telegram Bot API Go framework
MIT License
502 stars 46 forks source link

Add some unit tests #92

Open renatosaksanni opened 2 weeks ago

renatosaksanni commented 2 weeks ago

Just check this repo and wanna contribute some unit tests

negasus commented 1 week ago

Hi, we don't use external dependencies for this project. You can fix your PR and remove deps?

renatosaksanni commented 1 week ago

Ok @negasus will fix this one

renatosaksanni commented 1 week ago

Hi @negasus just add some fixes, could you give some reviews. thanks

renatosaksanni commented 1 week ago

Hi, from the remaining test failures, I found that we need to improve the webhook_handler.go logic to ensure that updates are not sent to the bot.updates channel when the request context is canceled. Therefore, I propose adding some lines to handle this.