dotnet / dotnet/machinelearning

Some Roslyn analyzer tests have unexpected compilation errors

Open
#2,588 0 comments 1 reaction 1 assignee Claimed by @vejbomar View on GitHub
area-Infrastructure Priority:3 test-bug
Dominant language
C#
Stars
9.4k
Forks
2k
Avg merge
2d 20h
Merged PRs (30d)
11

Description

In #2562, @sharwell updates the code analyzer tests to utilize some existing convenience classes. This is great, but it also reveals an issue with the existing tests that should be fixed.

The old mechanism that was based on the VS Roslyn templates was restrictive in the diagnostics it "listened" to and reported. But this went even so far as not even reporting actual compilation errors, and some of the test sources have compilation errors of one form or another (missing declarations mostly, it seems like, based on files from `src` that were included in compilation, seems to be a common theme).

For these, it was not intended that there be compilation errors in the code compiled for testing the Roslyn analyzers, so that should be addressed and fixed. (Possibly be inserting dummy declarations into the test input to get it to compile, or some other mechanism.)

See the above PR #2562, for those changes where actual compilation errors are now listed among "expected" errors. The goal of this issue would be to get it so that those new "additions" are no longer necessary.

This FYI is *not* API critical work for v1.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.