Skip to content

Commit

Permalink
[tests] Improve the AttributeTest.FindSupportedOnElementsThatDoNotExi…
Browse files Browse the repository at this point in the history 
…stInThatAssembly cecil test. (dotnet#17367)

* Fix an issue where it would not compute the correct grouping key for each member,
  effectively grouping unrelated members together and coming up with weird and incorrect
  results.
* Make it match failures exactly, which makes it possible to detect (and report,
  which it now does) when a known failure is fixed.
* Ignore any hidden members (EditorBrowsableState.Never), because they're most
  likely mistakes.
* Ignore any members in AppKit and UIKit, because these namespaces have a lot of
  conflicting availability attributes. This is tracked in a separate bug (dotnet#17292).

Co-authored-by: GitHub Actions Autoformatter <[email protected]>
  • Loading branch information
@rolfbjarne
rolfbjarne and GitHub Actions Autoformatter authored Jan 25, 2023
1 parent a7e5034 commit 688fa45
Show file tree
Hide file tree
Showing 2 changed files with 210 additions and 252 deletions.
Loading

0 comments on commit 688fa45

Please sign in to comment.