feat: add ovoscope end2end intent-routing tests#49
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Greetings! I've analyzed your changes and have some results to share. 🖖I've aggregated the results of the automated checks for this PR below. 📋 Repo HealthScanning for any signs of repository fatigue. 😫 ✅ All required files present. Latest Version: ✅ 🔍 LintChecking the boxes and crossing the T's. 🖋️ ✅ ruff: no issues 🎙️ SkillIs the skill's voice clear and confident? Let's see. 🗣️ 🎙️ (unknown skill_id) — 4 languages en-US: 6 intents · 4 vocab · 11 dialogs · skill.json Translation coverage — 3 languages (3 complete)
⚖️ License CheckScanning for any hidden proprietary blobs. 🌑 ✅ No license violations found (48 packages). License distribution: 12× MIT License, 9× MIT, 6× Apache Software License, 5× Apache-2.0, 2× BSD-3-Clause, 2× ISC License (ISCL), 2× PSF-2.0, 2× Python Software Foundation License, +7 more Full breakdown — 48 packages
Copyright (c) 2022 Phil Ewels Permission is hereby granted, free of charge, to any person obtaining a copy The above copyright notice and this permission notice shall be included in all THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR Policy: Apache 2.0 (universal donor). StrongCopyleft / NetworkCopyleft / WeakCopyleft / Other / Error categories fail. MPL allowed. 🏷️ Release PreviewEnsuring the 'Contributors' list is sorted and complete. 👥 Current:
✅ PR title follows conventional commit format. 🚀 Release Channel Compatibility Predicted next version:
🔒 Security (pip-audit)Ensuring our project remains safe and secure. 🛡️ ✅ No known vulnerabilities found (67 packages scanned). 🔨 Build TestsChecking the alignment of our build components. 📏
❌ 3.10: Build OK, install failed 📊 CoverageEvaluating the thoroughness of our test suite. 🔎 🔌 Skill Tests (ovoscope)Let's see how the skill performs under the microscope. 🔬 ❌ 0/23 passed, 23 failed ❌ **TestPadatious1_Make_a_choice_intent** — 0/2
❌ **TestPadatious2_Pick_a_number_intent** — 0/5
❌ **TestPadatious3_Flip_a_coin_intent** — 0/1
❌ **TestPadatious4_Fortune_teller_intent** — 0/5
❌ **TestPadatious5_Roll_single_die_intent** — 0/5
❌ **TestPadatious6_Roll_multiple_dice_intent** — 0/5
🚌 Bus CoverageScanning for any 'unasserted' emissions! 🚨 The silent guardian of the dev branch. 🦇 |
…ious assertions Suppresses speak/mycroft.audio.play_sound noise so message-count checks are not thrown off by extra bus events; routing verified via activation_points. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Reconcile the unit-test assertions and skill.json skill_id with the entry-point id declared in pyproject so e2e and unit tests agree.
…p ovos-* deps at next major
Auto-generated ovoscope intent-routing tests.
Covers Padatious intents (exact message-type assertions) and Adapt intents
(speak-response assertions). One test method per canonical utterance.
Generated by
tools/gen_ovoscope_tests.pyfrom the dataset atknowledge/datasets/ovoscope/test_dataset.jsonl.Run:
pytest test/end2end/ -v --timeout=60