-
Notifications
You must be signed in to change notification settings - Fork 31
Conversation
132b689
to
65542be
Compare
@@ -117,3 +118,9 @@ gulp.task('build-js', function() { | |||
.bundle() | |||
.pipe(callback('static/js/common.js')); | |||
}); | |||
|
|||
gulp.task('test', function() { | |||
return mochify('./tests/unit/**.js') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking we'd keep dir organization here mirrored with the actual js. SO in this case, we'd have tests/unit/modules
or tests/unit/pages
. What do you think of that?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. I separated the election lookup logic into a resuable module (modules/election-lookup) and a page that imports and instantiates the module (pages/election-lookup), so the tests have moved to tests/unit/modules/election-lookup.
So when you change the zip input it clears the state and district filters, and vice versa.
Updated with style improvements from @noahmanger. Want to merge now for testing tomorrow? If not, I can deploy this branch directly to the dev server. |
[WIP] Feature/election lookup
Looks great. Let me know when you push it up. |
Prototype of #354.