Skip to content

Commit 50ff130

Browse files
committed
Refactor skill module tests to enable translator instead of disabling tests
1 parent 76719cc commit 50ff130

1 file changed

Lines changed: 30 additions & 23 deletions

File tree

test/test_skills_module.py

Lines changed: 30 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -79,26 +79,33 @@ def test_neon_skills_service(self, run):
7979
from neon_core.skills.service import NeonSkillService
8080
from neon_core.skills.skill_manager import NeonSkillManager
8181

82-
config = {"skills": {
83-
"disable_osm": False,
84-
"auto_update": True,
85-
"directory": join(dirname(__file__), "skill_module_skills"),
86-
"run_gui_file_server": True
87-
},
88-
"location": {"timezone": {"code": "America/Los_Angeles",
89-
"name": "Pacific Standard Time",
90-
"dstOffset": 3600000,
91-
"offset": -28800000},
92-
"coordinate": {"latitude": 47.482880,
93-
"longitude": -122.217064},
94-
"city": {"code": "Renton",
95-
"name": "Renton",
96-
"state": {"code": "WA", "name": "Washington",
97-
"country": {"code": "US",
98-
"name": "United States"}
99-
}
100-
}
101-
},
82+
config = {
83+
"utterance_transformers": {
84+
"neon_utterance_translator_plugin": {"active": True}
85+
},
86+
"skills": {
87+
"disable_osm": False,
88+
"auto_update": True,
89+
"directory": join(dirname(__file__), "skill_module_skills"),
90+
"run_gui_file_server": True
91+
},
92+
"location": {
93+
"timezone": {
94+
"code": "America/Los_Angeles",
95+
"name": "Pacific Standard Time",
96+
"dstOffset": 3600000,
97+
"offset": -28800000},
98+
"coordinate": {
99+
"latitude": 47.482880,
100+
"longitude": -122.217064},
101+
"city": {
102+
"code": "Renton",
103+
"name": "Renton",
104+
"state": {"code": "WA", "name": "Washington",
105+
"country": {"code": "US", "name": "United States"}
106+
}
107+
}
108+
},
102109
}
103110

104111
started = Event()
@@ -306,9 +313,9 @@ def _handle_languages_response(msg):
306313

307314
# Patch things
308315
real_config = self.intent_service.language_config
309-
# self.assertIn("neon_utterance_translator_plugin",
310-
# self.intent_service.transformers.loaded_modules,
311-
# self.intent_service.transformers.loaded_modules)
316+
self.assertIn("neon_utterance_translator_plugin",
317+
self.intent_service.transformers.loaded_modules,
318+
self.intent_service.transformers.loaded_modules)
312319
translator = self.intent_service.transformers.loaded_modules.get(
313320
'neon_utterance_translator_plugin')
314321
real_plug = translator.translator

0 commit comments

Comments
 (0)