Merge pull request #268 from North-Two-Five/feature/throws-over-32kb

throws error when message is over 32kb

Author: pooyaj

index.js

@@ -62,15 +62,7 @@ class Analytics {
   }
 
   _validate (message, type) {
-    try {
-      looselyValidate(message, type)
-    } catch (e) {
-      if (e.message === 'Your message must be < 32kb.') {
-        console.log('Your message must be < 32kb. This is currently surfaced as a warning to allow clients to update. Versions released after August 1, 2018 will throw an error instead. Please update your code before then.', message)
-        return
-      }
-      throw e
-    }
+    looselyValidate(message, type)
   }
 
   /**

test.js

@@ -561,7 +561,7 @@ test('isErrorRetryable', t => {
   t.false(client._isErrorRetryable({ response: { status: 200 } }))
 })
 
-test('allows messages > 32kb', t => {
+test('dont allow messages > 32kb', t => {
   const client = createClient()
 
   const event = {
@@ -573,7 +573,7 @@ test('allows messages > 32kb', t => {
     event.properties[i] = 'a'
   }
 
-  t.notThrows(() => {
+  t.throws(() => {
     client.track(event, noop)
   })
 })