34ee3a9a84
Upgrade websocket-client to 1.2.1 ( #2879 )
...
* Upgrade websocket-client to 1.2.1
core equivalent of https://github.com/MycroftAI/mycroft-messagebus-client/pull/21
There was an incompability with the latest websocket-client and the
messagebus which needed fixing for Linux distro compatibility. Since
messagebus-client was going to depend on websocket-client 1.2.1, let's
do the same in core
* Bump message bus client and tornado versions
New version of the messagebus-client released.
Upgraded Tornado to match new function signature.
* Revert removal of websocket-client
It is still used in the `mycroft.client.text.gui_server`
Should investigate removal of this so that websocket client versions do not need to be
kept in sync between mycroft-core and mycroft-messagebus-client.
Co-authored-by: Kris Gesling <kris.gesling@mycroft.ai>
2021-10-28 14:42:23 +09:30
df78af15cb
Merge pull request #3002 from MycroftAI/test/extend-criteria-matcher
...
Extended the idea of the VK CriteriaWaiter
2021-10-07 10:59:50 -05:00
4b729dc978
Make matcher class signatures more consistent.
2021-09-29 17:57:31 -05:00
acd6d4065a
Apply code review changes.
2021-09-28 13:08:16 -05:00
7f7460a6cd
Fix a PEP8 speaks issue
2021-09-14 14:49:39 -05:00
1856296ea7
Extended the idea of the criteria matcher to handle other types of matching
2021-09-14 14:40:27 -05:00
0896c3ceda
fixed an error with message formatting in a ValueError message.
2021-09-14 14:31:35 -05:00
9e9f2f74e5
Merge pull request #2975 from MycroftAI/feature/vk-only-shutdown-on-ci
...
VK: Only stop Mycroft services if running in CI
2021-09-13 16:37:49 +09:30
5a1e4ed8b9
Merge pull request #2990 from MycroftAI/bugfix/vk-bus-clear-messages
...
Fix messages clearing in the InterceptAllBusClient
2021-09-10 16:29:14 -05:00
59c473cac8
Fix pulseaudio daemon failing to start in CI
2021-09-03 14:54:09 +09:30
df94e2192a
fixed a few spelling errors.
2021-09-01 13:21:50 -05:00
b6066f90c6
fixed an issue with how messages are cleared in the InterceptAllBusClient
2021-09-01 13:15:53 -05:00
8d11349b65
Only stop Mycroft services if running in CI
2021-08-12 10:24:38 +09:30
170ebc44ec
Merge pull request #2951 from forslund/feature/then-wait-with-event-handler
...
Use event handler to capture messages in then_wait()
2021-08-05 15:40:04 +09:30
e40530a8ac
Merge pull request #2948 from forslund/bugfix/vk-message-race
...
Fix Possible vk message race
2021-08-05 15:39:12 +09:30
313f4e8759
Merge pull request #2946 from MycroftAI/refactor/vk-test-helpers
...
Improve the speed of waiting for dialogs helper function
2021-08-05 09:29:17 +09:30
b40fcf0e93
Update VK then step for checking messagetype
...
Simplify the function and use the standard then_wait() for the heavy lifting,
this makes it utilize the new event driven functionality.
2021-07-23 09:46:07 +02:00
171d3840f3
Remove sleeps from end of scenarios and features
2021-07-23 09:45:44 +02:00
578a3ec439
Use event to capture messages in then_wait()
...
Instead of busily polling the bus for new messages an event handler is
registered (and teared down after check is complete)
2021-07-23 09:45:44 +02:00
cbd17a8dc3
Remove mycroft-stock from install list
2021-07-22 17:20:04 +02:00
8f41d176d0
remove print statement used for testing
2021-07-09 16:39:49 -05:00
6fd97b4e85
fixed PEP8 issue
2021-07-09 15:45:07 -05:00
4b66fb1dd1
Add docstrings to InterceptAllBusClient
2021-07-09 17:06:33 +02:00
cf355360ad
Fix message race when using clear_messages()
...
This handles a scenario that a message arrives between a call to get_messages() and clear_messages(). clear_messages() will only clear the messages that has been evaluated atleast once.
A new method clear_all_messages() has been added to clear the entire message stack and is used between scenarios to reset the list.
2021-07-09 17:06:30 +02:00
105a5b4be4
Improve the speed of waiting for dialogs by exiting the loop after a match is spoken. Also provide error handling for when a match is not found.
2021-07-06 14:26:23 -05:00
18cb28088d
Added comments to document race condition
2021-07-06 13:48:52 -05:00
3959267726
Reordered wait_while_speaking and sleep to fix a race condition that was occurring with the converse logic in MycroftSkill.get_response()
2021-07-01 22:31:22 -05:00
8a3cb6eb88
Merge pull request #2741 from MycroftAI/bugfix/revert-vk-sleep
...
revert a temporary bugfix from PR #2626
2021-06-08 06:46:34 +09:30
e75a05557b
docstring cleanup
2021-05-11 15:10:32 +09:30
6dd059e888
Consume whole queue before considering skill done
...
This will handle any extra messages after the skill complete message was detected. For example messages from certain fallback skills
2021-05-02 09:27:47 +02:00
053d29b92c
Use the correct event for fallback handler
2021-05-02 09:26:29 +02:00
512fa9c820
Implement wait_for_response for old skill tester
...
Previously this was just skipped over but is now needed for the fallback intents
2021-05-02 09:25:14 +02:00
3970f84e4b
Fix Padatious setup
...
This was broken when Padatious moved from a separate "skill" into the
service manager. This updates the code to call the correct methods
2021-05-01 08:35:01 +02:00
89555380d8
Add VK method to wait for audio service
2021-01-18 11:16:41 +09:30
83ed811dba
Add check to make sure speak message has metadata
...
This can occur if a custom made speak message is injected and VK should
handle malformed speak messages.
2021-01-10 21:03:25 +01:00
35fd035337
Add timeout waiting for config change
2021-01-10 21:02:54 +01:00
8b107ce733
Merge pull request #2742 from forslund/bugfix/vk-reset-config-spam
...
Fix Voight kampff reset of patched config
2020-11-06 14:38:28 +09:30
dfe909aa2e
Add a log for when config is patched
...
Since we have a reset log a matching info message for when updating the
config should be shown
2020-10-29 08:05:32 +01:00
3de34003cf
Use custom cleanup to reset patched config
...
original_config will now only live in the scenario part of the context
being automatically cleared at the end of each scenario. To reset the
patch a custom cleanup is added when the patch is performed.
2020-10-29 08:05:28 +01:00
e0c7016b4d
revert a temporary bugfix from PR #2626
2020-10-28 16:30:10 +09:30
0a9de0ab7c
Update to pyee 8.1.0
...
pyee 8.1.0 adds a small change to make the once call to be more safe in
multithreaded environments.
This switches back from the now deprecated BaseEventEmitter to the
standard EventEmitter.
2020-10-22 11:43:06 +02:00
52b908f350
Merge pull request #2663 from forslund/feature/given-timeout
...
Voight Kampff given timeout
2020-09-09 14:32:25 +09:30
3eee84f053
Merge pull request #2679 from forslund/refactor/remove-imp
...
Refactor/remove imp
2020-08-24 01:28:34 +00:00
e2a7fe9b2a
Load test environment with importlib instead of imp
2020-08-21 08:00:57 +02:00
d70c425854
switch deprecated EventEmitter to other classes
2020-08-18 15:58:42 +09:30
9b4e47567e
Add context parameter to wait_for_dialog
...
This allows wait for dialog to respect the timeout set for the current
scenario.
2020-08-16 23:34:29 +02:00
03002bcf47
Add "given a timeout of X seconds"
...
The new given sets the timeout of the steps in a scenario. The timeout is reset after each scenario.
2020-08-16 23:16:51 +02:00
d4abc37861
sleep prior to tests to assist in bug diagnosis
2020-07-17 15:53:59 +09:30
247c6d30a3
add test - specific skill should not reply
2020-06-18 22:33:22 +09:30
e08f630bea
Minor header fixes in voight_kampff
...
- Copyright year
- typo
2020-05-18 14:48:19 +02:00
Bart Ribbers
Chris Veilleux
Kris Gesling
Åke Forslund
Kris Gesling