API-1 - Logging API codeless block improvement

src/logging/index.js

@@ -97,11 +97,13 @@ export default class Logging {
   }
 
   push(logger, logLevel, message, exception) {
-    if (typeof message !== 'string') {
-      throw new Error('"message" must be a string')
-    }
-
-    this.messages.push({ logger, message, exception, 'log-level': logLevel, timestamp: Date.now() })
+    this.messages.push({
+      logger,
+      message    : convertMessageToString(message),
+      exception,
+      'log-level': logLevel,
+      timestamp  : Date.now()
+    })
 
     this.checkMessagesLen()
   }
@@ -160,5 +162,20 @@ export default class Logging {
 
     this.checkMessagesLimit()
   }
+}
+
+function convertMessageToString(message) {
+  if (typeof message === 'string') {
+    return message
+  }
+
+  if (typeof message === 'undefined') {
+    return 'undefined'
+  }
+
+  if (typeof message === 'function') {
+    return Object.prototype.toString.call(message)
+  }
 
+  return JSON.stringify(message)
 }

test/unit/specs/logging.js

@@ -212,19 +212,17 @@ describe('<Logging>', function() {
     expect(req1.body[4].exception).to.include('fatal exception')
   })
 
-  it('throws an error when message is not string', async () => {
-    const errorMsg = '"message" must be a string'
-
+  it('doesnt throw an error if the message is not a string', async () => {
     async function check(method) {
-      expect(() => logger[method](0)).to.throw(errorMsg)
-      expect(() => logger[method](123)).to.throw(errorMsg)
-      expect(() => logger[method](true)).to.throw(errorMsg)
-      expect(() => logger[method](false)).to.throw(errorMsg)
-      expect(() => logger[method](null)).to.throw(errorMsg)
-      expect(() => logger[method](undefined)).to.throw(errorMsg)
-      expect(() => logger[method](_ => _)).to.throw(errorMsg)
-      expect(() => logger[method]({ bar: 123 })).to.throw(errorMsg)
-      expect(() => logger[method](['foo', 123, true, false, null, undefined, { bar: 123 }])).to.throw(errorMsg)
+      expect(() => logger[method](0)).to.not.throw()
+      expect(() => logger[method](123)).to.not.throw()
+      expect(() => logger[method](true)).to.not.throw()
+      expect(() => logger[method](false)).to.not.throw()
+      expect(() => logger[method](null)).to.not.throw()
+      expect(() => logger[method](undefined)).to.not.throw()
+      expect(() => logger[method](_ => _)).to.not.throw()
+      expect(() => logger[method]({ bar: 123 })).to.not.throw()
+      expect(() => logger[method](['foo', 123, true, false, null, undefined, { bar: 123 }])).to.not.throw()
     }
 
     await check('debug')
@@ -235,6 +233,36 @@ describe('<Logging>', function() {
     await check('trace')
   })
 
+  it('converts non-string values to strings', async () => {
+    const req = prepareMockRequest({})
+
+    logger.debug(0)
+    logger.debug('4')
+    logger.debug(123)
+    logger.debug(true)
+    logger.debug(false)
+    logger.debug(null)
+    logger.debug(undefined)
+    logger.debug(_ => _)
+    logger.debug({ foo: 'bar' })
+    logger.debug(['foo', 123, true, false, null, undefined, { bar: 123 }])
+
+    await Backendless.Logging.flush()
+
+    expect(req.body.map(b => b.message)).to.deep.equal([
+      '0',
+      '4',
+      '123',
+      'true',
+      'false',
+      'null',
+      'undefined',
+      '[object Function]',
+      '{\"foo\":\"bar\"}',
+      '[\"foo\",123,true,false,null,null,{\"bar\":123}]'
+    ])
+  })
+
   it('send messages pool by timer', async () => {
     const req1 = prepareMockRequest()