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 test cases for constructors with SFVC
as a parameter.
#5269
Conversation
a0626d9
to
8c2e044
Compare
compiler/testData/codegen/box/reflection/callBy/inlineClassDefaultArguments.kt
Outdated
Show resolved
Hide resolved
...n/callBy/valueClasses/nonNullObject/defaultArguments/constructorWithInlineClassParameters.kt
Show resolved
Hide resolved
data class TestCtor1_1(val x: A = A("0")) | ||
data class TestCtor1_2(val x: A? = A("0")) |
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 am adding a function to verify the nullability
of a parameter.
Should I add a case where the default argument is null
?
@udalov |
...n/callBy/valueClasses/nonNullObject/defaultArguments/constructorWithInlineClassParameters.kt
Outdated
Show resolved
Hide resolved
.../callBy/valueClasses/nullableObject/defaultArguments/constructorWithInlineClassParameters.kt
Show resolved
Hide resolved
ba8ba1d
to
8880205
Compare
Too much time had passed, so I |
ced6ebc
to
5ccc8d7
Compare
Sorry for delays, merged in 0ed4cab. Thank you! |
Completed missing test cases regarding the type that
value class
wraps and nullability as a parameter.For
SFCV
, these test cases are largely missing.I would like to continue to contribute to the completion of such test cases.
By the way, are there any rules on how to write commit messages?
Also, are there any recommended per-commit guidelines?
I have had these corrected several times and I apologize for that.