-
Notifications
You must be signed in to change notification settings - Fork 4.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve test coverage for IsMemberDeclaration method #48163
Labels
Area-IDE
help wanted
The issue is "up for grabs" - add a comment if you are interested in working on it
Test-Gap
Describes a specific feature or scenario that does not have test coverage
Milestone
Comments
After figuring out what this method affects, we need to decide whether to update it with init accessor or not. |
I could get test failures in #48158 by the last commit in it. But still, doesn't cover everything in the method. |
This is in reference to the following method: Lines 1292 to 1334 in 60d89c9
|
Closing out as speculative. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
Area-IDE
help wanted
The issue is "up for grabs" - add a comment if you are interested in working on it
Test-Gap
Describes a specific feature or scenario that does not have test coverage
I don't think it's a good idea to update this one without knowing the change it introduce.
This whole method isn't covered in the tests. See how #48158 is green.
Originally posted by @Youssef1313 in #48137 (comment)
The text was updated successfully, but these errors were encountered: