Merge pull request #69 from volna80/master

JAVAMONEY-93 throw arithmetic exception for Double.NaN. Thanks for the fix!

Author: atsticks

src/main/java/org/javamoney/moneta/spi/ConvertBigDecimal.java

@@ -42,7 +42,7 @@ BigDecimal getDecimal(Number num) {
 		@Override
 		BigDecimal getDecimal(Number num) {
             double d = num.doubleValue();
-            if (d == Double.NaN || d == Double.POSITIVE_INFINITY || d == Double.NEGATIVE_INFINITY) {
+            if (Double.isNaN(d)|| Double.isInfinite(d)) {
                 throw new ArithmeticException("NaN, POSITIVE_INFINITY and NEGATIVE_INFINITY cannot be used as " +
                         "parameters for monetary operations.");
             }

src/test/java/org/javamoney/moneta/FastMoneyTest.java

@@ -1137,4 +1137,18 @@ public void testFromInversed(){
         assertEquals(m, m2);
     }
 
+    @Test(expectedExceptions = ArithmeticException.class)
+    public void testCreatingFromDoubleNan(){
+        FastMoney.of(Double.NaN, "XXX");
+    }
+
+    @Test(expectedExceptions = ArithmeticException.class)
+    public void testCreatingFromDoublePositiveInfinity(){
+        FastMoney.of(Double.POSITIVE_INFINITY, "XXX");
+    }
+
+    @Test(expectedExceptions = ArithmeticException.class)
+    public void testCreatingFromDoubleNegativeInfinity(){
+        FastMoney.of(Double.NEGATIVE_INFINITY, "XXX");
+    }
 }