FutureTense / keymaster

Home Assistant integration for managing Z-Wave enabled locks

MIT License

219 stars 43 forks source link

add another test #249

Closed firstof9 closed 2 years ago

firstof9 commented 2 years ago

Proposed change

Add test for starting slot not being 1

Type of change

[ ] Dependency upgrade
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (which adds functionality)
[ ] Breaking change (fix/feature causing existing functionality to break)
[x] Code quality improvements to existing code or addition of tests

Additional information

This PR is related to issue: #248

TODO:

[x] Mock get_usercodes function

codecov-commenter commented 2 years ago

Codecov Report

Merging #249 (940f792) into main (c5527e9) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #249   +/-   ##
=======================================
  Coverage   88.55%   88.55%           
=======================================
  Files          10       10           
  Lines         987      987           
=======================================
  Hits          874      874           
  Misses        113      113