Skip to content

Pester Test file created, one test fails#1

Closed
SimonWahlin wants to merge 4 commits intodsccommunity:devfrom
SimonWahlin:dev
Closed

Pester Test file created, one test fails#1
SimonWahlin wants to merge 4 commits intodsccommunity:devfrom
SimonWahlin:dev

Conversation

@SimonWahlin
Copy link
Copy Markdown
Contributor

Pester Test file created, one test fails since resource doesnt handle domain flat name correctly.

The Test-TargetResource splits the actual and specified domain names on '.' and compares the arrays. This assumes that the Domain Flat NetBios name is the first part of the FQDN. Since this isn't a requirement the resource should compare given domain name with both domain FQDN and the actual Flat NetBios name instead.

@msftclas
Copy link
Copy Markdown

Hi @SimonWahlin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@vors
Copy link
Copy Markdown
Contributor

vors commented Apr 23, 2015

This is awesome! Would you like to fix it?

@msftclas
Copy link
Copy Markdown

@SimonWahlin, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

@TravisEz13 TravisEz13 self-assigned this May 1, 2015
@TravisEz13 TravisEz13 added the enhancement The issue is an enhancement request. label May 1, 2015
@TravisEz13
Copy link
Copy Markdown
Contributor

This looks great. We need tho tests passing before we can merge the request. If you don't plan on fixing the failure, you can comment out the broken test, open an issue.

@SimonWahlin SimonWahlin closed this May 2, 2015
@SimonWahlin SimonWahlin deleted the dev branch May 2, 2015 20:19
@TravisEz13
Copy link
Copy Markdown
Contributor

FYI, Contribution Guidelines say not to change the version number.

@SimonWahlin
Copy link
Copy Markdown
Contributor Author

I closed this Request and added a new one just with the tests.
Since the later updates to the resource doesnt handle netbios names at all (which is even better) there is no longer need to test for them to be handled correctly.

tysonjhayes pushed a commit that referenced this pull request Jan 5, 2016
Test-TargetResource returning non-boolean value.
PlagueHO pushed a commit that referenced this pull request Sep 7, 2017
xComputer: Added Description Property
PlagueHO added a commit that referenced this pull request Sep 19, 2017
* BREAKING CHANGE- Converted all xScheduledTask Interval/Duration type parameters over to be string format

* Fix parameter mismatch

* Change AppVeyor Environment

* Fixed xScheduledTask Integration tests to run on older OS

* Improve test coverage

* BREAKING CHANGE- Converted all xScheduledTask Interval/Duration type parameters over to be string format

* Fix parameter mismatch

* Change AppVeyor Environment

* Fixed xScheduledTask Integration tests to run on older OS

* Improve test coverage

* Added support for setting repetition duration to Indefinitely.

* BREAKING CHANGE- Converted all xScheduledTask Interval/Duration type parameters over to be string format

Fix parameter mismatch

Change AppVeyor Environment

Fixed xScheduledTask Integration tests to run on older OS

Improve test coverage

BREAKING CHANGE- Converted all xScheduledTask Interval/Duration type parameters over to be string format

Fix parameter mismatch

Change AppVeyor Environment

Fixed xScheduledTask Integration tests to run on older OS

Improve test coverage

* Added support for setting repetition duration to Indefinitely.

* Correct examples

* Fix up tests

* Added more verbose message to try and identify problem in V4

* Fix problems in creating triggers

* Rewriting trigger repetition code

* Another fix for WS2012

* Add some info to help debug test failures

* More changes for testing

* Another attempt at fixing WS2012R2

* One more try

* Final fix up of Scheduled Task

* Final tweaks
PlagueHO pushed a commit that referenced this pull request Apr 5, 2019
Releasing version 6.3.0.0
PlagueHO pushed a commit that referenced this pull request Sep 7, 2019
nickgw added a commit to wayfair-contribs/ComputerManagementDsc that referenced this pull request Nov 17, 2022
Changes needed to pass HQRM tests and initiade the reosurce in a simple unit test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement The issue is an enhancement request.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants