Merge pull request #336 from benvinegar/fix-is-string

Improve isString to cover String objects

Author: mattrobenolt

src/raven.js

@@ -424,7 +424,7 @@ function isFunction(what) {
 }
 
 function isString(what) {
-    return typeof what === 'string';
+    return objectPrototype.toString.call(what) === '[object String]';
 }
 
 function isObject(what) {

test/raven.test.js

@@ -209,6 +209,8 @@ describe('globals', function() {
     describe('isString', function() {
         it('should do as advertised', function() {
             assert.isTrue(isString(''));
+            assert.isTrue(isString(String('')));
+            assert.isTrue(isString(new String('')));
             assert.isFalse(isString({}));
             assert.isFalse(isString(undefined));
             assert.isFalse(isString(function(){}));