ui: ACL Policies. Catch all server errors (#5836)

Always show any server errors under Rules, not just invalid HCL

ui-v2/app/components/policy-selector.js

@@ -4,6 +4,7 @@ import { inject as service } from '@ember/service';
 import updateArrayObject from 'consul-ui/utils/update-array-object';
 
 const ERROR_PARSE_RULES = 'Failed to parse ACL rules';
+const ERROR_INVALID_POLICY = 'Invalid service policy';
 const ERROR_NAME_EXISTS = 'Invalid Policy: A Policy with Name';
 
 export default ChildSelectorComponent.extend({
@@ -38,10 +39,11 @@ export default ChildSelectorComponent.extend({
     const err = e.error;
     if (typeof err.errors !== 'undefined') {
       const error = err.errors[0];
-      let prop;
+      let prop = 'Rules';
       let message = error.detail;
       switch (true) {
         case message.indexOf(ERROR_PARSE_RULES) === 0:
+        case message.indexOf(ERROR_INVALID_POLICY) === 0:
           prop = 'Rules';
           message = error.detail;
           break;

ui-v2/app/templates/components/policy-form.hbs

@@ -28,7 +28,7 @@
           <strong>{{item.error.Name.validation}}</strong>
         {{/if}}
     </label>
-    <label class="type-text">
+    <label class="type-text" data-test-rules>
         <span>Rules <a href="{{env 'CONSUL_DOCUMENTATION_URL'}}/guides/acl.html#rule-specification" rel="help noopener noreferrer" target="_blank">(HCL Format)</a></span>
 {{#if (eq item.template '') }}
         {{code-editor syntax='hcl' class=(if item.error.Rules 'error') name=(concat name '[Rules]') value=item.Rules onkeyup=(action 'change' (concat name '[Rules]'))}}

ui-v2/tests/acceptance/dc/acls/policies/as-many/add-new.feature

@@ -67,6 +67,33 @@ Feature: dc / acls / policies / as many / add new: Add new policy
     | token     |
     | role      |
     -------------
+  Scenario: Adding a new policy as a child of [Model] and getting an error
+    Given the url "/v1/acl/policy" responds with from yaml
+    ---
+    status: 500
+    body: |
+      Invalid service policy: acl.ServicePolicy{Name:"service", Policy:"", Sentinel:acl.Sentinel{Code:"", EnforcementLevel:""}, Intentions:""}
+    ---
+    Then I fill in the policies.form with yaml
+    ---
+      Name: New-Policy
+      Description: New Policy Description
+      Rules: key {}
+    ---
+    And I click submit on the policies.form
+    Then the last PUT request was made to "/v1/acl/policy?dc=datacenter" with the body from yaml
+    ---
+      Name: New-Policy
+      Description: New Policy Description
+      Rules: key {}
+    ---
+    And I see error on the policies.form.rules like 'Invalid service policy: acl.ServicePolicy{Name:"service", Policy:"", Sentinel:acl.Sentinel{Code:"", EnforcementLevel:""}, Intentions:""}'
+  Where:
+    -------------
+    | Model     |
+    | token     |
+    | role      |
+    -------------
 @ignore:
   Scenario: Click the cancel form
     Then ok

ui-v2/tests/pages.js

@@ -48,7 +48,7 @@ const cancelable = createCancelable(clickable, is);
 
 const tokenList = tokenListFactory(clickable, attribute, collection, deletable);
 
-const policyForm = policyFormFactory(submitable, cancelable, radiogroup);
+const policyForm = policyFormFactory(submitable, cancelable, radiogroup, text);
 const policySelector = policySelectorFactory(clickable, deletable, collection, alias, policyForm);
 
 const roleForm = roleFormFactory(submitable, cancelable, policySelector);

ui-v2/tests/pages/components/policy-form.js

@@ -1,11 +1,16 @@
-export default (submitable, cancelable, radiogroup) => () => {
+export default (submitable, cancelable, radiogroup, text) => (
+  scope = '[data-test-policy-form]'
+) => {
   return {
     // this should probably be settable
     resetScope: true,
-    scope: '[data-test-policy-form]',
+    scope: scope,
     prefix: 'policy',
     ...submitable(),
     ...cancelable(),
     ...radiogroup('template', ['', 'service-identity'], 'policy'),
+    rules: {
+      error: text('[data-test-rules] strong'),
+    },
   };
 };

ui-v2/tests/pages/components/policy-selector.js

@@ -5,7 +5,7 @@ export default (clickable, deletable, collection, alias, policyForm) => (
   return {
     scope: scope,
     create: clickable(createSelector),
-    form: policyForm(),
+    form: policyForm('#new-policy-toggle + div'),
     policies: alias('selectedOptions'),
     selectedOptions: collection(
       '[data-test-policies] [data-test-tabular-row]',

ui-v2/tests/steps/assertions/page.js

@@ -96,18 +96,28 @@ export default function(scenario, assert, find, currentPage) {
     .then(['I see $property'], function(property) {
       assert.ok(currentPage()[property], `Expected to see ${property}`);
     })
-    .then(['I see $property on the $component like "$value"'], function(
+    .then(
-      property,
+      [
-      component,
+        'I see $property on the $component like "$value"',
-      value
+        "I see $property on the $component like '$value'",
-    ) {
+      ],
-      const target = currentPage()[component][property];
+      function(property, component, value) {
-      assert.equal(
+        let target;
-        target,
+        try {
-        value,
+          if (typeof component === 'string') {
-        `Expected to see ${property} on ${component} as ${value}, was ${target}`
+            property = `${component}.${property}`;
-      );
+          }
-    })
+          target = find(property);
+        } catch (e) {
+          throw e;
+        }
+        assert.equal(
+          target,
+          value,
+          `Expected to see ${property} on ${component} as ${value}, was ${target}`
+        );
+      }
+    )
     .then(['I see $property like "$value"'], function(property, value) {
       const target = currentPage()[property];
       assert.equal(target, value, `Expected to see ${property} as ${value}, was ${target}`);