Added getMapType and getListType fixes

Added type checking for valueType parameter

Modified error messages

Ran npm lint

Author: DomPeliniAerospike

lib/exp_lists.js

@@ -26,6 +26,10 @@ const _rtype = _valueExp(exp.ops.VAL_RTYPE, 'intVal')
  *********************************************************************************/
 
 function getListType (type, returnType, isMulti) {
+  if (!Object.values(exp.type).includes(type)) {
+    throw new TypeError('invalid value for valueType parameter')
+  }
+
   let expected = type
 
   switch (returnType & ~lists.returnType.INVERTED) {
@@ -45,7 +49,7 @@ function getListType (type, returnType, isMulti) {
       break
     case lists.returnType.NONE:
     default:
-      throw new TypeError('either set the value type as auto or match with return object data type')
+      throw new TypeError('either set the return type as auto or match with return object data type')
   }
 
   if (type === exp.type.AUTO || type === expected) {

lib/exp_maps.js

@@ -11,6 +11,10 @@ const _rtype = _valueExp(exp.ops.VAL_RTYPE, 'intVal')
  *********************************************************************************/
 
 function getMapType (type, returnType, isMulti) {
+  if (!Object.values(exp.type).includes(type)) {
+    throw new TypeError('invalid value for valueType parameter')
+  }
+
   let expected = type
 
   switch (returnType & ~maps.returnType.INVERTED) {
@@ -32,11 +36,11 @@ function getMapType (type, returnType, isMulti) {
     case maps.returnType.KEY_VALUE:
     case maps.returnType.ORDERED_MAP:
     case maps.returnType.UNORDERED_MAP:
-          expected = exp.type.MAP
+      expected = exp.type.MAP
       break
     case maps.returnType.NONE:
     default:
-      throw new TypeError('either set the value type as auto or match with return object data type')
+      throw new TypeError('either set the return type as auto or match with return object data type')
   }
 
   if (type === exp.type.AUTO || type === expected) {