-
Notifications
You must be signed in to change notification settings - Fork 309
Adjust test cases for fixed enum case rename #1471
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
newName: "newName(_:)", | ||
expectedPrepareRenamePlaceholder: "myCase", | ||
expectedPrepareRenamePlaceholder: "myCase(label:)", | ||
expected: """ | ||
enum MyEnum { | ||
case newName(_ label: String) |
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.
Should we be dropping the label completely here?
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.
I think it’s valuable to keep it as an internal label, similar to what we do for function completion (and yes, also easier to implement 🙈)
Enum case rename is fixed by changes in sourcekitd. We can remove our workaround for the issue and add test cases that test the rename behavior. Fixes swiftlang#1228 rdar://127646036
bd7e9e3
to
7a61ff9
Compare
@swift-ci Please test |
swiftlang/swift-syntax#2678 @swift-ci Please test Windows |
swiftlang/swift-syntax#2678 @swift-ci Please test macOS |
swiftlang/swift-syntax#2678 @swift-ci Please test Linux |
Linked to swiftlang/swift-syntax#2678 and swiftlang/swift#74241
Enum case rename is fixed by changes in sourcekitd. We can remove our workaround for the issue and add test cases that test the rename behavior.
Fixes #1228
rdar://127646036