Permissive Learning Mode 6/6 Testing #589
Open
lilybarkley-msft wants to merge 2 commits into
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📖 Description
PR 6 of 6 — stacked on PR5. Final PR; restores the testing artifacts that were temporarily held back for readability of PRs 1–5.
src/testing/plm_tester— Windows-only workload binary (plmtester) that deliberately trips the FS, capability, and UI access-denied paths PLM is meant to capture:src/Cargo.toml.tests/configs/plm_configs/— input configs that pair withplmtesterinvocations, one per access category (filesystem promoted / not promoted / readonly / readwrite, capabilities, UI categories).tests/scripts/run_plm_test.ps1— end-to-end driver: runswxc-exec --auditagainst eachplm_configs/*.jsonworkload, captures the resultingAdjusted_*.json, and asserts the expected fields were added (or correctly withheld for the deny-listed cases).After this PR merges, the tree on top of
mainis bit-identical to the pre-split source branch.🔗 References
user/lilybarkley/plm-pr5-ui-policy)🔍 Validation
cargo build -p plm -p plmtester --target x86_64-pc-windows-msvc— cleancargo fmt --all -- --check— cleancargo clippy -p plm -p plmtester --target x86_64-pc-windows-msvc --all-targets -- -D warnings— cleancargo test -p plm --target x86_64-pc-windows-msvc— 126 passedgit diff <pr6-tip> <original-branch-tip>— empty (verified locally; the six commits in order reproduce the original tree exactly).tests/scripts/run_plm_test.ps1end-to-end driver included for manual + CI execution.✅ Checklist
📋 Issue Type
GitHub Actions runs the PR validation build automatically. The ADO pipeline
(
MXC-PR-Build) is the official build pipeline that signs the binaries; itruns on merge to
mainand nightly, and Microsoft reviewers can trigger iton a PR with
/azp run. See docs/pull-requests.md.Microsoft Reviewers: Open in CodeFlow