OPENEUROPA-1500: Add custom permission for webtools analytics config.

openeuropa · Jan 8, 2019 · b6ab8c1 · b6ab8c1
1 parent cec4035
commit b6ab8c1
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/modules/oe_webtools_analytics/oe_webtools_analytics.permissions.yml b/modules/oe_webtools_analytics/oe_webtools_analytics.permissions.yml
@@ -0,0 +1,3 @@
+administer webtools analytics:
+  title: 'Administer Webtools Analytics'
+  restrict access: false
diff --git a/modules/oe_webtools_analytics/oe_webtools_analytics.routing.yml b/modules/oe_webtools_analytics/oe_webtools_analytics.routing.yml
@@ -4,4 +4,4 @@ oe_webtools_analytics.settings:
     _form: 'Drupal\oe_webtools_analytics\Form\WebtoolsAnalyticsSettingsForm'
     _title: 'Webtools Analytics settings'
   requirements:
-    _permission: 'administer site configuration'
+    _permission: 'administer webtools analytics'
diff --git a/tests/features/analytics.feature b/tests/features/analytics.feature
@@ -10,11 +10,6 @@ Feature: Webtools Analytics
   Scenario: Create Webtools Analytics settings
     Given I am on "admin/config/system/oe_webtools_analytics"
     Then I should see "Webtools Analytics settings"
-    When I fill in "Site ID" with "INFO"
-    And I fill in "Site path" with "ec.europa.eu/info"
-    And I fill in "Instance" with "ec.europa.eu"
-    And I press "Save configuration"
-    Then I should see the message "The value must be numeric."
     When I fill in "Site ID" with "123456"
     And I fill in "Site path" with "ec.europa.eu/info"
     And I fill in "Instance" with "ec.europa.eu"