Thanks for jumping on this so quickly.
I suspect that since we are adding a new flow path through the code, we will encounter a coverage issue because that path is not tested.

We will likely need at least one new test to test this path.

Thanks for jumping on this so quickly. I suspect that since we are adding a new flow path through the code, we will encounter a coverage issue because that path is not tested.

We will likely need at least one new test to test this path.

I can add a unit test for this if you like

Unit test was failing as exception thrown in python 3.7 and 3.8 and higher version was different. changed the assertion to use regex and tested locally with both the version. Tests should pass now.

@Gaurang033
I had multiple issues and PRs come together at the same time including this PR.

I took various parts of each of them and have merged them into a single PR.
This PR will be closed as soon as the tests and review are complete on PR #1836

So please don't devote any additional time to this PR.
If you have comments, thoughts, ideas or suggestions for #1836, feel free to add them there. Thanks so much for getting this ball rolling and offering your time and help. It is much appreciated.

Closed by the merger of #1836