apache / apache/rocketmq-dashboard

Add regression coverage for the deprecated static mock flag in config

Open Beginner friendly
#4,010 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Java
Stars
1.4k
Forks
683
Avg merge
2d 12h
Merged PRs (30d)
62

Description

## Problem
web/src/config.ts exports USE_MOCK for backward compatibility, but the existing test only covers API_BASE_URL. The deprecated flag is not protected by a regression test, so it could be accidentally flipped and reintroduce the legacy mock-data code path.

## Expected behavior
- USE_MOCK remains alse for the current Studio runtime.
- A focused Vitest case fails if the flag is changed.

## Scope
Only add regression coverage for web/src/config.test.ts. No runtime code changes.

## Acceptance criteria
- [ ] A Vitest assertion verifies USE_MOCK === false.
- [ ] The targeted test file passes.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start with web/src/config.ts to confirm the exported USE_MOCK value, then open web/src/config.test.ts and run that targeted Vitest file. Add the focused assertion described in the issue; done means the test verifies USE_MOCK === false and the targeted test passes.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
frontend, testing
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
92/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.