Throw when oid is not an integer (Fix #128) (#129)

Author: kapouer

index.js

@@ -33,6 +33,9 @@ function setTypeParser (oid, format, parseFn) {
     parseFn = format
     format = 'text'
   }
+  if (!Number.isInteger(oid)) {
+    throw new TypeError('oid must be an integer: ' + oid)
+  }
   typeParsers[format][oid] = parseFn
 };
 

test/index.js

@@ -1,6 +1,6 @@
 
 const test = require('tape')
-const getTypeParser = require('../').getTypeParser
+const { getTypeParser, setTypeParser } = require('../')
 const types = require('./types')
 
 test('types', function (t) {
@@ -64,4 +64,14 @@ test('types', function (t) {
     t.equal(correct, 300)
     t.end()
   })
+
+  t.test('setTypeParser should throw when oid is not an integer', function (t) {
+    t.throws(function () {
+      setTypeParser(null, function () {})
+    }, /^TypeError: oid must be an integer/)
+    t.throws(function () {
+      setTypeParser('a', function () {})
+    }, /^TypeError: oid must be an integer/)
+    t.end()
+  })
 })