accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki" <bhava...@clouderagovt.com>
Subject Re: Review Request 19428: ACCUMULO-2503 add formatter tests
Date Thu, 20 Mar 2014 14:17:44 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19428/#review37887
-----------------------------------------------------------



core/src/main/java/org/apache/accumulo/core/util/format/BinaryFormatter.java
<https://reviews.apache.org/r/19428/#comment69661>

    This is the setter method that needs renaming, right?



core/src/main/java/org/apache/accumulo/core/util/format/FormatterFactory.java
<https://reviews.apache.org/r/19428/#comment69664>

    This would be a great candidate for conversion to a regular class instead of static only,
but that may be out of scope for this effort.



core/src/test/java/org/apache/accumulo/core/util/format/DefaultFormatterTest.java
<https://reviews.apache.org/r/19428/#comment69662>

    Test suggestion: an empty byte array.



core/src/test/java/org/apache/accumulo/core/util/format/DeleterFormatterTest.java
<https://reviews.apache.org/r/19428/#comment69663>

    You don't need to replay a mock if you don't expect any calls on it, but are just using
it as a stub to be passed around or returned.



core/src/test/java/org/apache/accumulo/core/util/format/HexFormatterTest.java
<https://reviews.apache.org/r/19428/#comment69667>

    Test suggestion: empty byte array.



core/src/test/java/org/apache/accumulo/core/util/format/HexFormatterTest.java
<https://reviews.apache.org/r/19428/#comment69666>

    There should also be a half-trip test that ensures the hex conversion is correct, i.e.,
that a 9 byte becomes 9, a 10 byte becomes a, etc.


- Bill Havanki


On March 19, 2014, 7:27 p.m., Mike Drob wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19428/
> -----------------------------------------------------------
> 
> (Updated March 19, 2014, 7:27 p.m.)
> 
> 
> Review request for accumulo.
> 
> 
> Bugs: ACCUMULO-2503
>     https://issues.apache.org/jira/browse/ACCUMULO-2503
> 
> 
> Repository: accumulo
> 
> 
> Description
> -------
> 
> ACCUMULO-2503 add formatter tests
> 
> 
> Diffs
> -----
> 
>   core/src/main/java/org/apache/accumulo/core/util/format/BinaryFormatter.java 5021d66fc9117ff97203f6a46ec7422b9db3a598

>   core/src/main/java/org/apache/accumulo/core/util/format/DefaultFormatter.java ee4a2200a433d2e5a0268066421426af4594d346

>   core/src/main/java/org/apache/accumulo/core/util/format/DeleterFormatter.java 8547f12e4c7f7d2435cf890be37f5fd9279dcb63

>   core/src/main/java/org/apache/accumulo/core/util/format/FormatterFactory.java 5451843b114ff8d605511ce7dd21850c4048e893

>   core/src/main/java/org/apache/accumulo/core/util/format/HexFormatter.java 1def7127a58c73313618c739bdd7f50bb8bc772f

>   core/src/main/java/org/apache/accumulo/core/util/format/ShardedTableDistributionFormatter.java
577167a150b8861de714fdde79d2ba41ed06fb89 
>   core/src/main/java/org/apache/accumulo/core/util/format/StatisticsDisplayFormatter.java
dd9de6ce6b967210322bec6c9b704f46b6031917 
>   core/src/test/java/org/apache/accumulo/core/util/format/DateStringFormatterTest.java
PRE-CREATION 
>   core/src/test/java/org/apache/accumulo/core/util/format/DefaultFormatterTest.java PRE-CREATION

>   core/src/test/java/org/apache/accumulo/core/util/format/DeleterFormatterTest.java PRE-CREATION

>   core/src/test/java/org/apache/accumulo/core/util/format/FormatterFactoryTest.java PRE-CREATION

>   core/src/test/java/org/apache/accumulo/core/util/format/HexFormatterTest.java PRE-CREATION

>   core/src/test/java/org/apache/accumulo/core/util/format/ShardedTableDistributionFormatterTest.java
PRE-CREATION 
>   core/src/test/java/org/apache/accumulo/core/util/format/StatisticsDisplayFormatterTest.java
PRE-CREATION 
>   core/src/test/resources/log4j.properties dfc93bf2f1bf04de85e5dd8fca1b966dc803e067 
> 
> Diff: https://reviews.apache.org/r/19428/diff/
> 
> 
> Testing
> -------
> 
> New and old unit tests pass.
> 
> 
> Thanks,
> 
> Mike Drob
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message