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

   ### Motivation
   
   The existing `ProducerControllerTest` covers the connection payload, the 
topic guard and the missing instance id, but a whitespace-only `instanceId` on 
the connection endpoint and the pure-`instanceId` form of the group-suggestion 
endpoint were untested.
   
   ### Changes
   
   - `listConnectionsShouldRejectBlankInstanceId`: a whitespace-only instance 
id is rejected with 400 `instanceId is required` and no service interaction.
   - `listProducerGroupsShouldAllowOnlyInstanceId`: when `topic`, `query` and 
`limit` are absent the suggestion query still reaches the service with the 
optional filters left null.
   
   ### Verification
   
   ```
   mvn -B test -Dtest=ProducerControllerTest
   [INFO] Tests run: 8, 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]

Reply via email to