Skip to content

Update deps + parameterized test support#234

Open
mattdurak wants to merge 2 commits intomasterfrom
parameterized_tests
Open

Update deps + parameterized test support#234
mattdurak wants to merge 2 commits intomasterfrom
parameterized_tests

Conversation

@mattdurak
Copy link
Contributor

Updates c-testrunnerswitcher submodule to master, bringing in PARAMETERIZED_TEST_FUNCTION macro support.

No test refactoring: existing tests use unique COM types and mock expectations per test, not suitable for parameterization.

- Update deps/c-testrunnerswitcher to master (adds PARAMETERIZED_TEST_FUNCTION support)
- No test refactoring: existing tests use unique COM types and mock expectations per test, not suitable for parameterization

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Contributor

@microsoft-github-policy-service microsoft-github-policy-service bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR contains deps update, engage rubber stamping!

@mattdurak
Copy link
Contributor Author

Parameterized Test Review — com-wrapper PR #234

✅ No test changes — submodule bump only

This PR only updates the c-testrunnerswitcher submodule (a62e70648b0066). There are no .c file changes — no tests converted to parameterized form, no TEST_FUNCTIONs deleted, no CASEs added.

Checklist result: All items N/A. No issues found.


Automated review by Copilot CLI

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copy link
Contributor

@microsoft-github-policy-service microsoft-github-policy-service bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR contains deps update, engage rubber stamping!

@mattdurak mattdurak marked this pull request as ready for review March 13, 2026 21:59
@mattdurak
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

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