Skip to content
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

Parser tests for sealed properties #73

Merged
merged 4 commits into from
Apr 6, 2023

Conversation

nomisRev
Copy link
Contributor

@nomisRev nomisRev commented Apr 6, 2023

Adds a couple tests for sealed class / interface, and two failing test-cases related to #72 that show the failing parser error when the sealed class/interface have properties that get implemented by a data class, but not in the constructor.

If they're implemented by the constructor it works fine, but I didn't add a test-case for this.

@traceyyoshima traceyyoshima merged commit 74da050 into openrewrite:main Apr 6, 2023
@nomisRev nomisRev deleted the sealed-properties branch April 7, 2023 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants