-
Notifications
You must be signed in to change notification settings - Fork 11
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix/restore_dead_code #256
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||
---|---|---|---|---|
|
@@ -3,7 +3,6 @@ | |||
from unittest.mock import patch, Mock | ||||
|
||||
from ovos_bus_client.session import Session | ||||
from ovos_config import Configuration | ||||
from ovos_utils.fakebus import FakeBus, Message | ||||
|
||||
from ovos_plugin_manager.templates.tts import TTS, TTSContext | ||||
|
@@ -119,24 +118,8 @@ def test_format_speak_tags_with_speech_no_tags(self): | |||
tagged_with_exclusion = TTS.format_speak_tags("Don't<speak>Speak This.</speak>But Not this.", False) | ||||
self.assertEqual(tagged_with_exclusion, valid_output) | ||||
|
||||
def test_playback_thread(self): | ||||
pass | ||||
# TODO | ||||
|
||||
def test_tts_context(self): | ||||
pass | ||||
# TODO | ||||
|
||||
def test_tts_validator(self): | ||||
pass | ||||
# TODO | ||||
|
||||
def test_concat_tts(self): | ||||
pass | ||||
# TODO | ||||
|
||||
def test_remote_tt(self): | ||||
pass | ||||
from ovos_plugin_manager.templates.tts import TTSValidator | ||||
# TODO | ||||
|
||||
|
||||
|
@@ -193,15 +176,15 @@ def test_get_tts_config(self, get_config): | |||
self.CONFIG_SECTION, None) | ||||
|
||||
def test_get_voice_id(self): | ||||
pass | ||||
from ovos_plugin_manager.tts import get_voice_id | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove unused import: get_voice_id The import Apply this diff to remove the unused import: - from ovos_plugin_manager.tts import get_voice_id Committable suggestion
Suggested change
ToolsRuff
|
||||
# TODO | ||||
|
||||
def test_scan_voices(self): | ||||
pass | ||||
from ovos_plugin_manager.tts import scan_voices | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove unused import: scan_voices The import Apply this diff to remove the unused import: - from ovos_plugin_manager.tts import scan_voices Committable suggestion
Suggested change
ToolsRuff
|
||||
# TODO | ||||
|
||||
def test_get_voices(self): | ||||
pass | ||||
from ovos_plugin_manager.tts import get_voices | ||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Remove unused import: get_voices The import Apply this diff to remove the unused import: - from ovos_plugin_manager.tts import get_voices Committable suggestion
Suggested change
ToolsRuff
|
||||
# TODO | ||||
|
||||
|
||||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Remove unused import: TTSValidator
The import
from ovos_plugin_manager.templates.tts import TTSValidator
is unused in the current context. This aligns with the static analysis tool's suggestion to remove it to clean up the codebase.Apply this diff to remove the unused import:
- from ovos_plugin_manager.templates.tts import TTSValidator
Committable suggestion
Tools
Ruff