Add more tests and handle corner cases for `st2 login`

[Mierdin]

There was a lot of discussion in #3226 for st2 login regarding refactoring and adding more tests for corner cases. This should be added for the 2.2.1 release

[stale]

Thanks for contributing to this issue. As it has been 90 days since the last activity, we are automatically marking is as stale. If this issue is not relevant or applicable anymore (problem has been fixed in a new version or similar), please close the issue or let us know so we can close it. On the contrary, if the issue is still relevant, there is nothing you need to do, but if you have any additional details or context which would help us when working on this issue, please include it as a comment to this issue.