[PR] [ISSUE-3822][test] Deepen aliyunCatalog tests with instance search parameter coverage [rocketmq-dashboard]

["yyqdbngt (via GitHub)" <gi...@apache.org>]

yyqdbngt opened a new pull request, #3822:
URL: https://github.com/apache/rocketmq-dashboard/pull/3822

   ### Motivation
   
   The existing `aliyunCatalog.test.ts` covers region listing and instance listing with credential/region params, but the optional search term of `listAliyunInstances` was not asserted.
   
   ### Changes
   
   - `includes the search term when listing instances`: when a search term is passed, the request carries `{ credentialId, regionId, search }` and the empty result set is returned.
   
   ### Verification
   
   ```
   ./node_modules/.bin/vitest run src/api/aliyunCatalog.test.ts   # 3 passed
   ./node_modules/.bin/tsc --noEmit                               # clean
   ./node_modules/.bin/eslint src/api/aliyunCatalog.test.ts       # clean
   ```


-- 
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: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] [ISSUE-3822][test] Deepen aliyunCatalog tests with instance search parameter coverage [rocketmq-dashboard]

["lizhimins (via GitHub)" <gi...@apache.org>]

lizhimins closed pull request #3822: [ISSUE-3822][test] Deepen aliyunCatalog tests with instance search parameter coverage
URL: https://github.com/apache/rocketmq-dashboard/pull/3822


-- 
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: commits-unsubscribe@rocketmq.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org