-
Notifications
You must be signed in to change notification settings - Fork 140
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
xActiveDirectory: Move MSFT_xADCommon helper functions #320
Conversation
- Move helper function from MSFT_xADCommon to the module xActiveDirectory.Common
Codecov Report
@@ Coverage Diff @@
## dev #320 +/- ##
===================================
- Coverage 92% 91% -2%
===================================
Files 20 19 -1
Lines 2307 2310 +3
Branches 10 10
===================================
- Hits 2143 2115 -28
- Misses 154 185 +31
Partials 10 10 |
are not in desired state
I have no clue why Pester is reporting misses, the missing rows are hit when run locally. 🤔 For example the entire |
Rewrote the unit test for |
So the tests is run, but for some reason some of the rows in the xActiveDirectory.Common does not get hit. 🤔 |
It seems the culprit is the unit tests for xADUser. Narrowed it down so far, not sure yet what in those tests make this happen. |
Added issue #325 to track the code coverage problem. Leaving it for now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 14 of 18 files at r1, 2 of 4 files at r2, 17 of 17 files at r4.
Reviewable status: complete! all files reviewed, all discussions resolved
Pull Request (PR) description
xActiveDirectory.Common (issue xActiveDirectory: Move xADCommon to xActiveDirectory.Common.psm1 #288)
helper function had design flaws too.
This Pull Request (PR) fixes the following issues
Task list
Entry should say what was changed, and how that affects users (if applicable).
and comment-based help.
This change is