Skip to content

Conversation

guimafelipe
Copy link
Contributor

@guimafelipe guimafelipe commented Sep 5, 2025

This pull request enables the Access Control tests. The list of fixes for this are:

  • Fixing package cleaning up of the test. This was affecting other tests in the pipeline.
  • RunAs:Class -> RunAs. We want inheritance of this attribute for the test methods.
  • Removed unnecessary try catches.
  • Added .testdef file.
  • Added test entries to bypass list for images without VCLibs.

A microsoft employee must use /azp run to validate using the pipelines below.

WARNING:
Comments made by azure-pipelines bot maybe inaccurate.
Please see pipeline link to verify that the build is being ran.

For status checks on the main branch, please use TransportPackage-Foundation-PR
(https://microsoft.visualstudio.com/ProjectReunion/_build?definitionId=81063&_a=summary)
and run the build against your PR branch with the default parameters.

@guimafelipe guimafelipe marked this pull request as ready for review September 5, 2025 00:28
@lauren-ciha
Copy link
Member

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@guimafelipe
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@guimafelipe guimafelipe requested a review from ssparach September 8, 2025 22:32
@guimafelipe guimafelipe merged commit 6d8c059 into main Sep 9, 2025
34 checks passed
@guimafelipe guimafelipe deleted the user/felipeda/accesscontroltests2 branch September 9, 2025 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants