HBASE-21272 Re-add assertions for RS Group admin tests

This commit is contained in:
tedyu 2018-10-05 21:34:46 -07:00
parent ae606c3d7a
commit cbc2876ac2
1 changed files with 5 additions and 1 deletions

View File

@ -141,6 +141,8 @@ public class TestRSGroupsAdmin1 extends TestRSGroupsBase {
String groupName = tablePrefix+"_foo"; String groupName = tablePrefix+"_foo";
LOG.info("testNamespaceConstraint"); LOG.info("testNamespaceConstraint");
rsGroupAdmin.addRSGroup(groupName); rsGroupAdmin.addRSGroup(groupName);
assertTrue(observer.preAddRSGroupCalled);
assertTrue(observer.postAddRSGroupCalled);
admin.createNamespace(NamespaceDescriptor.create(nsName) admin.createNamespace(NamespaceDescriptor.create(nsName)
.addConfiguration(RSGroupInfo.NAMESPACE_DESC_PROP_GROUP, groupName) .addConfiguration(RSGroupInfo.NAMESPACE_DESC_PROP_GROUP, groupName)
.build()); .build());
@ -161,6 +163,8 @@ public class TestRSGroupsAdmin1 extends TestRSGroupsBase {
//test add non-existent group //test add non-existent group
admin.deleteNamespace(nsName); admin.deleteNamespace(nsName);
rsGroupAdmin.removeRSGroup(groupName); rsGroupAdmin.removeRSGroup(groupName);
assertTrue(observer.preRemoveRSGroupCalled);
assertTrue(observer.postRemoveRSGroupCalled);
try { try {
admin.createNamespace(NamespaceDescriptor.create(nsName) admin.createNamespace(NamespaceDescriptor.create(nsName)
.addConfiguration(RSGroupInfo.NAMESPACE_DESC_PROP_GROUP, "foo") .addConfiguration(RSGroupInfo.NAMESPACE_DESC_PROP_GROUP, "foo")
@ -431,4 +435,4 @@ public class TestRSGroupsAdmin1 extends TestRSGroupsBase {
initialize(); initialize();
} }
} }