Skip to content
This repository has been archived by the owner on Apr 5, 2021. It is now read-only.

Add cli commands pause resume; fix stop #88

Open
wants to merge 66 commits into
base: master
Choose a base branch
from

Conversation

Wandersalamander
Copy link

The command line options pause and resume were added and tested with a Deebot 900.
Besides the stop command was not working, which is fixed as well.

bmartin5692 and others added 30 commits January 10, 2019 22:40
Add EcoVacsIOT in VacBot
Add SpotArea option for D90X w/ Mapping
Add clean_status for IOT
- Reports cleaning or paused
Add backward action
Fix tests
Add tests for iot
Modify Clean to use kwargs for SpotArea
Add spotarea command tests
Add spotclean to cli
Cleanup init code from adding IOT
Remove egg & update gitignore for vscode files
Fix xmpp to work with iot
Handle already charging
Small cleanup
Set timeout to 1.25 secs for IOT API
- Some commands (Move) never return a status, so this allows things to continue moving along without waiting too long
WIP: Add initial EcoVacsMQTT client
- Connect and get message

TODO: Parse messages and plumb to events
MQTT Client plumbed up
- Connect, disconnect, statuses working
Add test MQTTPing & Fix tests
Fix clean from CLI
Inject an action='start' for IOTvacs
Add to protocol.md
Fix tables in protocol
Add more iot tests
Add mqtt tests
Added verify False to requests. Needed for work on Bumper
Make ssl verification optional
Can be set via config manually, or used from library
Fix for custom commands with multiple inner tags
Updated tests
stop printing positiion updates
Fix spot area clean
bmartin5692 and others added 30 commits February 18, 2019 22:22
Added paho-mqtt dependency.
Update SpotArea and CLI command to more closely reflect the app
Library:
SpotArea namedarea -> area
SpotArea customarea -> map_position

CLI Commands:
spotclean -> area
Ex: sucks area 0,1
  - will clean areas 0 and 1 / A and B

area options:
--map-postion|-p
  - will clean a specified map coordinate
Ex: sucks area -p "-602,1812,800,723"
  - will clean the specified custom map coordinates
Update protocol.md to reflect changes to CLI area command.
- Changed ClientXMPP to login/bind with resource
- Changed clean to always add the action
- Wrap command calls a new getReqID to add an ID to the ctl if needed
  - This was required for the ozmo commands, and shouldn't affect other boths

- Updated tests
Remove failing python 3.5 test for now
Is 3.5 failing because of a space?
It wasn't because of a space.... The tests were failing depending on the run because the xml string may have the id in a different order from run to run.

Changed test to convert string to xml for testing id exists or matches customid.
Add D600 (dl8fht) to iotmqdevices - CR: Fredric Palmgren
Removing a test based on the assumption that iot products may not work as iotmq.  This broke because D600 was added to the list of iotmq devices, but we can't assume the other devices from the productiotmap may not be added to the iotmq list in the future and this will just keep breaking.  It really doesn't provide much value from a test perspective.
Add SetWaterPermeability setting to protocol.md - wpietri#64
Update README API example to show import properly per wpietri#65.
Add verify_ssl option to cli login
Change str_to_bool func allowing for providing a CA Cert - Useful for Bumper
Change portal url to use portal-ww instead of portal-continent, this seems to fix the issue with logging in from de.

Change SetIOTMQDevices to check for device company instead of pre-defined list.
forgot to update the tests again...
Add retry for set token error
first attempt retries as-is
second attempt retries with "ww" continent
`clean 0` will clean until the bot returns and is charging again
- equivalent of 'auto' from the bot / app
- sucks cli will execute the next command after the bot is charging again
Added command line arguments pause, resume and corresponding classes.
Fixed stop command, which was not working my robot.
Tested with Deebot 900
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants