-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Add some tests from records test plan #46466
Conversation
} | ||
"; | ||
var comp = CreateCompilation(new[] { src, IsExternalInitTypeDefinition }, options: TestOptions.DebugExe, parseOptions: TestOptions.RegularPreview); | ||
comp.VerifyDiagnostics(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.VerifyDiagnostics() [](start = 16, length = 20)
Consider calling VarifyDiagnostics on result of CompileAndVerify. That set will include any warnings that might be reported by emit phase.
} | ||
"; | ||
CreateCompilation(src).VerifyDiagnostics( | ||
// (2,1): error CS0267: The 'partial' modifier can only appear immediately before 'class', 'struct', 'interface', or 'void' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
before 'class', 'struct', 'interface', or 'void' [](start = 91, length = 48)
Is this list adjusted in another PR?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. I'm merged master into this PR and updated the comment now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (iteration 1)
Since this PR contains no product code change (test and updating a resource string), I'll go ahead and merge with a single review once CI is green. Thanks |
Relates to #40726 (test plan for records)