unbridled-41 commented on PR #3982: URL: https://github.com/apache/rocketmq-dashboard/pull/3982#issuecomment-5556653036
## Verification evidence - **CI**: this branch head `9c617fd57cd13b0ff0632f69e412452a0ede090d` triggers the upstream `CI` workflow, which reports `startup_failure` (workflow never starts — repository-level failure, not a job failure): 0 check runs. Recent runs on the base branch fail the same way, so no CI signal is available for any PR in this repository; local verification below substitutes for it. - **Red-light output (before implementation)**: `npx vitest run src/api/instance.test.ts src/pages/instance/__tests__/InstancePage.test.tsx` → `Tests 3 failed | 29 passed (32)`; the two page tests failed with `Unable to find an accessible element with the role "button"` (no export control) and the API test failed on the missing `exportInstancesCsv` export. Backend `mvn test -Dtest='InstanceServiceTest,InstanceControllerTest'` failed at testCompile with 5 × `cannot find symbol: method exportInstancesCsv(...)`. - **Module tests (after implementation)**: same two vitest files → `Tests 32 passed (32)`; `mvn test -Dtest='InstanceServiceTest,InstanceControllerTest'` → `Tests run: 98, Failures: 0, Errors: 0` (84 service + 14 controller), `BUILD SUCCESS`. - **Full suites + baseline comparison**: backend `mvn test` → 2039 tests (pristine baseline 2035 + 4 new), 4 failures: `AuthCorsIntegrationTest` ×2 and `AliyunInstanceProviderTest.getGroupProgressShouldMapLagRowsTest` are byte-identical to the pristine baseline; `OpenAiCompatibleLlmGatewayTest.successfulAndFailedStreamsEmitOneTerminalSequence` is the known load-flaky case (passes 9/9 in isolation, verified on the sibling PR's identical full run). Zero new failures. Web `npx vitest run --testTimeout=60000` → 925 tests (922 pristine + 3 new), failures only in `ConsumerPage.test.tsx` (the known load-flaky file untouched by this PR; re-run in isolation: 29/29 passed). Zero new failures. - **Build**: `npm run build` → `✓ built in 16.16s`; `tsc -b` clean; `eslint` on the five touched frontend files clean; backend main+test compilation covered by the surefire run above. - **Diff self-check**: 9 files, +207/−1 — one read-only endpoint, one service export method, one API client function, one service-layer wrapper (with mock-mode fallback), one button + handler, and tests only. No formatting churn, no unrelated edits. -- 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]
