Skip to content

Add tags and skip condition to test cases#1434

Open
brianreidc7 wants to merge 1 commit intomaester365:mainfrom
brianreidc7:patch-5
Open

Add tags and skip condition to test cases#1434
brianreidc7 wants to merge 1 commit intomaester365:mainfrom
brianreidc7:patch-5

Conversation

@brianreidc7
Copy link
Contributor

Skip condition on the Describe line causes the tags to be ignored. So when running the tests and this specific test is skipped because it is "LongRunning" but this tag is not shown in the results, so it is not clear why the test was skipped.

Description

Steps: Invoke-Maester, wait for results, filter by "Not Tested" and then find this test - it shows it was "Not Tested" (becuase LongRunning was not specified in Invoke-... but the Tags shown in the result do not show "LongRunning" so it is unclear why the test was skipped. Move the -Skip to the It line and copy the tags to the It line and you get a better result in Maester results page.

Contribution Checklist

Before submitting this PR, please confirm you have completed the following:

  • 📖 Read the guidelines for contributing to this repository.
  • 🧪 Ensure the build and unit tests pass by running /powershell/tests/pester.ps1 on your local system.

 

Join us at the Maester repository discussions 💬 or Entra Discord 🧑‍💻 for more help and conversations!

Skip condition on the Describe line causes the tags to be ignored. So when running the tests and this specific test is skipped because it is "LongRunning" but this tag is not shown in the results, so it is not clear why the test was skipped. 
Steps: Invoke-Maester, wait for results, filter by "Not Tested" and then find this test - it shows it was "Not Tested" (becuase LongRunning was not specified in Invoke-... but the Tags shown in the result do not show "LongRunning" so it is unclear why the test was skipped. Move the -Skip to the It line and copy the tags to the It line and you get a better result in Maester results page.
@brianreidc7 brianreidc7 requested a review from a team as a code owner February 9, 2026 18:05
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.

1 participant