Copilot commented on code in PR #8263:
URL: https://github.com/apache/hbase/pull/8263#discussion_r3279429661
##########
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestHRegionInfo.java:
##########
@@ -127,7 +120,7 @@ public void testIsOverlap() {
}
/**
- * Tests {@link RegionInfo#isOverlap(RegionInfo[])}
+ * Tests {@link RegionInfo#isOverlap(RegionInfo)}
*/
@Test
public void testIsOverlaps() {
Review Comment:
The `@Test` method testIsOverlaps currently has no assertions or other
verification logic (it just builds a few RegionInfo instances and returns), so
it will always pass and provides no coverage. Consider adding overlap
assertions (or remove/disable the test) so it actually validates
RegionInfo#isOverlap behavior.
--
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]
