JAVAMONEY-69: renamed isXXXBound to hasXXXBound.

Author: atsticks

src/main/java/javax/money/convert/ProviderContext.java

@@ -85,7 +85,7 @@ public Set<RateType> getRateTypes() {
 	 * Returns the starting date/time this rate is valid. The result can also be
 	 * {@code null}, since it is possible, that an {@link ExchangeRate} does not
 	 * have starting validity range. This also can be queried by calling
-	 * {@link #isLowerBound()}.
+	 * {@link #hasLowerBound()}.
 	 * <p>
 	 * Basically all date time types that are available on a platform must be
 	 * supported. On SE this includes Date, Calendar and the new 310 types
@@ -105,7 +105,7 @@ public <T> T getValidFrom(Class<T> type) {
 	 * <p>
 	 * This is modelled as {@link Long} instaed of {@code long}, since it is
 	 * possible, that an {@link ExchangeRate} does not have starting validity
-	 * range. This also can be queried by calling {@link #isLowerBound()}.
+	 * range. This also can be queried by calling {@link #hasLowerBound()}.
 	 * 
 	 * @return The UTC timestamp of the rate, defining valid from, or
 	 *         {@code null}, if no starting validity constraint is set.
@@ -118,7 +118,7 @@ public Long getValidFromMillis() {
 	 * Returns the ending date/time this rate is valid. The result can also be
 	 * {@code null}, since it is possible, that an {@link ExchangeRate} does not
 	 * have ending validity range. This also can be queried by calling
-	 * {@link #isUpperBound()}.
+	 * {@link #hasUpperBound()}.
 	 * <p>
 	 * Basically all date time types that are available on a platform must be
 	 * supported. On SE this includes Date, Calendar and the new 310 types
@@ -139,7 +139,7 @@ public <T> T getValidTo(Class<T> type) {
 	 * <p>
 	 * This is modelled as {@link Long} instaed of {@code long}, since it is
 	 * possible, that an {@link ExchangeRate} does not have ending validity
-	 * range. This also can be queried by calling {@link #isUpperBound()}.
+	 * range. This also can be queried by calling {@link #hasUpperBound()}.
 	 * 
 	 * @return the duration of validity in milliseconds, or {@code null} if no
 	 *         ending validity constraint is set.
@@ -175,7 +175,7 @@ public boolean isInScope(long timestamp) {
 	 * @return {@code true} if {@link #getValidFromMillis()} is not {@code null}
 	 *         .
 	 */
-	public boolean isLowerBound() {
+	public boolean hasLowerBound() {
 		return getValidFrom(Long.class) != null;
 	}
 
@@ -185,7 +185,7 @@ public boolean isLowerBound() {
 	 * 
 	 * @return {@code true} if {@link #getValidToMillis()} is not {@code null}.
 	 */
-	public boolean isUpperBound() {
+	public boolean hasUpperBound() {
 		return getValidTo(Long.class) != null;
 	}
 

src/test/java/javax/money/convert/ProviderContextTest.java

@@ -10,7 +10,6 @@
  */
 package javax.money.convert;
 
-import org.junit.Ignore;
 import org.junit.Test;
 
 import java.util.Date;
@@ -97,24 +96,24 @@ public void testIsInScope() throws Exception{
     @Test
     public void testIsLowerBound() throws Exception{
         ProviderContext ctx = new ProviderContext.Builder("myprov").setValidTo(222L).create();
-        assertTrue(ctx.isUpperBound());
-        assertFalse(ctx.isLowerBound());
+        assertTrue(ctx.hasUpperBound());
+        assertFalse(ctx.hasLowerBound());
     }
 
     @Test
     public void testIsUpperBound() throws Exception{
         ProviderContext ctx = new ProviderContext.Builder("myprov").setValidFrom(222L).create();
-        assertFalse(ctx.isUpperBound());
-        assertTrue(ctx.isLowerBound());
+        assertFalse(ctx.hasUpperBound());
+        assertTrue(ctx.hasLowerBound());
     }
 
     @Test
     public void testIsUpperLowerBound() throws Exception{
         ProviderContext ctx = new ProviderContext.Builder("myprov").setValidFrom(222L).setValidTo(230L).create();
-        assertTrue(ctx.isUpperBound());
-        assertTrue(ctx.isLowerBound());
+        assertTrue(ctx.hasUpperBound());
+        assertTrue(ctx.hasLowerBound());
         ctx = new ProviderContext.Builder("myprov").create();
-        assertFalse(ctx.isUpperBound());
-        assertFalse(ctx.isLowerBound());
+        assertFalse(ctx.hasUpperBound());
+        assertFalse(ctx.hasLowerBound());
     }
 }