yyqdbngt opened a new pull request, #3796: URL: https://github.com/apache/rocketmq-dashboard/pull/3796
### Motivation The existing `AiEngineLocaleTest` pins locale-independent engine resolution but nothing else. The `normalizeEngine` defaults and the `isReady` gating rules on `LlmConfigVO` were untested. ### Changes - `normalizeEngineShouldDefaultToHttpForMissingOrBlankEngine`: a missing or whitespace-only engine falls back to the HTTP engine constant. - `normalizeEngineShouldTrimAndLowercaseEngine`: mixed-case engines with surrounding whitespace normalise to the canonical lower-case form. - `isReadyShouldReturnFalseWhenDisabledOrMissingModel`: a disabled config or one without a model is never ready. - `isReadyShouldRequireEndpointAndKeyForHttpProviders`: an `ollama` HTTP provider is ready with just a base URL, while other HTTP providers additionally require an API key. ### Verification ``` mvn -B test -Dtest=AiEngineLocaleTest [INFO] Tests run: 5, Failures: 0, Errors: 0, Skipped: 0 [INFO] BUILD SUCCESS ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
