Skip to content

Adds Test for correct ResourceIds - #18

Open
ChristophHannappel wants to merge 4 commits into
raandree:mainfrom
ChristophHannappel:tests/CheckResourceIDs
Open

Adds Test for correct ResourceIds#18
ChristophHannappel wants to merge 4 commits into
raandree:mainfrom
ChristophHannappel:tests/CheckResourceIDs

Conversation

@ChristophHannappel

Copy link
Copy Markdown
Contributor

Pull Request

Pull Request (PR) description

Adds a Test, of the ResourceIDs of the the Composite DSC Ressources end with a [ResourceName]ResourceName.
This is necessary for the DSCWorkshop Framework to use it at the DependsOn Level and helps avoiding an error where the resource can not find its dependency.

Added

  • Test the compiled Mof File if the RessourceID ends with the ResourceName

Task list

  • The PR represents a single logical change. i.e. Cosmetic updates should go in different PRs.
  • Added an entry under the Unreleased section of in the CHANGELOG.md as per format.
  • Local clean build passes without issue or fail tests (build.ps1 -ResolveDependency).
  • Resource documentation added/updated in README.md.
  • Resource parameter descriptions added/updated in README.md, schema.mof
    and comment-based help.
  • Comment-based help added/updated.
  • Localization strings added/updated in all localization files as appropriate.
  • Examples appropriately added/updated.
  • Unit tests added/updated. See DSC Resource Testing Guidelines.
  • Integration tests added/updated (where possible). See DSC Resource Testing Guidelines.
  • New/changed code adheres to DSC Resource Style Guidelines and Best Practices.

@ChristophHannappel
ChristophHannappel marked this pull request as ready for review March 10, 2026 18:51
@raandree

Copy link
Copy Markdown
Owner

Hi @ChristophHannappel, I am sorry for causing some extra efforts. I am in the process to archiving DscConfig.Demo and using CommonTasks in the DscWorkshop. Would you mind moving your changes to the other repo? I can do as well if you don't have the time for it.

@ChristophHannappel

Copy link
Copy Markdown
Contributor Author

CommonTasks

sure thing, i'll do that tomorrow.

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.

2 participants