style(test): use is_multiple_of instead of modulo comparison

farhan-syah · farhan-syah · commit cc61e5a212d3 · 2026-07-21T19:01:30.000+08:00
diff --git a/nodedb/tests/native_txn_commit_visibility.rs b/nodedb/tests/native_txn_commit_visibility.rs
@@ -86,7 +86,7 @@ fn collection_on_nonzero_core() -> String {
         let vshard =
             VShardId::from_collection_in_database(nodedb::types::DatabaseId::DEFAULT, &name)
                 .as_u32();
-        if vshard as usize % NUM_CORES != 0 {
+        if !(vshard as usize).is_multiple_of(NUM_CORES) {
             return name;
         }
     }

Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ fn collection_on_nonzero_core() -> String {`
`86`	`86`	`let vshard =`
`87`	`87`	`VShardId::from_collection_in_database(nodedb::types::DatabaseId::DEFAULT, &name)`
`88`	`88`	`.as_u32();`
`89`		`- if vshard as usize % NUM_CORES != 0 {`
	`89`	`+ if !(vshard as usize).is_multiple_of(NUM_CORES) {`
`90`	`90`	`return name;`
`91`	`91`	`}`
`92`	`92`	`}`