Fix sniffer-issues

Author: tvdijen

codecov.yml

@@ -1,3 +1,5 @@
+---
+
 coverage:
   status:
     project:

routing/routes/routes.yaml

@@ -1,29 +1,43 @@
+---
+
 negotiate-auth:
-    path:       /auth
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::auth' }
-    methods:    [GET]
+  path: /auth
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::auth'
+  }
+  methods: [GET]
 
 negotiate-error:
-    path:       /error
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::error' }
-    methods:    [GET]
+  path: /error
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::error'
+  }
+  methods: [GET]
 
 negotiate-enable:
-    path:       /enable
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::enable' }
-    methods:    [GET]
+  path: /enable
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::enable'
+  }
+  methods: [GET]
 
 negotiate-disable:
-    path:       /disable
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::disable' }
-    methods:    [GET]
+  path: /disable
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::disable'
+  }
+  methods: [GET]
 
 negotiate-retry:
-    path:       /retry
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::retry' }
-    methods:    [GET]
+  path: /retry
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::retry'
+  }
+  methods: [GET]
 
 negotiate-fallback:
-    path:       /fallback
-    defaults:   { _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::fallback' }
-    methods:    [GET]
+  path: /fallback
+  defaults: {
+    _controller: 'SimpleSAML\Module\negotiateext\Controller\NegotiateController::fallback'
+  }
+  methods: [GET]

tests/src/Controller/NegotiateControllerTest.php

@@ -18,12 +18,9 @@
  *
  * @package SimpleSAML\Test
  */
-class NegotiateControllerTest extends TestCase
+final class NegotiateControllerTest extends TestCase
 {
-    /** @var \SimpleSAML\Configuration */
     protected Configuration $config;
-
-    /** @var \SimpleSAML\Session */
     protected Session $session;
 
     /**
@@ -125,8 +122,6 @@ public function testErrorInvalidState(): void
     public function testEnable(): void
     {
         $c = new Controller\NegotiateController($this->config, $this->session);
-
-        /** @var \SimpleSAML\XHTML\Template $response */
         $response = $c->enable();
 
         // Validate response
@@ -157,8 +152,6 @@ public function testEnable(): void
     public function testDisable(): void
     {
         $c = new Controller\NegotiateController($this->config, $this->session);
-
-        /** @var \SimpleSAML\XHTML\Template $response */
         $response = $c->disable();
 
         // Validate response