search

slackapi / node-slack-interactive-messages

Slack Buttons, Menus, and Dialogs made simpler for Node
MIT License
133 stars 41 forks source link

Async dispatch #34

Closed aoberoi closed 6 years ago

aoberoi commented 6 years ago

Summary

Fixes #33

Lands tests for express-middleware.

Requirements (place an x in each [ ])

codecov[bot] commented 6 years ago

Codecov Report

Merging #34 into master will increase coverage by 20.34%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master      #34       +/-   ##
===========================================
+ Coverage   79.06%   99.41%   +20.34%     
===========================================
  Files           3        3               
  Lines         172      170        -2     
===========================================
+ Hits          136      169       +33     
+ Misses         36        1       -35
Impacted Files Coverage Δ
src/adapter.js 99.15% <100%> (+0.01%) :arrow_up:
src/express-middleware.js 100% <100%> (+85.36%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a44a9bf...2e3a723. Read the comment docs.