Add Per Camera Snooze Functionality
Description:
Implements a per camera snooze function as present here: https://github.com/MattTW/BlinkMonitorProtocol/issues/67
Currently supports:
- Getting the remaining snooze time for the camera in the format of "this time stamp is when snoozing will expire".
- Setting the camera to snooze notifications
- I was unable to find a value other than "240" minutes (4 hours) that worked.
There is more than can be implemented, such as snoozing an entire sync module's cameras.
As far as testing:
- Currently blocked by another PR (https://github.com/fronzbot/blinkpy/pull/974)
- I have tried writing tests for this PR but just not familiar enough with tox/mock testing. Seems like only 3 tests are needed:
- "snooze_till" property
- "snooze" function
- "request_camera_snooze" request
Reason for removing my previous PR: Somehow messed up the git history while trying to squash to a single commit.
Related issue (if applicable): fixes #
- https://github.com/fronzbot/blinkpy/issues/955
Checklist:
- [ ] Local tests with
toxrun successfully PR cannot be meged unless tests pass - [X] Changes tested locally to ensure platform still works as intended
- [ ] Tests added to verify new code works
Just merged #974 so you should be able to pull those changes in and retrigger the tests. I'm hoping to be able to review this PR over the weekend. Been super busy, sorry for the delay!
I have added, but not manually tested, per sync module snooze. The api is documented in a comment on https://github.com/MattTW/BlinkMonitorProtocol/issues/67.
I have not added automated testing for my methods yet, but will take another stab at it now that the aiohttp PR has been merged.
There hasn't been any activity on this pull request recently. This pull request has been automatically marked as stale because of that and will be closed if no further activity occurs within 7 days. Thank you for your contributions.