Service Principal Auth and Unit test restructure

webtonize commented 11 months ago

Description

This PR enables service principal authentication and restructures pester unit tests for the modules functions

Related Issue

Fix #39
Fix #40

Motivation and Context

Enterprise use requires a service account to be available
Tests were unmanageable in previous state

How Has This Been Tested?

Pester unit tests

Screenshots (if appropriate):

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

github-actions[bot] commented 11 months ago

Test Results

377 tests 377 :heavy_check_mark: 6s :stopwatch: 134 suites     0 :zzz:     1 files     0 :x:

Results for commit 3753185b.

:recycle: This comment has been updated with latest results.

codecov[bot] commented 11 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (e720a65) 99.14% compared to head (3753185) 99.19%. Report is 1 commits behind head on main.

Files	Patch %	Lines
...ules.AzureDevOps/Classes/AzureDevOpsConnection.ps1	98.18%	1 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #58 +/- ## ========================================== + Coverage 99.14% 99.19% +0.04% ========================================== Files 8 9 +1 Lines 351 494 +143 ========================================== + Hits 348 490 +142 - Misses 3 4 +1 ``` | [Flag](https://app.codecov.io/gh/cloudyspells/PSRule.Rules.AzureDevOps/pull/58/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cloudyspells) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/cloudyspells/PSRule.Rules.AzureDevOps/pull/58/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cloudyspells) | `99.19% <99.48%> (+0.04%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cloudyspells#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.