search

gratipay / grtp.co

Gratipay Widgets + API
https://grtp.co/
MIT License
26 stars 19 forks source link

Explain tests better #132

Closed mattbk closed 8 years ago

mattbk commented 8 years ago

Copy test changes from team-experiments branch.

Based on discussion at https://github.com/gratipay/grtp.co/pull/112#issuecomment-212478637. Breaking up that PR into more manageable chunks.

Note: Removed test for anonymous receiving because there is no more anonymous receiving, only anonymous giving. See comment below.

chadwhitacre commented 8 years ago

@mattbk I redid the commit to prefix the anonymous-receiving test instead of removing it, on the theory that we should remove that test when we remove or modify the rest of the receiving code (cf. https://github.com/gratipay/grtp.co/issues/103#issuecomment-215236272). With that, I'm good to merge here if you are.

mattbk commented 8 years ago

With that, I'm good to merge here if you are.

I just saw this. Since you touched it last, I'll merge.