Skip to content
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

Workaround for GetDynamicParameters() bug #295

Merged
merged 1 commit into from
Mar 20, 2015
Merged

Conversation

dlwyatt
Copy link
Member

@dlwyatt dlwyatt commented Mar 20, 2015

Weird ActiveDirectory module is weird.

Fixes #294. There are no automated tests to verify this yet; we either need to run the test on a 2008 R2 system with the RSAT feature installed, or write some C# code to reproduce the behavior of the AD module. (The latter would be preferable, but will take some more time.)

Weird ActiveDirectory module is weird.

Fixes pester#294.  There are no automated tests to verify this yet; we either need to run the test on a 2008 R2 system with the RSAT feature installed, or write some C# code to reproduce the behavior of the AD module.  (The latter would be preferable, but will take some more time.)
dlwyatt added a commit that referenced this pull request Mar 20, 2015
Workaround for GetDynamicParameters() bug
@dlwyatt dlwyatt merged commit b3122ed into pester:master Mar 20, 2015
@dlwyatt dlwyatt deleted the MockFix branch March 20, 2015 02:10
@ephracis
Copy link

Is this included in the 3.3.6 release? I downloaded it a few days ago, and did a remove-module + import-module but #294 is still present.

@nohwnd
Copy link
Member

nohwnd commented Apr 15, 2015

@ephracis It is not. It's still unreleased as described in the changelog.

Unreleased

@dlwyatt
Copy link
Member Author

dlwyatt commented Apr 15, 2015

We may as well do another minor release now. Don't want to leave things sitting around for months like last time. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cannot Mock Get-ADGroup
3 participants