Add policy evaluation details related classes (#703)

`PolicyViolationErrorInfo` is missing `@JsonIgnoreProperties(ignoreUnknown = true)` annotation and `ObjectMapper` wasn't configured for ignoring unknown fields.
This caused an exception when Azure returned `evaluationDetails` in it's response, making impossible to handle the error gracefully.

In this commit:
- Configured `ObjectMapper` with `DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES` to avoid issues when a new field is introduced on Azure side
- created `EvaluationDetails` and subsequent classes to parse the response
- added `toString` implementation for related and newly created classes for easier logging
- added unit test to verify the parsing

Author: lacikaaa

azure-client-runtime/src/main/java/com/microsoft/azure/EvaluationDetails.java

@@ -0,0 +1,25 @@
+/**
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License. See License.txt in the project root for
+ * license information.
+ */
+
+package com.microsoft.azure;
+
+import java.util.List;
+
+public class EvaluationDetails {
+
+    private List<ExpressionEvaluationDetails> evaluatedExpressions;
+
+    public List<ExpressionEvaluationDetails> getEvaluatedExpressions() {
+        return evaluatedExpressions;
+    }
+
+    @Override
+    public String toString() {
+        return "EvaluationDetails{"
+                + "evaluatedExpressions=" + evaluatedExpressions
+                + '}';
+    }
+}

azure-client-runtime/src/main/java/com/microsoft/azure/ExpressionEvaluationDetails.java

@@ -0,0 +1,60 @@
+/**
+ * Copyright (c) Microsoft Corporation. All rights reserved.
+ * Licensed under the MIT License. See License.txt in the project root for
+ * license information.
+ */
+
+package com.microsoft.azure;
+
+import java.util.List;
+
+public class ExpressionEvaluationDetails {
+
+    private String expression;
+
+    private String expressionValue;
+
+    private String operator;
+
+    private String path;
+
+    private String result;
+
+    private List<String> targetValue;
+
+    public String getExpression() {
+        return expression;
+    }
+
+    public String getExpressionValue() {
+        return expressionValue;
+    }
+
+    public String getOperator() {
+        return operator;
+    }
+
+    public String getPath() {
+        return path;
+    }
+
+    public String getResult() {
+        return result;
+    }
+
+    public List<String> getTargetValue() {
+        return targetValue;
+    }
+
+    @Override
+    public String toString() {
+        return "ExpressionEvaluationDetails{"
+                + "expression='" + expression + '\''
+                + ", expressionValue='" + expressionValue + '\''
+                + ", operator='" + operator + '\''
+                + ", path='" + path + '\''
+                + ", result='" + result + '\''
+                + ", targetValue=" + targetValue
+                + '}';
+    }
+}

azure-client-runtime/src/main/java/com/microsoft/azure/PolicyViolation.java

@@ -9,6 +9,7 @@
 import java.io.IOException;
 
 import com.fasterxml.jackson.core.JsonParseException;
+import com.fasterxml.jackson.databind.DeserializationFeature;
 import com.fasterxml.jackson.databind.JsonMappingException;
 import com.fasterxml.jackson.databind.MapperFeature;
 import com.fasterxml.jackson.databind.ObjectMapper;
@@ -22,22 +23,24 @@ public class PolicyViolation extends TypedErrorInfo {
      * Policy violation error details.
      */
     private PolicyViolationErrorInfo policyErrorInfo;
-    
+
     /**
      * Initializes a new instance of PolicyViolation.
-     * @param type the error type
+     *
+     * @param type            the error type
      * @param policyErrorInfo the error details
-     * @throws JsonParseException if the policyErrorInfo has invalid content.
-     * @throws JsonMappingException if the policyErrorInfo's JSON does not match the expected schema. 
-     * @throws IOException if an IO error occurs.
+     * @throws JsonParseException   if the policyErrorInfo has invalid content.
+     * @throws JsonMappingException if the policyErrorInfo's JSON does not match the expected schema.
+     * @throws IOException          if an IO error occurs.
      */
     public PolicyViolation(String type, ObjectNode policyErrorInfo) throws JsonParseException, JsonMappingException, IOException {
         super(type, policyErrorInfo);
-        ObjectMapper objectMapper = new ObjectMapper();
-        objectMapper.configure(MapperFeature.ACCEPT_CASE_INSENSITIVE_PROPERTIES, true);
+        ObjectMapper objectMapper = new ObjectMapper()
+                .configure(MapperFeature.ACCEPT_CASE_INSENSITIVE_PROPERTIES, true)
+                .configure(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
         this.policyErrorInfo = objectMapper.readValue(policyErrorInfo.toString(), PolicyViolationErrorInfo.class);
     }
-    
+
     /**
      * @return the policy violation error details.
      */

azure-client-runtime/src/main/java/com/microsoft/azure/PolicyViolationErrorInfo.java

@@ -78,7 +78,9 @@ public class PolicyViolationErrorInfo {
      * The policy set definition display name.
      */
     private String policySetDefinitionDisplayName;
-    
+
+    private EvaluationDetails evaluationDetails;
+
     /**
      * @return the policy definition id.
      */
@@ -170,17 +172,48 @@ public String getPolicySetDefinitionDisplayName() {
         return policySetDefinitionDisplayName;
     }
 
+    public EvaluationDetails getEvaluationDetails() {
+        return evaluationDetails;
+    }
+
+    @Override
+    public String toString() {
+        return "PolicyViolationErrorInfo{"
+                + "policyDefinitionId='" + policyDefinitionId + '\''
+                + ", policySetDefinitionId='" + policySetDefinitionId + '\''
+                + ", policyDefinitionReferenceId='" + policyDefinitionReferenceId + '\''
+                + ", policySetDefinitionName='" + policySetDefinitionName + '\''
+                + ", policyDefinitionName='" + policyDefinitionName + '\''
+                + ", policyDefinitionEffect='" + policyDefinitionEffect + '\''
+                + ", policyAssignmentId='" + policyAssignmentId + '\''
+                + ", policyAssignmentName='" + policyAssignmentName + '\''
+                + ", policyAssignmentDisplayName='" + policyAssignmentDisplayName + '\''
+                + ", policyAssignmentScope='" + policyAssignmentScope + '\''
+                + ", policyAssignmentParameters=" + policyAssignmentParameters
+                + ", policyDefinitionDisplayName='" + policyDefinitionDisplayName + '\''
+                + ", policySetDefinitionDisplayName='" + policySetDefinitionDisplayName + '\''
+                + ", evaluationDetails=" + evaluationDetails
+                + '}';
+    }
+
     /**
      * An instance of this class provides policy parameter value.
      */
     public static class PolicyParameter {
         private JsonNode value;
-        
+
         /**
          * @return the parameter value.
          */
         public JsonNode getValue() {
             return value;
         }
+
+        @Override
+        public String toString() {
+            return "PolicyParameter{"
+                    + "value=" + value
+                    + '}';
+        }
     }
 }