-
Notifications
You must be signed in to change notification settings - Fork 237
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
Adding compiler warnings for empty test asserts in alloy service #3286
base: master
Are you sure you want to change the base?
Conversation
Add Compiler Warnings for empty test asserts
@@ -221,6 +221,11 @@ public static Root_meta_legend_service_metamodel_Test processServiceTest(Service | |||
throw new EngineException("Test does not match execution type", serviceTest.sourceInformation, EngineErrorType.COMPILATION); | |||
} | |||
SingleExecutionTest singleExecutionTest = (SingleExecutionTest) serviceTest; | |||
|
|||
if (singleExecutionTest.asserts != null && singleExecutionTest.asserts.isEmpty()) |
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 not this be an or... is null or is empty? Same on the multi-execution check.
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 will make the above change and create the merge request again. Thanks!
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 have done the change.
What type of PR is this?
Bug Fix
What does this PR do / why is it needed ?
Adding compiler warnings for empty test asserts in alloy service