You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jd...@apache.org on 2010/10/19 23:48:35 UTC

svn commit: r1024425 - /hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java

Author: jdcryans
Date: Tue Oct 19 21:48:35 2010
New Revision: 1024425

URL: http://svn.apache.org/viewvc?rev=1024425&view=rev
Log:
HBASE-2156 finally fixing the test

Modified:
    hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java

Modified: hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java
URL: http://svn.apache.org/viewvc/hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java?rev=1024425&r1=1024424&r2=1024425&view=diff
==============================================================================
--- hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java (original)
+++ hbase/trunk/src/test/java/org/apache/hadoop/hbase/client/TestFromClientSide.java Tue Oct 19 21:48:35 2010
@@ -3686,10 +3686,11 @@ public class TestFromClientSide {
     assertTrue(Bytes.equals(result.getValue(HConstants.CATALOG_FAMILY, null), one));
   }
 
-    /**
+  /**
    * For HBASE-2156
    * @throws Exception
    */
+  @Test
   public void testScanVariableReuse() throws Exception {
     Scan scan = new Scan();
     scan.addFamily(FAMILY);
@@ -3700,7 +3701,8 @@ public class TestFromClientSide {
     scan = new Scan();
     scan.addFamily(FAMILY);
 
-    assertTrue(scan.getFamilyMap().get(FAMILY).size() == 0);
+    assertTrue(scan.getFamilyMap().get(FAMILY) == null);
+    assertTrue(scan.getFamilyMap().containsKey(FAMILY));
   }
 
   /**