[#51] Add permission to administer Redirect settings

redirect.permissions.yml

@@ -1,2 +1,4 @@
 administer redirects:
   title: 'Administer URL redirections'
+change redirect settings:
+  title: 'Change redirect settings'

redirect.routing.yml

@@ -44,15 +44,15 @@ redirect.settings:
     _form: '\Drupal\redirect\Form\RedirectSettingsForm'
     _title: 'Settings'
   requirements:
-    _permission: 'administer redirects'
+    _permission: 'change redirect settings'
 
 redirect.fix_404:
   path: '/admin/config/search/redirect/404'
   defaults:
     _title: 'Fix 404 pages'
     _form: '\Drupal\redirect\Form\RedirectFix404Form'
   requirements:
-    _permission: 'administer redirects'
+    _permission: 'change redirect settings'
 
 #redirect.devel_generate:
 #  requirements:

src/Tests/RedirectUITest.php

@@ -21,6 +21,11 @@
  */
 class RedirectUITest extends WebTestBase {
 
+  /**
+   * @var array
+   */
+  protected $userPermissions;
+
   /**
    * @var \Drupal\Core\Session\AccountInterface
    */
@@ -48,15 +53,16 @@ protected function setUp() {
     parent::setUp();
 
     $this->drupalCreateContentType(array('type' => 'article', 'name' => 'Article'));
-    $this->adminUser = $this->drupalCreateUser(array(
+    $this->userPermissions = [
       'administer redirects',
       'access site reports',
       'access content',
       'bypass node access',
       'create url aliases',
       'administer taxonomy',
       'administer url aliases',
-    ));
+    ];
+    $this->adminUser = $this->drupalCreateUser($this->userPermissions);
 
     $this->repository = \Drupal::service('redirect.repository');
 
@@ -228,6 +234,18 @@ public function testRedirectUI() {
   public function testFix404Pages() {
     $this->drupalLogin($this->adminUser);
 
+    // Test the permission "Change Redirect settings".
+    $this->drupalGet('admin/config/search/redirect/404');
+    $this->assertResponse(403);
+
+    // Now create and log in a user with the proper permissions.
+    $adminUser = $this->drupalCreateUser(array_merge($this->userPermissions, ['change redirect settings']));
+    $this->drupalLogin($adminUser);
+
+    // Test access again.
+    $this->drupalGet('admin/config/search/redirect/404');
+    $this->assertResponse(200);
+
     // Visit a non existing page to have the 404 watchdog entry.
     $this->drupalGet('non-existing');
 
@@ -251,6 +269,25 @@ public function testFix404Pages() {
     $this->assertUrl('node');
   }
 
+  /**
+   * Tests the Settings page.
+   */
+  public function testSettingsPage() {
+    $this->drupalLogin($this->adminUser);
+
+    // Test the permission "Change Redirect settings".
+    $this->drupalGet('admin/config/search/redirect/settings');
+    $this->assertResponse(403);
+
+    // Now create and log in a user with the proper permissions.
+    $adminUser = $this->drupalCreateUser(array_merge($this->userPermissions, ['change redirect settings']));
+    $this->drupalLogin($adminUser);
+
+    // Test access again.
+    $this->drupalGet('admin/config/search/redirect/settings');
+    $this->assertResponse(200);
+  }
+
   /**
    * Tests redirects being automatically created upon path alias change.
    */
@@ -474,7 +511,8 @@ public function testExternal() {
     $redirect->setStatusCode(301);
     $redirect->save();
     $this->assertRedirect('a-path', 'https://www.example.org');
-    $this->drupalLogin($this->adminUser);
+    $adminUser = $this->drupalCreateUser(array_merge($this->userPermissions, ['change redirect settings']));
+    $this->drupalLogin($adminUser);
     $this->drupalPostForm('admin/config/search/redirect/settings', ['redirect_deslash' => 1], t('Save configuration'));
     $this->drupalGet('/2015/10/10/');
     $this->assertResponse(404);