Skip to content
This repository has been archived by the owner on Sep 8, 2024. It is now read-only.

Add Mozilla TTS connection validation #2828

Merged
merged 1 commit into from
Feb 8, 2021
Merged

Conversation

krisgesling
Copy link
Contributor

Replication of PR #2719
Original code from @JarbasAl

Description

Validates the ability to connect to the defined Mozilla TTS instance.

How to test

Run Mozilla TTS and configure Mycroft to use it.

Contributor license agreement signed?

@devops-mycroft devops-mycroft added the CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors) label Feb 8, 2021
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

Copy link
Collaborator

@forslund forslund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the original PR (see #2719") we discussed changing the url to the root so config would be "url": "http://0.0.0.0:5002 so the test doesn't have remove the last part of the url, instead it would be added in the get_tts()

Replication of PR #2719
Original code from JarbasAl
@devops-mycroft
Copy link

Voight Kampff Integration Test Succeeded (Results)

Copy link
Collaborator

@forslund forslund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works great.

@forslund forslund merged commit be2d3c9 into dev Feb 8, 2021
@forslund forslund deleted the feature/mozilla-tts-validation branch February 8, 2021 21:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA: Yes Contributor License Agreement exists (see https://github.com/MycroftAI/contributors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants