💥 Move setResponse from response to after.response middleware + fix missing unit test methods

[aswetlow]

Proposed changes

Fixes

• Fix request/responses methods in Conversation Actions that are used in unit tests.
• Fix object copy in jovo-plugin-inbox from shallow to deep

Deprecation

• Remove jovo-analytics-voicehero after service shutdown (issue Drop VoiceHero integration package #899)

Breaking change

Currently, platform responses are called in the response middleware. Other plugins - like any analytics integration - hook into the response middleware to post data after the setResponse of the host (Lambda, etc) has been called. This works because the request is not completed until all processes have run through. This won't work for Lambda with enabled callbackWaitsForEmptyEventLoop. Tasks in the response middleware pipe could be canceled before they are completed.

With this change setResponse is called in after.response. This would complete every task in response before the payload is returned.

Depending on depth and type of customization this could break plugins that hook into the response middleware.

Quick fix for these cases would be changing the middleware from response to after.response

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• My code follows the code style of this project
• My change requires a change to the documentation
• I have updated the documentation accordingly
• I have added tests to cover my changes
• All new and existing tests passed