Merge pull request #1740 from andre-nunes/math

Test missing parameter behavior

Author: pjcozzi

Specs/Core/MathSpec.js

@@ -70,10 +70,22 @@ defineSuite([
         expect(CesiumMath.toRadians(360.0)).toEqual(2 * Math.PI);
     });
 
+    it('toRadians throws for undefined', function() {
+        expect(function() {
+            CesiumMath.toRadians();
+        }).toThrowDeveloperError();
+    });
+
     it('toDegrees', function() {
         expect(CesiumMath.toDegrees(Math.PI)).toEqual(180.0);
     });
 
+    it('toDegrees throws for undefined', function() {
+        expect(function() {
+            CesiumMath.toDegrees();
+        }).toThrowDeveloperError();
+    });
+
     it('convertLongitudeRange (1)', function() {
         expect(CesiumMath.convertLongitudeRange(CesiumMath.THREE_PI_OVER_TWO)).toEqualEpsilon(-CesiumMath.PI_OVER_TWO, CesiumMath.EPSILON16);
     });
@@ -86,6 +98,12 @@ defineSuite([
         expect(CesiumMath.convertLongitudeRange(Math.PI)).toEqualEpsilon(-Math.PI, CesiumMath.EPSILON16);
     });
 
+    it('convertLongitudeRange throws for undefined', function() {
+        expect(function() {
+            CesiumMath.convertLongitudeRange();
+        }).toThrowDeveloperError();
+    });
+
     it('negativePiToPi positive', function() {
         expect(CesiumMath.negativePiToPi((Math.PI / 2) * Math.PI)).toEqualEpsilon((Math.PI / 2) * Math.PI - CesiumMath.TWO_PI, CesiumMath.EPSILON16);
         expect(CesiumMath.negativePiToPi(Math.PI / 0.5)).toEqualEpsilon(0.0, CesiumMath.EPSILON16);
@@ -103,6 +121,36 @@ defineSuite([
         expect(CesiumMath.negativePiToPi(-Math.PI + 1)).toEqualEpsilon(-Math.PI + 1, CesiumMath.EPSILON16);
     });
 
+    it('negativePiToPi throws for undefined', function() {
+        expect(function() {
+            CesiumMath.negativePiToPi();
+        }).toThrowDeveloperError();
+    });
+
+    it('zeroToTwoPi throws for undefined', function() {
+        expect(function() {
+            CesiumMath.zeroToTwoPi();
+        }).toThrowDeveloperError();
+    });
+
+    it('equalsEpsilon throws for undefined left', function() {
+        expect(function() {
+            CesiumMath.equalsEpsilon(undefined, 5.0, CesiumMath.EPSILON16);
+        }).toThrowDeveloperError();
+    });
+
+    it('equalsEpsilon throws for undefined right', function() {
+        expect(function() {
+            CesiumMath.equalsEpsilon(1.0, undefined, CesiumMath.EPSILON16);
+        }).toThrowDeveloperError();
+    });
+
+    it('equalsEpsilon throws for undefined', function() {
+        expect(function() {
+            CesiumMath.equalsEpsilon();
+        }).toThrowDeveloperError();
+    });
+
     it('factorial produces the correct results', function() {
         var factorials = [1, 1, 2, 6, 24, 120, 720, 5040, 40320, 362880, 3628800, 39916800, 479001600, 6227020800, 87178291200, 1307674368000, 20922789888000, 355687428096000, 6402373705728000,
                           121645100408832000, 2432902008176640000, 51090942171709440000, 1124000727777607680000, 25852016738884976640000, 620448401733239439360000];
@@ -118,6 +166,12 @@ defineSuite([
         expect(CesiumMath.incrementWrap(10, 10)).toEqual(0);
     });
 
+    it('incrementWrap throws for undefined', function() {
+        expect(function() {
+            CesiumMath.incrementWrap();
+        }).toThrowDeveloperError();
+    });
+
     it('isPowerOfTwo finds powers of two', function() {
         expect(CesiumMath.isPowerOfTwo(1)).toEqual(true);
         expect(CesiumMath.isPowerOfTwo(2)).toEqual(true);
@@ -205,4 +259,10 @@ defineSuite([
             CesiumMath.nextPowerOfTwo();
         }).toThrowDeveloperError();
     });
+
+    it('clamp throws for undefined', function() {
+        expect(function() {
+            CesiumMath.clamp();
+        }).toThrowDeveloperError();
+    });
 });