search

caxlsx / caxlsx_rails

A Rails plugin to provide templates for the axlsx gem
MIT License
742 stars 84 forks source link

Add a testing section #98

Closed sly7-7 closed 6 years ago

sly7-7 commented 6 years ago

As asked in https://github.com/straydogstudio/axlsx_rails/issues/31#issuecomment-371884437 :smile:

sly7-7 commented 6 years ago

@straydogstudio Here is it. Obviously, any suggestions are welcomed.

coveralls commented 6 years ago

Coverage Status

Coverage remained the same at 98.161% when pulling 332261f9fadae96740f69b42057641c0b0a4eec2 on sly7-7:patch-2 into fe5ca6c98e3890887eef7549eeaccd43a95c4563 on straydogstudio:master.

straydogstudio commented 6 years ago

@sly7-7 Thank you very much! I think it looks great. At some point I'll add some simple testing examples outside of RSpec also. No need for now though.

sly7-7 commented 6 years ago

You're welcome :)

sly7-7 commented 6 years ago

@straydogstudio Was it reopened on purpose ?

straydogstudio commented 6 years ago

Ha. I closed it before I merged it. I didn't realize it would remove the merge button if I closed it. :-0

sly7-7 commented 6 years ago

Ha, ok, so nvm, I thought it was still open. I just forgot that merging automatically close the PR.