-
Notifications
You must be signed in to change notification settings - Fork 10.6k
[cxx-interop] [test] Refactor nonescapable test to use -verify #86106
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
Conversation
|
@swift-ci please smoke test |
|
@susmonteiro @Xazax-hun It might also be a good idea to split up this test. It's quite difficult to read, and a bit slow to run because of the dependency on But I'll leave it up to y'all since you were the ones who actually wrote this test (-: I just need this patch because my ongoing work elsewhere (#86048) apparently changes one of the error messages and I am having a really hard time telling whether the new diagnostic behavior should be expected. |
Xazax-hun
left a comment
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.
Thanks a lot! This was long overdue and I wanted to get to it but always got distracted with something else. This is a huge improvement!
I have mixed feelings about splitting it up though. The more slightly related test files we have the harder to find the test case we are looking for or to figure out where to put new tests.
susmonteiro
left a comment
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.
Nice, thanks John!
egorzhdan
left a comment
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.
Thank you!
|
ah yes, I forgot about Windows.... I just need |
Checking errors with FileCheck is fragile (because of line numbers) and difficult to follow.
bb18f81 to
9ae5348
Compare
|
@swift-ci please smoke test |
|
@swift-ci please smoke test windows platform |
Checking errors with FileCheck is fragile (because of line numbers) and difficult to follow.