forked from cleveragents/cleveragents-core
freemo
55aee7cf22
The step_register_skills_table step called add_skill in a loop but only committed once at the end. Because SkillRepository.create() obtains a new session per call and only flushes (never commits), the intermediate sessions could be garbage-collected before the final commit, rolling back their transactions on the shared SQLite :memory: connection. Moving _commit_pending inside the loop ensures each skill is durably committed before the next session is created. ISSUES CLOSED: #418
34 lines
1.4 KiB
Gherkin
34 lines
1.4 KiB
Gherkin
Feature: Skill CLI uncovered lines
|
|
Cover uncovered lines in src/cleveragents/cli/commands/skill.py:
|
|
273, 343-344, 353, 393, 589-590, 593-594.
|
|
|
|
Scenario: Show skill with unregistered include
|
|
Given a clean skill service
|
|
And a skill with an include pointing to a non-registered skill
|
|
When I run skill show for the skill with unregistered include
|
|
Then the show output contains the not registered marker
|
|
|
|
Scenario: Show skill when capability summary raises ValueError
|
|
Given a clean skill service
|
|
And a registered skill for capability error testing
|
|
When I run skill show with capability summary raising ValueError
|
|
Then the show output contains OK and no capability panel
|
|
|
|
Scenario: Show skill with dependent actors
|
|
Given a clean skill service
|
|
And a registered skill for dependent actors testing
|
|
When I run skill show with dependents returning actors
|
|
Then the show output contains the actors reference line
|
|
|
|
Scenario: Tools command with agent_skill entries
|
|
Given a clean skill service
|
|
And a registered skill with agent_skill sources
|
|
When I run skill tools for the agent_skill skill
|
|
Then the tools output contains agent_skill source type
|
|
|
|
Scenario: Remove skill with dependent actors
|
|
Given a clean skill service
|
|
And a registered skill for removal with actor dependents
|
|
When I run skill remove with dependents returning actors
|
|
Then the remove output contains the actor dependency warning
|