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
Lng: Add cs-cz noise words #3135
base: dev
Are you sure you want to change the base?
Conversation
Voight Kampff Integration Test Failed (Results). |
Hi, VK for Wiki failed, what about adding Github action to it (same as in HA skill)? |
I think the latest version of wiki doesn't have the issue but is held back waiting for a mycroft release. The wiki-skill is a bit troublesome, the problem is due to wikipedia returning a new answer for a question. It was working on last release of the wiki-skill but has since changed. I think using something like betamax in VK providing a set of stable data from the web might be one way to go. The problem is that it means that the tests won't tell the devs whether the skill is actually working alright with real world data. That said, per skill unit/integrationtests is a great idea :) |
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.
Google translate tells me this matches well with the english noise words. Nice! (or should I say Noise!)
Testing against real integration is always better than just testing against fixed data. That is why I use real Home Assistant installation and not just mocked subset. Also, CI per repo give you instant feedback when doing changes. If you want it for wiki skill, let me know and I will prepare it. |
In ovos and neon we have unittests per skill, i agree it is the way to go and way more useful than VK imho Eg https://github.com/OpenVoiceOS/skill-ovos-wikipedia/tree/dev/tests/unittests |
With HA skill, I run both VK and unittest (allure report is generated). |
I think using vk in core is valid, even with fixed data. It helps to ensure that a change in core doesn't break. I think both unit tests and the high level tests vk provides are needed to guard against regressions. For validating skills functionality during (skill) development unittest are probably more valuable. |
To wrap it, should I prepare PR with GitHub Action? |
Description
Hi, PR contain Czech version of noise words.
Contributor license agreement signed?
CLA [x] (Whether you have signed a CLA - Contributor Licensing Agreement