Merge pull request #11 from MrMaximus/ISS-10

ISS-10: Do not stringify body if isBase64Encoded is true

Author: aceew

src/index.js

@@ -78,9 +78,11 @@ export default class Alpr {
 
       responseData.statusCode = Number.isInteger(data.statusCode) ? data.statusCode : 200;
       responseData.headers = typeof data.headers === 'object' ? data.headers : {};
-      responseData.body = JSON.stringify(data.body ? data.body : data);
       if (data.isBase64Encoded) {
+        responseData.body = data.body || '';
         responseData.isBase64Encoded = true;
+      } else {
+        responseData.body = JSON.stringify(data.body || data);
       }
 
       return instancedClass.callback(null, responseData);

test/index.js

@@ -204,7 +204,7 @@ test.cb(
     t.plan(3);
     const lambdaCallback = (err, response) => {
       t.is(typeof response, 'object');
-      t.deepEqual(response.body, '{}');
+      t.deepEqual(response.body, {});
       t.true(response.isBase64Encoded);
       t.end();
     };
@@ -221,6 +221,29 @@ test.cb(
   }
 );
 
+test.cb(
+  'Route handler > returns isBase64Encoded on the response object when true in response data',
+  (t) => {
+    t.plan(3);
+    const lambdaCallback = (err, response) => {
+      t.is(typeof response, 'object');
+      t.deepEqual(response.body, '');
+      t.true(response.isBase64Encoded);
+      t.end();
+    };
+
+    const alprParams = Object.assign({}, data);
+    alprParams.callback = lambdaCallback;
+
+    const alprLocal = new Alpr(alprParams);
+    alprLocal.route({
+      method: data.event.httpMethod,
+      path: data.event.resource,
+      handler: (requestData, response) => response({ isBase64Encoded: true }),
+    });
+  }
+);
+
 test.cb(
   'Route handler > returns response object without isBase64Encoded when undefined in response data',
   (t) => {