Untergang1 opened a new issue, #11402:
URL: https://github.com/apache/rocketmq/issues/11402

   ### Before Creating the Enhancement Request
   
   - [x] I have confirmed that this should be classified as an enhancement 
rather than a bug/feature.
   
   
   ### Summary
   
   TopicSysFlag and SubscriptionSysFlag currently lack direct unit tests, while 
other helpers in the same package, such as PullSysFlag, already have dedicated 
tests.
   
   ### Motivation
   
   These classes are used across RocketMQ components, including broker, 
namesrv, and tools. Direct unit tests would help catch regressions in bit 
manipulation and make future refactoring safer.
   
   ### Describe the Solution You'd Like
   
   Add TopicSysFlagTest and SubscriptionSysFlagTest under 
common/src/test/java/org/apache/rocketmq/common/sysflag/.
   
   The tests should cover all buildSysFlag combinations, set/has/clear 
operations, flag independence, preservation of unrelated bits, and idempotency.
   
   No production code changes are needed.
   
   ### Describe Alternatives You've Considered
   
   Existing broker or namesrv tests could cover these flags indirectly, but 
dedicated unit tests would be simpler, faster, and more focused.
   
   ### Additional Context
   
   Related to #8262 


-- 
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