Closed
Description
#2523 is adding a test run to check that there are no extra warnings produced by the analyzer. The list of test failures so far is below, along with an initial triage. I created a few new dedicated issues for tracking purposes.
Issues blocking integration to main
- [DAM analyzer] don't produce warnings for unsupported intrinsics #2578
- Support filter regions (tracked by [DAM analyzer] support exceptional control flow #2415)
- [DAM analyzer] Add support for attribute warning suppressions #2579
- [feature/damAnalyzer] Warning code differ between the linker and analyzer #2583
- Test infra issues (fixing in [DAM analyzer] Check for unexpected analyzer warnings in linker tests #2523)
- [DAM analyzer] Don't produce dataflow warnings for annotations on invalid types #2598
Note: this does not include tests which are currently skipped, such as the LinkAtributes tests - I will update the list to include those once I run the skipped tests in the "no extra warnings" mode.
Tests failing in #2523
- ReflectionTests.MembersUsedViaReflection [FAIL]
unsupported intrinsic: GetMember with binding flags - ReflectionTests.TypeUsedViaReflection [FAIL]
RUC on GetType - ReflectionTests.FieldsUsedViaReflection [FAIL]
unsupported intrinsic binding flags - ReflectionTests.ExpressionNewType [FAIL]
RUC on GetType - ReflectionTests.ConstructorsUsedViaReflection [FAIL]
unsupported intrinsic: GetConstructors with binding flags - ReflectionTests.EventsUsedViaReflection [FAIL]
unsupported intrinsic: GetEvents with binding flags - ReflectionTests.ActivatorCreateInstance [FAIL]
RUC on CreateInstance - ReflectionTests.ExpressionCallString [FAIL]
RUC on Expression.Call - ReflectionTests.TypeUsedViaReflectionTypeDoesntExist [FAIL]
RUC on Type.GetType - ReflectionTests.ExpressionPropertyMethodInfo [FAIL]
RUC on Expression.Property - ReflectionTests.AssemblyImportedViaReflectionWithSweptReferences [FAIL]
RUC on Type.GetType - ReflectionTests.MethodsUsedViaReflection [FAIL]
unsupported intrinsic: Type.GetMethods with binding flags - ReflectionTests.TypeHierarchySuppressions [FAIL]
unsupported intrinsic: GetType call on instance, where type has DAM,
and we don't look for DAM on type - ReflectionTests.ConstructorUsedViaReflection [FAIL]
unsupported intrinsic: GetConstructors with binding flags - ReflectionTests.PropertiesUsedViaReflection [FAIL]
unsupported intrinsic: GetProperties with binding flags - ReflectionTests.NestedTypesUsedViaReflection [FAIL]
unsupported intrinsic: GetNestedTypes with binding flags - ReflectionTests.MemberUsedViaReflection [FAIL]
unsupported intrinsic: Type.GetMember with binding flags - ReflectionTests.ObjectGetType [FAIL]
unsupported intrinsic: object.GetType - ReflectionTests.TypeHierarchyReflectionWarnings [FAIL]
unsupported intrinsic: object.GetType - UnreachableBlockTests.TryFilterBlocks [FAIL]
no support for filter regions - LoggingTests.SourceLines [FAIL]
is OK, test infra doesn't support ExpectedWarning with SourceLine - AdvancedTests.TypeCheckRemoval [FAIL]
no support for filter regions - TestFrameworkTests.VerifyResourceInAssemblyAttributesBehavior [FAIL]
test infra problem (needs investigation) - TestFrameworkTests.CanSandboxDependenciesUsingType [FAIL]
test infra problem (needs investigation) - Warnings.WarningSuppressionTests.SuppressWarningsInMembersAndTypesUsingTarget
warning suppressions in code usingTarget = "..."
syntax - Warnings.WarningSuppressionTests.SuppressWarningsInCompilerGeneratedCode
extra warnings in analyzer, needs investigation - Warnings.WarningSuppressionTests.SuppressWarningsViaXml
analyzer needs to support XML attributes
@agocke hopefully this helps track things - let me know if you want me to adjust anything.