yyqdbngt opened a new pull request, #3872: URL: https://github.com/apache/rocketmq-dashboard/pull/3872
### Motivation The `batchDeleteTopics` helper had a single unit test covering only the mixed success/failure path. This PR expands the suite to six cases that pin down the result contract callers (the topic list batch-delete flow) rely on. ### Changes - Empty selection resolves to empty `deleted`/`failed` buckets and never touches the api. - The instance id is forwarded to every underlying `deleteTopic` attempt. - An all-successful selection is reported in the original order. - A rejection carrying a non-Error value (e.g. a plain string) is still treated as a failure and does not abort the remaining attempts. - When the api rejects for every topic, each name lands in `failed` and all attempts are made. ### Verification - `vitest run src/services/topicService.batchDelete.test.ts`: 6/6 passed - `tsc --noEmit`: clean - `eslint src/services/topicService.batchDelete.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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
