[incubator-kie-issues#1409]-Fixed matches function behavior

[bncriju]

Fixes apache/incubator-kie-issues#1409

How to retest this PR or trigger a specific build:

• for pull request and downstream checks

  • Push a new commit to the PR. An empty commit would be enough.

• for a full downstream build

  • for github actions job: add the label run_fdb

• for Jenkins PR check only

  • If you are an ASF committer for KIE podling, login to Jenkins (https://ci-builds.apache.org/job/KIE/job/drools/), go to the specific PR job, and click on Build Now button.

[gitgabrio]

Many thanks @bncriju great job!!!

[yesamer]

@bncriju is this import used / correct?

[bncriju]

@yesamer Whenever input/pattern is null, this is the most appropriate one to be used. it seems. This is used to catch this condition and whenever we verify this case in test as well.

[yesamer]

@bncriju My point is: that expection is part of the java.security package. That means it should be used in the security management scope (as described here https://docs.oracle.com/javase/8/docs/api/java/security/InvalidParameterException.html).
In our context, IllegalArgumentException should be used

[yesamer]

@bncriju I run your changes against the TCK tests, and cases 5. and 6. are still not working. Is that intended?