Changes to how shell in terminal is identified

[DonJayamanne]

For #5743

• Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR)
• Title summarizes what is changing
• Has a news entry file (remember to thank yourself!)
• Appropriate comments and documentation strings in the code
• [n/a] Has sufficient logging.
• Has telemetry for enhancements.
• Unit tests & system/integration tests are added/updated
• [n/a] Test plan is updated as appropriate
• [n/a] package-lock.json has been regenerated by running npm install (if dependencies have changed)
• [n/a] The wiki is updated with any design decisions/details.

[DonJayamanne]

@karrtikr Merged this PR, as we need this ASAP. and the issue was only with missing test.
I've added the test, I am assuming you are ok with me merging the PR once the tests have been added.

[Tyriar]

@DonJayamanne thanks for fixing so quickly 😃

[DonJayamanne]

@Tyriar
Team effort.