Fix sniffer-issues

Author: tvdijen

codecov.yml

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

routing/routes/routes.yaml

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

tests/src/Controller/NegotiateControllerTest.php

@@ -23,18 +23,11 @@
  * @package SimpleSAML\Test
  * @psalm-suppress PropertyNotSetInConstructor
  */
-class NegotiateControllerTest extends TestCase
+final class NegotiateControllerTest extends TestCase
 {
-    /** @var \SimpleSAML\Configuration */
     protected Configuration $config;
-
-    /** @var Logger \SimpleSAML\Logger */
-    protected \SimpleSAML\Logger $logger;
-
-    /** @var \SimpleSAML\Module */
+    protected Logger $logger;
     protected Module $module;
-
-    /** @var \SimpleSAML\Session */
     protected Session $session;