Merge pull request #812 from valtlfelipe/fix-parser

Fix parser constructor error when passing payload

packages/inbound-mail-parser/src/parser.d.ts

@@ -6,6 +6,7 @@ declare interface ParseConfig {
 
 declare interface ParseRequest {
   body?: {};
+  payload?: {};
   files?: any[];
 }
 

packages/inbound-mail-parser/src/parser.js

@@ -39,7 +39,7 @@ class Parse {
   constructor(config, request) {
     this.keys = config.keys;
     this.request = request;
-    this.payload = request.body || {};
+    this.payload = request.body || request.payload || {};
     this.files = request.files || [];
   }
 

packages/inbound-mail-parser/src/parser.spec.js

@@ -2,7 +2,7 @@ const Parse = require('./parser');
 
 describe('test_parse', () => {
   describe('test_parse_key_values', () => {
-    it('should return the key values specified in the config from the payload', () => {
+    it('should return the key values specified in the config from the body', () => {
       const config = {
         keys: ['to', 'from'],
       };
@@ -24,6 +24,29 @@ describe('test_parse', () => {
       expect(keyValues).to.be.an('object');
       expect(keyValues).to.deep.equal(expectedValues);
     });
+
+    it('should return the key values specified in the config from the payload', () => {
+      const config = {
+        keys: ['to', 'from'],
+      };
+      const request = {
+        payload: {
+          to: '[email protected]',
+          from: 'Test User <[email protected]>',
+          subject: 'Test Subject',
+        },
+      };
+
+      const parse = new Parse(config, request);
+      const keyValues = parse.keyValues();
+      const expectedValues = {
+        to: '[email protected]',
+        from: 'Test User <[email protected]>',
+      };
+
+      expect(keyValues).to.be.an('object');
+      expect(keyValues).to.deep.equal(expectedValues);
+    });
   });
 
   describe('test_parse_get_raw_email', () => {