search

luizpicolo / rails_admin_content_builder

:eyes: Easy way for create contents using rails_admin
MIT License
21 stars 5 forks source link

increase coverage test #12

Open luizpicolo opened 8 years ago

luizpicolo commented 8 years ago

https://coveralls.io/github/luizpicolo/rails_admin_content_builder?branch=master

sampatbadhe commented 8 years ago

@luizpicolo I can help you on this.

Do I know the reason of having :id in routes of search_content? https://github.com/luizpicolo/rails_admin_content_builder/blob/master/config/routes.rb#L2

I dont see any such use of id here.

sampatbadhe commented 8 years ago

Hello @luizpicolo

I have made some changes for this https://github.com/luizpicolo/rails_admin_content_builder/compare/master...sampatbadhe:increase_test_coverage?expand=1

Can you please check and let me know if I can raise a PR for the same.

jacobherrington commented 7 years ago

Writing some tests for this, will PR later today.

luizpicolo commented 7 years ago

😃