Skip to content

test: unskip the should discover an API skipped test - #9249

Open
danieljbruce wants to merge 3 commits into
mainfrom
discover-api-test-unskip
Open

test: unskip the should discover an API skipped test#9249
danieljbruce wants to merge 3 commits into
mainfrom
discover-api-test-unskip

Conversation

@danieljbruce

@danieljbruce danieljbruce commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Description:

Improves our test coverage. Looks like a maybe dependency downstream changed or something and now we can unskip our test as a result because the unit test passes.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request re-enables a previously skipped test case ('should discover an API') in the discovery test suite. I have no feedback to provide as there are no review comments.

@danieljbruce
danieljbruce marked this pull request as ready for review September 4, 2026 18:57
@danieljbruce
danieljbruce requested a review from a team as a code owner September 4, 2026 18:57
@github-actions
github-actions Bot requested a review from bshaffer September 4, 2026 18:57
@danieljbruce
danieljbruce enabled auto-merge (squash) September 4, 2026 18:57
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