Update unit tests

[troy0820]

What does this change

Added more unit test to increase coverage around the controllers and api surface.

• Add unit tests to controllers package

What issue does it fix

Partially closes #220
Strategy may be necessary to utilize the integration test suite to e2e the controller(s) flow

Notes for the reviewer

There is a method that is on all the reconcilers called retry which does the same thing but for different resources. Not sure if this makes sense to pull this out and move it to a different package being that it does the same thing on every reconciler but just a different object.

Checklist

• Did you write tests?
• Did you write documentation?
• Did you make any API changes? Update the corresponding API documentation.

[codecov]

Codecov Report

Merging #221 (ba8c3f2) into main (26f9e86) will increase coverage by 3.39%.
The diff coverage is n/a.

❗ Current head ba8c3f2 differs from pull request most recent head 67c4d20. Consider uploading reports for the commit 67c4d20 to get more accurate results

@@            Coverage Diff             @@
##             main     #221      +/-   ##
==========================================
+ Coverage   69.35%   72.75%   +3.39%     
==========================================
  Files          13       13              
  Lines        2160     1556     -604     
==========================================
- Hits         1498     1132     -366     
+ Misses        544      291     -253     
- Partials      118      133      +15     

Flag	Coverage Δ
unit-tests	72.75% <ø> (?)
unittests	?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 13 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[troy0820]

/azp run porter-integration

[azure-pipelines]

No pipelines are associated with this pull request.