You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@rocketmq.apache.org by "yyqdbngt (via GitHub)" <gi...@apache.org> on 2026/09/06 01:26:04 UTC
[PR] test: expand data source test DTO coverage [rocketmq-dashboard]
yyqdbngt opened a new pull request, #3912:
URL: https://github.com/apache/rocketmq-dashboard/pull/3912
### Motivation
The `DataSourceTestDTO` test covered credential redaction in `toString`, but the required-field validation and the fields that are safe to log were unasserted. This PR grows the suite from 1 to 3 cases.
### Changes
- `toString` includes the URL, type, auth mode, and username while never exposing password or bearer token.
- Missing url/type produce their required-field validation messages.
- A basic-auth payload without a bearer token logs the username but never the password.
### Verification
- `mvn -B test -Dtest=DataSourceTestDTOTest`: 3/3 passed, 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: commits-unsubscribe@rocketmq.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] test: expand data source test DTO coverage [rocketmq-dashboard]
Posted by "lizhimins (via GitHub)" <gi...@apache.org>.
lizhimins closed pull request #3912: test: expand data source test DTO coverage
URL: https://github.com/apache/rocketmq-dashboard/pull/3912
--
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