Avoid Number.isNaN

src/style-spec/util/get_type.js

@@ -1,8 +1,6 @@
 
 export default function getType(val) {
-    if (Number.isNaN(val)) {
-        return 'NaN';
-    } else if (val instanceof Number) {
+    if (val instanceof Number) {
         return 'number';
     } else if (val instanceof String) {
         return 'string';

src/style-spec/validate/validate_number.js

@@ -6,7 +6,12 @@ export default function validateNumber(options) {
     const key = options.key;
     const value = options.value;
     const valueSpec = options.valueSpec;
-    const type = getType(value);
+    let type = getType(value);
+
+    // eslint-disable-next-line no-self-compare
+    if (type === 'number' && value !== value) {
+        type = 'NaN';
+    }
 
     if (type !== 'number') {
         return [new ValidationError(key, value, `number expected, ${type} found`)];

test/unit/style-spec/fixture/numbers.input.json

@@ -0,0 +1,94 @@
+{
+  "version": 8,
+  "sources": {
+    "point": {
+      "type": "geojson",
+      "data": {
+        "type": "FeatureCollection",
+        "features": [
+          {
+            "type": "Feature",
+            "geometry": {
+              "type": "Point",
+              "coordinates": [ 0, 0 ]
+            }
+          }
+        ]
+      }
+    }
+  },
+  "layers": [
+    {
+      "id": "valid",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": 5
+      }
+    },
+    {
+      "id": "zero",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": 0
+      }
+    },
+    {
+      "id": "less-than-zero",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": -1
+      }
+    },
+    {
+      "id": "null-not-number",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": null
+      }
+    },
+    {
+      "id": "object-not-number",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": {}
+      }
+    },
+    {
+      "id": "array-not-number",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": []
+      }
+    },
+    {
+      "id": "boolean-not-number",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": true
+      }
+    },
+    {
+      "id": "expression",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": ["sqrt", 16]
+      }
+    },
+    {
+      "id": "expression-invalid",
+      "type": "circle",
+      "source": "point",
+      "paint": {
+        "circle-radius": ["/", 0, 0]
+      }
+    }
+  ]
+}

test/unit/style-spec/fixture/numbers.output.json

@@ -0,0 +1,21 @@
+[
+    {
+        "message": "layers[2].paint.circle-radius: -1 is less than the minimum value 0",
+        "line": 42
+    },
+    {
+        "message": "layers[3].paint.circle-radius: number expected, null found"
+    },
+    {
+        "message": "layers[4].paint.circle-radius: missing required property \"stops\"",
+        "line": 58
+    },
+    {
+        "message": "layers[5].paint.circle-radius: number expected, array found",
+        "line": 66
+    },
+    {
+        "message": "layers[6].paint.circle-radius: number expected, boolean found",
+        "line": 74
+    }
+]