Skip to content

Enable more tests to run on all 3 runtimes, part 8#10616

Merged
jonathanpeppers merged 24 commits intomainfrom
dev/grendel/test-runtimes-8
Dec 3, 2025
Merged

Enable more tests to run on all 3 runtimes, part 8#10616
jonathanpeppers merged 24 commits intomainfrom
dev/grendel/test-runtimes-8

Conversation

@grendello
Copy link
Contributor

@grendello grendello commented Nov 27, 2025

This pull request refactors the test data generation for ManifestTest by replacing the static array of test cases with a dynamic data provider method. This makes it easier to extend and maintain the test cases, especially as new AndroidRuntime values are added.

Test data generation improvements:

  • Replaced the static DebuggerAttributeCases array with the Get_DebuggerAttributeCases_Data method, which dynamically generates test cases for all values of the AndroidRuntime enum.
  • Added a helper method AddTestData within Get_DebuggerAttributeCases_Data to streamline the creation of each test case entry.

@grendello grendello marked this pull request as ready for review December 1, 2025 13:32
@grendello grendello force-pushed the dev/grendel/test-runtimes-8 branch from df5fb54 to 6b5ca89 Compare December 2, 2025 10:48
Copy link
Member

@jonathanpeppers jonathanpeppers left a comment

Choose a reason for hiding this comment

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

This looks good, I reran the failing lanes.

@grendello grendello force-pushed the dev/grendel/test-runtimes-8 branch from 6b5ca89 to 2901a2d Compare December 3, 2025 10:10
@jonathanpeppers jonathanpeppers merged commit 1c27ade into main Dec 3, 2025
57 of 59 checks passed
@jonathanpeppers jonathanpeppers deleted the dev/grendel/test-runtimes-8 branch December 3, 2025 19:39
@github-actions github-actions bot locked and limited conversation to collaborators Jan 3, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants