170 improve unit tests

[sgnn7]

What does this PR do?

• Refactors code to remove dead logic paths
• Adds tests for all new refactored code
• Adds better UX for certificate parsing issues
• Fixes escaping for account name during variable fetching
• Updates the documentation
• Improves RSpec dev experience

What ticket does this PR close?

Connected to #170 Connected to #115

Checklists

Change log

• [] The CHANGELOG has been updated, or
• [x] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

• [x] This PR includes new unit and integration tests to go with the code changes, or
• [ ] The changes in this PR do not require tests

Documentation

• [x] Docs (e.g. READMEs) were updated in this PR, and/or there is a follow-on issue to update docs, or
• [] This PR does not require updating any documentation

[izgeri]

@sgnn7 should these fixes have included change log entries? they seem like benefits to end users:

Adds better UX for certificate parsing issues Fixes escaping for account name during variable fetching

[sgnn7]

Hmm..yeah that's a good point. we can get those two tacked on the next changelog.