Begin adding main view unit tests

cfpb / regulations-site

(DEPRECATED) Web interface for viewing U.S. federal regulations and other regulatory information

Other

28 stars 43 forks source link

Closed ascott1 closed 8 years ago

ascott1 commented 9 years ago

This adds some unit tests for content in the main-view of the app's JS. This is an ongoing work in progress.

Statements   : 57.74% ( 619/1072 )
Branches     : 37.84% ( 140/370 )
Functions    : 39.87% ( 61/153 )
Lines        : 57.8% ( 619/1071 )

@KimberlyMunoz, @willbarton, or @imuchnik

imuchnik commented 9 years ago

Some minor concerns in inline comments, otherwise, looks good!

ascott1 commented 9 years ago

Thanks for the thorough review @imuchnik! I think I've addressed all of your comments, but need to fix travis before this gets merged.

ascott1 commented 8 years ago

@willbarton is my hero for fixing the Travis job.