Tidy linter warnings in StubbingIntegrationTest

rowanhill · Dec 30, 2021 · 1d13566 · 1d13566
1 parent 27af5d2
commit 1d13566
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 13 deletions.
diff --git a/src/WireMock/Client/EqualToXmlMatchingStrategy.php b/src/WireMock/Client/EqualToXmlMatchingStrategy.php
@@ -27,7 +27,7 @@ public function __construct(
     }
 
     /**
-     * @param string[] $comparisonTypes...
+     * @param string $comparisonTypes...
      * @return $this
      */
     public function exemptingComparisons($comparisonTypes)

diff --git a/test/WireMock/Integration/StubbingIntegrationTest.php b/test/WireMock/Integration/StubbingIntegrationTest.php
@@ -4,6 +4,7 @@
 
 require_once 'WireMockIntegrationTest.php';
 
+use Exception;
 use WireMock\Client\DateTimeMatchingStrategy;
 use WireMock\Client\WireMock;
 use WireMock\Client\XmlUnitComparisonType;
@@ -672,7 +673,7 @@ public function testStubIdCanBeSet()
     public function testStubIdIsReturnedInResponseHeader()
     {
         // given
-        $stubMapping = self::$_wireMock->stubFor(WireMock::get(WireMock::urlEqualTo('/some/url'))
+        self::$_wireMock->stubFor(WireMock::get(WireMock::urlEqualTo('/some/url'))
             ->withId('76ada7b0-49ae-4229-91c4-396a36f18e09')
             ->willReturn(WireMock::aResponse()));
 
@@ -697,7 +698,7 @@ public function testStubNameCanBeSet()
     public function testStubNameIsReturnedInResponseHeader()
     {
         // given
-        $stubMapping = self::$_wireMock->stubFor(WireMock::get(WireMock::urlEqualTo('/some/url'))
+        self::$_wireMock->stubFor(WireMock::get(WireMock::urlEqualTo('/some/url'))
             ->withName('stub-name')
             ->willReturn(WireMock::aResponse()));
 
@@ -881,7 +882,7 @@ public function testStubsCanBeEdited()
     }
 
     /**
-     * @throws \Exception
+     * @throws Exception
      */
     public function testStubsCanBeImported()
     {
@@ -893,7 +894,6 @@ public function testStubsCanBeImported()
         );
 
         // then
-        /** @var StubMapping[] $mappings */
         $mappings = getMappings();
         $urls = array_map(function(/** @var $m StubMapping */$m) {
             return $m->getRequest()->getUrlMatchingStrategy()->getMatchingValue();
@@ -903,7 +903,7 @@ public function testStubsCanBeImported()
     }
 
     /**
-     * @throws \Exception
+     * @throws Exception
      */
     public function testStubImportOverwritesStubsByDefault()
     {
@@ -919,15 +919,14 @@ public function testStubImportOverwritesStubsByDefault()
         ));
 
         // then
-        /** @var StubMapping[] $mappings */
         $mappings = getMappings();
         assertThat($mappings, is(arrayWithSize(1)));
         $mapping = $mappings[0];
         assertThat($mapping->getRequest()->getUrlMatchingStrategy()->getMatchingValue(), is('/path2'));
     }
 
     /**
-     * @throws \Exception
+     * @throws Exception
      */
     public function testStubImportCanBeSetToIgnoreDuplicates()
     {
@@ -943,15 +942,14 @@ public function testStubImportCanBeSetToIgnoreDuplicates()
         )->ignoreExisting());
 
         // then
-        /** @var StubMapping[] $mappings */
         $mappings = getMappings();
         assertThat($mappings, is(arrayWithSize(1)));
         $mapping = $mappings[0];
         assertThat($mapping->getRequest()->getUrlMatchingStrategy()->getMatchingValue(), is('/path'));
     }
 
     /**
-     * @throws \Exception
+     * @throws Exception
      */
     public function testStubImportLeavesExistingStubsInPlaceByDefault()
     {
@@ -965,7 +963,6 @@ public function testStubImportLeavesExistingStubsInPlaceByDefault()
         );
 
         // then
-        /** @var StubMapping[] $mappings */
         $mappings = getMappings();
         assertThat($mappings, is(arrayWithSize(3)));
         $urls = array_map(function(/** @var $m StubMapping */$m) {
@@ -976,7 +973,7 @@ public function testStubImportLeavesExistingStubsInPlaceByDefault()
     }
 
     /**
-     * @throws \Exception
+     * @throws Exception
      */
     public function testStubImportCanBeSetToReplaceAnyExistingStubs()
     {
@@ -991,7 +988,6 @@ public function testStubImportCanBeSetToReplaceAnyExistingStubs()
         );
 
         // then
-        /** @var StubMapping[] $mappings */
         $mappings = getMappings();
         assertThat($mappings, is(arrayWithSize(2)));
         $urls = array_map(function(/** @var $m StubMapping */$m) {