Fix not bug

[sbayer55]

Signed-off-by: Steven Bayer smbayer@amazon.com

Description

Update the types the not operator will accept.

unaryNotOperatorExpression
    : unaryOperator parenthesesExpression
    | unaryOperator primary
    ;

Issues Resolved

Previously not operator accepted any conditional statement. The order of operations parsed did not match the order defined in #1005

Check List

• New functionality includes testing.
• New functionality has been documented.
  • New functionality has javadoc added
• Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[chenqi0805]

LGTM. Thanks for the quick fix!

[sshivanii]

Looks good!

[cmanning09]

Hey @sbayer55, interesting find. Is my understand that this fix changes our expression language from writing expressions with the not operator from:

NOT "/foo/bar" == 5

to:

NOT ("/foo/bar" == 5)

NOT { 5 }

NOT "/foo/bar"

NOT someVariableName

I am not sure about that last one. Are we supporting variableNames for 1.3?