Skip to content

Commit cff7793

Browse files
committed
test: Add 12 new test cases for improved coverage
Added test cases breakdown: - 7 negative cases (other frameworks): FastAPI, SwiftUI, Kotlin, Next.js, Laravel, Rust, Go - 5 edge cases: Mixed framework mentions, ambiguous OData, version queries, CAP integration, comprehensive TS conversion Test results: - Total cases: 34 → 46 (+12, +35%) - Passing: 45/46 (97.8%) - Negative case coverage: 10/46 (21.7%, target 15-20% exceeded) Edge case coverage improved: - Mixed framework context (React → UI5 migration) - Ambiguous technology prompts (OData types) - Version-specific queries (1.90.0 features) - Complex multi-technology scenarios (UI5 + CAP) Accuracy maintained above 97% target. Note: 1 pre-existing test case failure (XML views $parameters test) remains from previous test suite - not introduced by new cases.
1 parent 5db5565 commit cff7793

1 file changed

Lines changed: 73 additions & 1 deletion

File tree

plugins/ui5-guidelines/test/fixtures/trigger-cases.json

Lines changed: 73 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,6 +177,78 @@
177177
"expected_skill": null,
178178
"should_trigger": false,
179179
"reason": "Angular, not UI5"
180+
},
181+
{
182+
"prompt": "Create a REST API with FastAPI",
183+
"expected_skill": null,
184+
"should_trigger": false,
185+
"reason": "Python FastAPI, not UI5"
186+
},
187+
{
188+
"prompt": "SwiftUI data binding patterns",
189+
"expected_skill": null,
190+
"should_trigger": false,
191+
"reason": "iOS SwiftUI, not UI5"
192+
},
193+
{
194+
"prompt": "Kotlin coroutines best practices",
195+
"expected_skill": null,
196+
"should_trigger": false,
197+
"reason": "Kotlin, not UI5"
198+
},
199+
{
200+
"prompt": "Next.js server components tutorial",
201+
"expected_skill": null,
202+
"should_trigger": false,
203+
"reason": "Next.js/React, not UI5"
204+
},
205+
{
206+
"prompt": "Laravel Eloquent ORM queries",
207+
"expected_skill": null,
208+
"should_trigger": false,
209+
"reason": "PHP Laravel, not UI5"
210+
},
211+
{
212+
"prompt": "Rust async/await patterns",
213+
"expected_skill": null,
214+
"should_trigger": false,
215+
"reason": "Rust, not UI5"
216+
},
217+
{
218+
"prompt": "Go channels and goroutines",
219+
"expected_skill": null,
220+
"should_trigger": false,
221+
"reason": "Go, not UI5"
222+
},
223+
{
224+
"prompt": "What's the best way to structure UI5 controls when migrating from React?",
225+
"expected_skill": "ui5-best-practices",
226+
"should_trigger": true,
227+
"reason": "UI5 is the target, React mentioned contextually"
228+
},
229+
{
230+
"prompt": "How do I use OData types in my UI5 application?",
231+
"expected_skill": "ui5-best-practices",
232+
"should_trigger": true,
233+
"reason": "OData types most relevant in UI5 context"
234+
},
235+
{
236+
"prompt": "What's new in UI5 version 1.90.0?",
237+
"expected_skill": "ui5-best-practices",
238+
"should_trigger": true,
239+
"reason": "Version detection and IAsyncContentCreation (1.90.0)"
240+
},
241+
{
242+
"prompt": "How to set up UI5 application with CAP OData backend integration?",
243+
"expected_skill": "ui5-best-practices",
244+
"should_trigger": true,
245+
"reason": "UI5 + CAP integration patterns"
246+
},
247+
{
248+
"prompt": "Convert UI5 app from JavaScript to TypeScript with tests",
249+
"expected_skill": "ui5-typescript-expert",
250+
"should_trigger": true,
251+
"reason": "Comprehensive TypeScript conversion including tests"
180252
}
181253
]
182-
}
254+
}

0 commit comments

Comments
 (0)