remove duplicated error checking from Rule class

Rule class has low coverage, because it has much error checking that is really not necessary because the same errors would have been caught earlier by the schema checker.

Except for the top-level evaluateRule() method the schema-related error checking should be reduced to:

///CLOVER:OFF
throw new IllegalStateException();
///CLOVER:ON

after the last else if block in the method.