More tests, more styling

index.js

@@ -304,8 +304,8 @@ WebTorrent.prototype.destroy = function (cb) {
  * @param  {Number} rate
  */
 WebTorrent.prototype.throttleDownload = function (rate) {
-  if (!Number(rate)) return
   var self = this
+  if (!Number(rate)) return
   self.downloadThrottleRate = rate
   self.throttleGroups.down = new ThrottleGroup({rate: rate})
   self.torrents.forEach(function (torrent) {

test/throttle.js

@@ -16,3 +16,19 @@ test('client download/upload throttle setting and rate reporting', function (t)
   client.destroy()
   t.end()
 })
+
+test('client download/upload throttle setting bad values', function (t) {
+  var client = new WebTorrent({ dht: false, tracker: false })
+
+  client.on('error', function (err) { t.fail(err) })
+  client.on('warning', function (err) { t.fail(err) })
+
+  client.throttleDownload("I'm a string")
+  client.throttleUpload(NaN)
+
+  t.equal(client.downloadThrottleRate, undefined)
+  t.equal(client.uploadThrottleRate, undefined)
+
+  client.destroy()
+  t.end()
+})