ZoranPandovski / flaskr-intro

:mailbox_with_no_mail: Mini blog application following the Flask tutorial :mailbox_with_no_mail:
MIT License
6 stars 11 forks source link

Add test for /search/ path #13

Closed vhugo closed 5 years ago

vhugo commented 5 years ago

Hi,

This change should increase coverage and contribute to issue #11 but the test is kind of worthless, as the actual functionality proposed for the method is not being tested nor is there.

I've tried to create a more robust test, using the actual test database with test data, but I'm not very familiar with Flask and SQLAlchemy so I don't see an easy solution without actually rewriting most of the work you already put in.

Anyway, I found this https://github.com/pallets/flask/tree/master/examples/tutorial/flaskr in the Flask tutorial pages, which could be taken into consideration for this project.

Best regards. 😉