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

functional tests: implement 'errors' tests for Kotlin #1689

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

pwrobeldev
Copy link
Contributor

@pwrobeldev pwrobeldev commented Mar 7, 2025

This change introduces functional tests for 'errors' test suite for Kotlin.
The logic was inspired by the functional test suite for Java to ensure the
same level of support in Kotlin.

It also adds two missing test cases related to errors handling to functional
tests of methods overloads for Kotlin.

Note: this change enabled also compilation of 'Blob' LIME input because
certain types were required for errors test too.

This change introduces functional tests for 'errors' test suite for Kotlin.
The logic was inspired by the functional test suite for Java to ensure the
same level of support in Kotlin.

Note: this change enabled also compilation of 'Blob' LIME input
because certain types were required for errors test too.

Signed-off-by: Patryk Wrobel <[email protected]>
@pwrobeldev pwrobeldev force-pushed the pwrobeldev/kotlin-functional-tests-errors branch from 116e2a9 to fffd90d Compare March 7, 2025 13:27
@pwrobeldev pwrobeldev requested a review from Hsilgos March 10, 2025 06:08
@@ -91,7 +91,7 @@ feature(MethodOverloading cpp android android-kotlin swift dart SOURCES
input/lime/InheritanceOverloads.lime
)

feature(Blob cpp android swift dart SOURCES
feature(Blob cpp android android-kotlin swift dart SOURCES
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The MR which implements tests for BLOB:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant