Skip to content

Conversation

@jimSampica
Copy link
Contributor

@jimSampica jimSampica commented Dec 31, 2025

added FindByTestId to bunit.web.query based on discussion from #1359

@jimSampica jimSampica changed the title feat: added FindByTestId added FindByTestId Dec 31, 2025
@jimSampica
Copy link
Contributor Author

@dotnet-policy-service agree

linkdotnet
linkdotnet previously approved these changes Dec 31, 2025
Copy link
Collaborator

@linkdotnet linkdotnet left a comment

Choose a reason for hiding this comment

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

Looking good! Small nitpick from my side.
Also, can you add an entry under: UNRELEASED
Basically, add a new ### ADDED section where you add the new functionality.

Something along:

## UNRELEASED
+
+ ### ADDED
+ - `FindByTestId` to `bunit.web.query` to gather elements by a given test id. By [@jimSampica](https://github.com/jimSampica)

@jimSampica
Copy link
Contributor Author

Looking good! Small nitpick from my side. Also, can you add an entry under: UNRELEASED Basically, add a new ### ADDED section where you add the new functionality.

Something along:

## UNRELEASED
+
+ ### ADDED
+ - `FindByTestId` to `bunit.web.query` to gather elements by a given test id. By [@jimSampica](https://github.com/jimSampica)

Should be addressed now ☺️

@linkdotnet
Copy link
Collaborator

Looking good! So let's get that in as a new years celebration!

@linkdotnet linkdotnet merged commit e686527 into bUnit-dev:main Jan 1, 2026
1 check passed
@jimSampica jimSampica deleted the by-testid branch January 2, 2026 16:07
@scottsauber
Copy link
Collaborator

Yay @jimSampica!

linkdotnet added a commit that referenced this pull request Jan 8, 2026
* Set version to '2.5-preview'

* feat: Added FindByTestId (#1802)

* fix: Add OverloadResolutionAttribute for Render method in .NET 9.0+ (#1801)

* Set version to '2.5'

---------

Co-authored-by: Jim Sampica <jtsampica@gmail.com>
Co-authored-by: Steven Giesel <stgiesel35@gmail.com>
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.

3 participants