search

hapijs / crumb

CSRF crumb generation and validation for hapi
Other
171 stars 50 forks source link

Update dependencies #124

Closed lenovouser closed 5 years ago

lenovouser commented 5 years ago

Closes #123

lenovouser commented 5 years ago

@Marsup when upgrading lab I get a few missing coverage notices:

How would I go on fixing / adding tests for them? They all seem to be lines accessing the internal _crumb variable. Not too familiar with this module.

https://github.com/hapijs/crumb/blob/bd115fb25ea11fbe6ab3719e2e4514c753b29b60/lib/index.js#L92-L94

https://github.com/hapijs/crumb/blob/bd115fb25ea11fbe6ab3719e2e4514c753b29b60/lib/index.js#L127-L129

https://github.com/hapijs/crumb/blob/bd115fb25ea11fbe6ab3719e2e4514c753b29b60/lib/index.js#L132-L136

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.