HBASE-15975 logic in TestHTableDescriptor#testAddCoprocessorWithSpecStr is wrong (Huaxiang Sun)
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/bccedc60 Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/bccedc60 Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/bccedc60 Branch: refs/heads/branch-1.1 Commit: bccedc6072281760816261735773475fb91ca7b9 Parents: 4418ba2 Author: Matteo Bertozzi <matteo.berto...@cloudera.com> Authored: Wed Jun 8 22:42:07 2016 -0700 Committer: Matteo Bertozzi <matteo.berto...@cloudera.com> Committed: Wed Jun 8 22:47:40 2016 -0700 ---------------------------------------------------------------------- .../hadoop/hbase/TestHTableDescriptor.java | 19 ++++++++++--------- 1 file changed, 10 insertions(+), 9 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/hbase/blob/bccedc60/hbase-client/src/test/java/org/apache/hadoop/hbase/TestHTableDescriptor.java ---------------------------------------------------------------------- diff --git a/hbase-client/src/test/java/org/apache/hadoop/hbase/TestHTableDescriptor.java b/hbase-client/src/test/java/org/apache/hadoop/hbase/TestHTableDescriptor.java index ccb3c68..0b0e188 100644 --- a/hbase-client/src/test/java/org/apache/hadoop/hbase/TestHTableDescriptor.java +++ b/hbase-client/src/test/java/org/apache/hadoop/hbase/TestHTableDescriptor.java @@ -54,35 +54,36 @@ public class TestHTableDescriptor { public void testAddCoprocessorWithSpecStr() throws IOException { HTableDescriptor htd = new HTableDescriptor(TableName.META_TABLE_NAME); String cpName = "a.b.c.d"; - boolean expected = false; try { htd.addCoprocessorWithSpec(cpName); + fail(); } catch (IllegalArgumentException iae) { - expected = true; + // Expected as cpName is invalid } - if (!expected) fail(); + // Try minimal spec. try { htd.addCoprocessorWithSpec("file:///some/path" + "|" + cpName); + fail(); } catch (IllegalArgumentException iae) { - expected = false; + // Expected to be invalid } - if (expected) fail(); + // Try more spec. String spec = "hdfs:///foo.jar|com.foo.FooRegionObserver|1001|arg1=1,arg2=2"; try { htd.addCoprocessorWithSpec(spec); } catch (IllegalArgumentException iae) { - expected = false; + fail(); } - if (expected) fail(); + // Try double add of same coprocessor try { htd.addCoprocessorWithSpec(spec); + fail(); } catch (IOException ioe) { - expected = true; + // Expect that the coprocessor already exists } - if (!expected) fail(); } @Test