On 10/07/2011 00:23, Mark Proctor wrote:
On 10/07/2011 00:20, Mark Proctor wrote:
On 09/07/2011 23:59, Mikael Lönneberg wrote:
Hi,
In trying to fix JBRULES-2730 and JBRULES-3063, I noticed that
fixing this issue will actually break MiscTest#testRuleReplacement.
So should this test case be rewritten to not allow rule replacement,
or are there circumstances where rule replacements are allowed and
if that is the case what are those circumstances?
Rule replacement is allowed and an important part of a dynamic rule
engine. I believe what we do is disallow it from the same resource
(should double check that) i.e. a drl that repeats the rule twice is
an error. If you load a later resource it will overwrite the previous.
Thinking about this some more. We should probably add a "getWarnings"
as well as getErrors. Rule overwriting would come with a warning.
We might even find a way build warning filters, a composite set of
warnings to fail on, but user defineable.
If you want to work on a separate jira for this?
1) add warnings list to packagebuilder
2) add hasWarnings and getWarnings, in mirror of errors.
3) allow hashWarnings and getWarnings to take positive/negative
warnings. i.e. ignore duplicate resource (hasWarnings would return
false), report duplicate file (is returned in list). This should be some
sort of chainable or compositable filter, with sane defaults.
For us to include this it will need to be well unit tested and ideally
report other types of warnings too... Ideas on other types of warnings
(note this is different to verification and analysis)
Mark
ConsequenceOffsetTest.testConsequenceOffset also fails due to
loading two resource files with the same package and rule name
defined in the different files.
Sounds like a bug, can you update some stronger tests and fix?
Mark
Kind regards
Mikael Lonneberg - gwendo
_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev
_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev
_______________________________________________
rules-dev mailing list
rules-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/rules-dev
