Merge pull request #159 from dblock/fix-157

Fix #157: SessionEndedRequest hangs without a defined sessionEndedFunc.

CHANGELOG.md

@@ -12,6 +12,7 @@
 * [#119](https://github.com/alexa-js/alexa-app/pull/119): Moved to the [alexa-js organization](https://github.com/alexa-js) - [@dblock](https://github.com/dblock).
 * [#118](https://github.com/matt-kruse/alexa-app/pull/118), [#117](https://github.com/matt-kruse/alexa-app/issues/117): Prevent updating session attributes directly - [@ajcrites](https://github.com/ajcrites).
 * [#133](https://github.com/matt-kruse/alexa-app/pull/133), [#71](https://github.com/matt-kruse/alexa-app/issues/71): Support asynchronous patterns in request handlers - [@ajcrites](https://github.com/ajcrites).
+* [#159](https://github.com/alexa-js/alexa-app/pull/159), [#157](https://github.com/alexa-js/alexa-app/issues/157): Fixed `SessionEndedRequest` hangs without a defined `sessionEndedFunc` - [@dblock](https://github.com/dblock).
 * Your contribution here.
 
 ### 2.4.0 (January 5, 2017)

index.js

@@ -451,6 +451,8 @@ alexa.app = function(name) {
               } else if (false !== sessionEndedResult) {
                 callbackHandler();
               }
+            } else {
+              response.send();
             }
           } else if (request.isAudioPlayer()) {
             var event = requestType.slice(12);

test/test_alexa_app_session_ended_request.js

@@ -37,6 +37,19 @@ describe("Alexa", function() {
             version: "1.0"
           });
         });
+
+        it("returns a response without a defined sessionEnded function", function() {
+          var subject = testApp.request(mockRequest);
+
+          return expect(subject).to.eventually.become({
+            response: {
+              directives: [],
+              shouldEndSession: true
+            },
+            sessionAttributes: {},
+            version: "1.0"
+          });
+        });
       });
     });
   });