korlov42 commented on code in PR #3590:
URL: https://github.com/apache/ignite-3/pull/3590#discussion_r1567403653
##########
modules/catalog/src/test/java/org/apache/ignite/internal/catalog/commands/DropZoneCommandValidationTest.java:
##########
@@ -47,15 +46,6 @@ void zoneNameMustNotBeNullOrBlank(String zone) {
);
}
- @Test
- void rejectToDropDefaultZone() {
Review Comment:
does it make sense to put test on validation into class with `Validation` in
the name?
I don't mind to have complex scenarios which may cover simple scenarios as
well in CatalogManagerSelfTest, but I would prefer to have comprehensive
validation test coverage in *ValidationTest class
--
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]
