-
Notifications
You must be signed in to change notification settings - Fork 52
Telegram v2 #328
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Telegram v2 #328
Conversation
…into telegram-v2
…into telegram-v2
…into telegram-v2
Codecov Report
@@ Coverage Diff @@
## develop #328 +/- ##
========================================
Coverage 98.05% 98.05%
========================================
Files 73 73
Lines 3194 3194
========================================
Hits 3132 3132
Misses 62 62 Continue to review full report at Codecov.
|
@@ -140,7 +140,7 @@ async def test_user_not_registered(telegram_client): | |||
assert response.status_code == status.HTTP_200_OK |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
prefer response.ok
@@ -140,7 +140,7 @@ async def test_user_not_registered(telegram_client): | |||
assert response.status_code == status.HTTP_200_OK | |||
assert b'Hello, Moshe!' in response.content | |||
assert b'To use PyLendar Bot you have to register' \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we shouldn't use \ for line breaks.
No description provided.