IEEEComputerSocietyUNB / ProjetoChatbot

A telegram bot to help you deal with emotions on a daily basis

MIT License

12 stars 3 forks source link

Weekly update #75

Closed vinicius0197 closed 4 years ago

vinicius0197 commented 6 years ago

PR Description

Adicionado integração com banco de dados SQLite (via SQLAlchemy) para guardar mensagens enviadas pelo usuário na semana, assim como a funcionalidade de lembrar o usuário de contar ao bot como foi sua semana.

Fixes:

New Feature Submissions

To run tests locally, use make if you're on Mac or Linux.

[ ] Does your submission pass all the tests?
[x] Have you lint your code locally prior to submission?
[x ] Have you written proper tests to your added code?

New Dialog Submissions

User messages have - -, bot messages have -

[ ] Does all your user messages have lower letters?
[ ] Have you lint your dialogs locally prior to submission?
[ ] Have you written proper tests to your added dialogs on test_dialogs?
[ ] Does your dialog have an existing category? If not, did you create it?

Changes to Core Features

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests or modified old ones for your core changes, as applicable?
[ ] Have you successfully run tests with your changes locally?

vinicius0197 commented 6 years ago

@alexandrebarbaruiva Percebi que está dando problema com a função weekly_update, que usa o job_queue, assim como a text_message.

codecov[bot] commented 5 years ago

Codecov Report

Merging #75 into master will decrease coverage by 0.17%. The diff coverage is 73.98%.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   81.42%   81.25%   -0.18%     
==========================================
  Files           6        7       +1     
  Lines         280      304      +24     
  Branches       22       21       -1     
==========================================
+ Hits          228      247      +19     
- Misses         45       50       +5     
  Partials        7        7

Impacted Files	Coverage Δ
bot/model/WeeklyLog.py	`100% <100%> (ø)`
bot/periodic_messages_util.py	`62.96% <62.96%> (ø)`
bot/application.py	`84.07% <75%> (+4.49%)`	:arrow_up:
bot/dbutils.py

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 37ba6ba...b2485c5. Read the comment docs.

williamsimiao commented 5 years ago

@vinicius0197 assim que o @alexandrebarbaruiva fizer o merge da branch periodic_messages na master pode fazer o merge da master nessa branch aqui, porque acho que a baixa cobertura de teste eu já tinha resolvido na periodic_messages, então depois desse merge o codecov vai passar.