enforce and test valid range of pitch min/max

src/ui/map.js

@@ -102,6 +102,7 @@ type MapOptions = {
 const defaultMinZoom = 0;
 const defaultMaxZoom = 22;
 
+// the default values, but also the valid range
 const defaultMinPitch = 0;
 const defaultMaxPitch = 60;
 
@@ -344,6 +345,14 @@ class Map extends Camera {
             throw new Error(`maxPitch must be greater than or equal to minPitch`);
         }
 
+        if (options.minPitch != null && options.minPitch < defaultMinPitch) {
+            throw new Error(`minPitch must be greater than or equal to ${defaultMinPitch}`);
+        }
+
+        if (options.maxPitch != null && options.maxPitch > defaultMaxPitch) {
+            throw new Error(`maxPitch must be less than or equal to ${defaultMaxPitch}`);
+        }
+
         const transform = new Transform(options.minZoom, options.maxZoom, options.minPitch, options.maxPitch, options.renderWorldCopies);
         super(transform, options);
 
@@ -675,6 +684,10 @@ class Map extends Camera {
 
         minPitch = minPitch === null || minPitch === undefined ? defaultMinPitch : minPitch;
 
+        if (minPitch < defaultMinPitch) {
+            throw new Error(`minPitch must be greater than or equal to ${defaultMinPitch}`);
+        }
+
         if (minPitch >= defaultMinPitch && minPitch <= this.transform.maxPitch) {
             this.transform.minPitch = minPitch;
             this._update();
@@ -706,6 +719,10 @@ class Map extends Camera {
 
         maxPitch = maxPitch === null || maxPitch === undefined ? defaultMaxPitch : maxPitch;
 
+        if (maxPitch > defaultMaxPitch) {
+            throw new Error(`maxPitch must be less than or equal to ${defaultMaxPitch}`);
+        }
+
         if (maxPitch >= this.transform.minPitch) {
             this.transform.maxPitch = maxPitch;
             this._update();

test/unit/ui/map.test.js

@@ -893,6 +893,20 @@ test('Map', (t) => {
         t.end();
     });
 
+    t.test('throw on maxPitch greater than valid maxPitch at init', (t) => {
+        t.throws(() => {
+            createMap(t, {maxPitch: 90});
+        }, new Error(`maxPitch must be less than or equal to 60`));
+        t.end();
+    });
+
+    t.test('throw on minPitch less than valid minPitch at init', (t) => {
+        t.throws(() => {
+            createMap(t, {minPitch: -10});
+        }, new Error(`minPitch must be greater than or equal to 0`));
+        t.end();
+    });
+
     t.test('#remove', (t) => {
         const map = createMap(t);
         t.equal(map.getContainer().childNodes.length, 3);