PROC-1219: Fix error message in case test fails

Make it clear that Strings 'true' and 'false' are valid

Author: Robert Danci

proctor-common/src/test/java/com/indeed/proctor/common/TestRuleEvaluator.java

@@ -49,10 +49,10 @@ public void testEmptyRule() {
 
     @Test
     public void testLiteralRule() {
-        for (final String rule : new String[] { "${true}", "${TRUE}", "${ TRUE }" }) {
+        for (final String rule : new String[] { "${true}", "${TRUE}", "${ TRUE }", "${'TRUE'}", "${'true'}" }) {
             assertTrue("rule '" + rule + "' should be true", ruleEvaluator.evaluateBooleanRule(rule, emptyMap()));
         }
-        for (final String rule : new String[] { "${false}", "${FALSE}", "${ FALSE }" }) {
+        for (final String rule : new String[] { "${false}", "${FALSE}", "${ FALSE }", "${'FALSE'}", "${'false'}" }) {
             assertFalse("rule '" + rule + "' should be false", ruleEvaluator.evaluateBooleanRule(rule, emptyMap()));
         }
     }
@@ -96,7 +96,7 @@ public void testStandaloneMethodCalls() {
         }
 
         for (String rule : ImmutableList.of("${!context.isValid()}", "${context.isFortyTwo('47')}")) {
-            assertFalse("rule '" + rule + "' should be true for " + context, ruleEvaluator.evaluateBooleanRule(rule, context));
+            assertFalse("rule '" + rule + "' should be false for " + context, ruleEvaluator.evaluateBooleanRule(rule, context));
         }
 
         assertThatThrownBy(() -> ruleEvaluator.evaluateBooleanRule("${context.isNotFortyTwo('42')}", context))