[Cherry-pick #1409] Fix for TranscriptLoggerMiddleware not checking responses type

[stevengum]

Fixes: #1404

Specific Changes

• add test to reproduce issue [Bug, Transcripts] TranscriptLoggerMiddleware should check for responses before checking responses.length #1404
• add test or Transcripts where SendActivitiesHandler is not properly implemented
• add bug fix for [Bug, Transcripts] TranscriptLoggerMiddleware should check for responses before checking responses.length #1404, more tests
• apply PR feedback and refactor code

[coveralls]

Pull Request Test Coverage Report for Build 90565

• 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.003%) to 84.325%

---

Totals
Change from base Build 90307:	0.003%
Covered Lines:	7837
Relevant Lines:	8959

---

💛 - Coveralls

[stevengum]

Closing as the current logic in master was not released in 4.6. Specifically the logic to stamp over the activity.id with the actual resourceresponse.id. In master we will pick up theses tests and revisit the logic in the TranscriptLoggerMiddleware for R7.