Add all cookie properties to serialized directive object (#448)

Signed-off-by: jsetton <[email protected]>
openhab · Dec 15, 2021 · 19e12cc · 19e12cc
1 parent e1225ee
commit 19e12cc
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 15 deletions.
diff --git a/lambda/alexa/smarthome/directive.js b/lambda/alexa/smarthome/directive.js
@@ -135,8 +135,9 @@ class AlexaDirective {
       name: this.name,
       ...(this.hasEndpoint && {
         endpointId: this._directive.endpoint.endpointId,
-        capabilities: JSON.parse(
-          (this._directive.endpoint.cookie && this._directive.endpoint.cookie.capabilities) || '[]'
+        cookie: Object.entries(this._directive.endpoint.cookie || {}).reduce(
+          (cookie, [key, value]) => ({ ...cookie, [key]: JSON.parse(value) }),
+          {}
         ),
         payload: this.payload
       }),

diff --git a/lambda/test/alexa/cases/alexa.test.js b/lambda/test/alexa/cases/alexa.test.js
@@ -1266,8 +1266,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'switch1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new RequestError(),
@@ -1294,8 +1293,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'switch1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new StatusCodeError(400),
@@ -1322,8 +1320,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'switch1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new StatusCodeError(401),
@@ -1350,8 +1347,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'switch1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new StatusCodeError(404),
@@ -1378,8 +1374,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'foobar1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new SyntaxError(),
@@ -1406,8 +1401,7 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'foobar1',
-        cookie: {}
+        endpointId: 'foobar1'
       }
     },
     error: new TypeError(),
@@ -1435,7 +1429,8 @@ module.exports = [
         name: 'ReportState'
       },
       endpoint: {
-        endpointId: 'foobar1'
+        endpointId: 'foobar1',
+        cookie: { foobar: '{}' }
       }
     },
     error: new Error(),