-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Drools 1180 Migrate QE tests upstream. #802
Conversation
Looks good to me, +1. |
then | ||
$message1.getType(); // just to test that both messages are not null | ||
$message2.getType(); | ||
end |
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.
Nitpicking here -- some of the DRL files use spaces and some tabs. Would be great to unify that (and not use tabs :))
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.
Fixed.
Looks good, just added few minor comments. |
+1 to merge once the build is green. |
…he#802) This PR cover case when the rdrl file contains abbreviated conditions. It means condition like: p : Person(age != 18 && > 15 && < 25) This PR is reproducer for the issue RHBRMS-2854
Signed-off-by: Ricardo Zanini <zanini@redhat.com>
This is third in a series of PRs that migrate some tests from QE regression test suite upstream (there will be more).
Migrated test classes that inherit from KieBasesTest class in QE regression test suite.