Skip to content

Verify semantic symbols in using expression tests - #85847

Draft
jjonescz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-useless-tests-in-usingstatements
Draft

jjonescz with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-useless-tests-in-usingstatements

Conversation

Copilot AI commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Two tests discarded a lazy symbol-verification iterator, so their assertions never ran. The helper also applies only to using declarations, while these tests cover using expressions.

  • Semantic assertions: Verify that the identifier expression binds to its declared local and that the object-creation expression resolves to MyManagedType.
  • Diagnostics: Verify clean compilation after removing unused using System; directives.
Assert.Equal(declaredSymbol, model.GetSymbolInfo(usingStatement.Expression).Symbol);

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: jjonescz <3669664+jjonescz@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix useless tests that do not verify anything in UsingStatementTests Verify semantic symbols in using expression tests Sep 30, 2026
Copilot AI requested a review from jjonescz September 30, 2026 16:26

This branch has not been deployed

No deployments
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.

Two tests are useless because they do not actually verify anything

2 participants