Cleanup

sebastianbergmann · sebastianbergmann · commit 7122409e1b10 · 2020-05-08T07:11:54.000+02:00
diff --git a/src/Differ.php b/src/Differ.php
@@ -12,9 +12,6 @@
 use SebastianBergmann\Diff\Output\DiffOutputBuilderInterface;
 use SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder;
 
-/**
- * Diff implementation.
- */
 final class Differ
 {
     public const OLD                     = 0;
diff --git a/tests/ChunkTest.php b/tests/ChunkTest.php
@@ -12,9 +12,9 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\Chunk
+ * @covers \SebastianBergmann\Diff\Chunk
  *
- * @uses SebastianBergmann\Diff\Line
+ * @uses \SebastianBergmann\Diff\Line
  */
 final class ChunkTest extends TestCase
 {
diff --git a/tests/DiffTest.php b/tests/DiffTest.php
@@ -12,9 +12,9 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\Diff
+ * @covers \SebastianBergmann\Diff\Diff
  *
- * @uses SebastianBergmann\Diff\Chunk
+ * @uses \SebastianBergmann\Diff\Chunk
  */
 final class DiffTest extends TestCase
 {
diff --git a/tests/DifferTest.php b/tests/DifferTest.php
@@ -12,12 +12,12 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\Differ
- * @covers SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
+ * @covers \SebastianBergmann\Diff\Differ
+ * @covers \SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
  *
- * @uses SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
- * @uses SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
+ * @uses \SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
  */
 final class DifferTest extends TestCase
 {
diff --git a/tests/Exception/ConfigurationExceptionTest.php b/tests/Exception/ConfigurationExceptionTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\ConfigurationException
+ * @covers \SebastianBergmann\Diff\ConfigurationException
  */
 final class ConfigurationExceptionTest extends TestCase
 {
diff --git a/tests/Exception/InvalidArgumentExceptionTest.php b/tests/Exception/InvalidArgumentExceptionTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\InvalidArgumentException
+ * @covers \SebastianBergmann\Diff\InvalidArgumentException
  */
 final class InvalidArgumentExceptionTest extends TestCase
 {
diff --git a/tests/LineTest.php b/tests/LineTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\Line
+ * @covers \SebastianBergmann\Diff\Line
  */
 final class LineTest extends TestCase
 {
diff --git a/tests/MemoryEfficientImplementationTest.php b/tests/MemoryEfficientImplementationTest.php
@@ -10,7 +10,7 @@
 namespace SebastianBergmann\Diff;
 
 /**
- * @covers SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
+ * @covers \SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
  */
 final class MemoryEfficientImplementationTest extends LongestCommonSubsequenceTest
 {
diff --git a/tests/Output/AbstractChunkOutputBuilderTest.php b/tests/Output/AbstractChunkOutputBuilderTest.php
@@ -13,11 +13,11 @@
 use SebastianBergmann\Diff\Differ;
 
 /**
- * @covers SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
  */
 final class AbstractChunkOutputBuilderTest extends TestCase
 {
diff --git a/tests/Output/DiffOnlyOutputBuilderTest.php b/tests/Output/DiffOnlyOutputBuilderTest.php
@@ -13,10 +13,10 @@
 use SebastianBergmann\Diff\Differ;
 
 /**
- * @covers SebastianBergmann\Diff\Output\DiffOnlyOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\DiffOnlyOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
  */
 final class DiffOnlyOutputBuilderTest extends TestCase
 {
diff --git a/tests/Output/Integration/StrictUnifiedDiffOutputBuilderIntegrationTest.php b/tests/Output/Integration/StrictUnifiedDiffOutputBuilderIntegrationTest.php
@@ -16,11 +16,11 @@
 use Symfony\Component\Process\Process;
 
 /**
- * @covers SebastianBergmann\Diff\Output\StrictUnifiedDiffOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\StrictUnifiedDiffOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
- * @uses SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator
  *
  * @requires OS Linux
  */
diff --git a/tests/Output/Integration/UnifiedDiffOutputBuilderIntegrationTest.php b/tests/Output/Integration/UnifiedDiffOutputBuilderIntegrationTest.php
@@ -14,10 +14,10 @@
 use Symfony\Component\Process\Process;
 
 /**
- * @covers SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
  *
  * @requires OS Linux
  */
diff --git a/tests/Output/StrictUnifiedDiffOutputBuilderTest.php b/tests/Output/StrictUnifiedDiffOutputBuilderTest.php
@@ -15,11 +15,11 @@
 use SebastianBergmann\Diff\Utils\UnifiedDiffAssertTrait;
 
 /**
- * @covers SebastianBergmann\Diff\Output\StrictUnifiedDiffOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\StrictUnifiedDiffOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
- * @uses SebastianBergmann\Diff\ConfigurationException
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\ConfigurationException
  */
 final class StrictUnifiedDiffOutputBuilderTest extends TestCase
 {
diff --git a/tests/Output/UnifiedDiffOutputBuilderTest.php b/tests/Output/UnifiedDiffOutputBuilderTest.php
@@ -13,11 +13,11 @@
 use SebastianBergmann\Diff\Differ;
 
 /**
- * @covers SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
+ * @covers \SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder
  *
- * @uses SebastianBergmann\Diff\Differ
- * @uses SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @uses \SebastianBergmann\Diff\Differ
+ * @uses \SebastianBergmann\Diff\Output\AbstractChunkOutputBuilder
+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
  */
 final class UnifiedDiffOutputBuilderTest extends TestCase
 {
@@ -65,13 +65,9 @@ public function headerProvider(): array
     }
 
     /**
-     * @param string $expected
-     * @param string $from
-     * @param string $to
-     *
      * @dataProvider provideDiffWithLineNumbers
      */
-    public function testDiffWithLineNumbers($expected, $from, $to): void
+    public function testDiffWithLineNumbers(string $expected, string $from, string $to): void
     {
         $differ = new Differ(new UnifiedDiffOutputBuilder("--- Original\n+++ New\n", true));
         $this->assertSame($expected, $differ->diff($from, $to));
@@ -88,7 +84,9 @@ public function provideDiffWithLineNumbers(): array
     public function testEmptyDiffProducesEmptyOutput(string $from, string $to): void
     {
         $differ = new Differ(new UnifiedDiffOutputBuilder('', false));
+
         $output = $differ->diff($from, $to);
+
         $this->assertEmpty($output);
     }
 
diff --git a/tests/ParserTest.php b/tests/ParserTest.php
@@ -13,11 +13,11 @@
 use SebastianBergmann\Diff\Utils\FileUtils;
 
 /**
- * @covers SebastianBergmann\Diff\Parser
+ * @covers \SebastianBergmann\Diff\Parser
  *
- * @uses SebastianBergmann\Diff\Chunk
- * @uses SebastianBergmann\Diff\Diff
- * @uses SebastianBergmann\Diff\Line
+ * @uses \SebastianBergmann\Diff\Chunk
+ * @uses \SebastianBergmann\Diff\Diff
+ * @uses \SebastianBergmann\Diff\Line
  */
 final class ParserTest extends TestCase
 {
@@ -145,7 +145,7 @@ public function testParseDiffForMulitpleFiles(): void
     }
 
     /**
-     * @param Diff[] $expected
+     * @psalm-param list<Diff> $expected
      *
      * @dataProvider diffProvider
      */
diff --git a/tests/TimeEfficientImplementationTest.php b/tests/TimeEfficientImplementationTest.php
@@ -10,7 +10,7 @@
 namespace SebastianBergmann\Diff;
 
 /**
- * @covers SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
+ * @covers \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator
  */
 final class TimeEfficientImplementationTest extends LongestCommonSubsequenceTest
 {
diff --git a/tests/Utils/UnifiedDiffAssertTraitTest.php b/tests/Utils/UnifiedDiffAssertTraitTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * @covers SebastianBergmann\Diff\Utils\UnifiedDiffAssertTrait
+ * @covers \SebastianBergmann\Diff\Utils\UnifiedDiffAssertTrait
  */
 final class UnifiedDiffAssertTraitTest extends TestCase
 {

Original file line number	Diff line number	Diff line change
`@@ -12,9 +12,6 @@`
`12`	`12`	`use SebastianBergmann\Diff\Output\DiffOutputBuilderInterface;`
`13`	`13`	`use SebastianBergmann\Diff\Output\UnifiedDiffOutputBuilder;`
`14`	`14`
`15`		`-/**`
`16`		`- * Diff implementation.`
`17`		`- */`
`18`	`15`	`final class Differ`
`19`	`16`	`{`
`20`	`17`	`public const OLD = 0;`
Original file line number	Diff line number	Diff line change
`@@ -12,9 +12,9 @@`
`12`	`12`	`use PHPUnit\Framework\TestCase;`
`13`	`13`
`14`	`14`	`/**`
`15`		`- * @covers SebastianBergmann\Diff\Chunk`
	`15`	`+ * @covers \SebastianBergmann\Diff\Chunk`
`16`	`16`	`*`
`17`		`- * @uses SebastianBergmann\Diff\Line`
	`17`	`+ * @uses \SebastianBergmann\Diff\Line`
`18`	`18`	`*/`
`19`	`19`	`final class ChunkTest extends TestCase`
`20`	`20`	`{`
Original file line number	Diff line number	Diff line change
`@@ -12,7 +12,7 @@`
`12`	`12`	`use PHPUnit\Framework\TestCase;`
`13`	`13`
`14`	`14`	`/**`
`15`		`- * @covers SebastianBergmann\Diff\ConfigurationException`
	`15`	`+ * @covers \SebastianBergmann\Diff\ConfigurationException`
`16`	`16`	`*/`
`17`	`17`	`final class ConfigurationExceptionTest extends TestCase`
`18`	`18`	`{`
Original file line number	Diff line number	Diff line change
`@@ -10,7 +10,7 @@`
`10`	`10`	`namespace SebastianBergmann\Diff;`
`11`	`11`
`12`	`12`	`/**`
`13`		`- * @covers SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator`
	`13`	`+ * @covers \SebastianBergmann\Diff\MemoryEfficientLongestCommonSubsequenceCalculator`
`14`	`14`	`*/`
`15`	`15`	`final class MemoryEfficientImplementationTest extends LongestCommonSubsequenceTest`
`16`	`16`	`{`
Original file line number	Diff line number	Diff line change
`@@ -13,10 +13,10 @@`
`13`	`13`	`use SebastianBergmann\Diff\Differ;`
`14`	`14`
`15`	`15`	`/**`
`16`		`- * @covers SebastianBergmann\Diff\Output\DiffOnlyOutputBuilder`
	`16`	`+ * @covers \SebastianBergmann\Diff\Output\DiffOnlyOutputBuilder`
`17`	`17`	`*`
`18`		`- * @uses SebastianBergmann\Diff\Differ`
`19`		`- * @uses SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator`
	`18`	`+ * @uses \SebastianBergmann\Diff\Differ`
	`19`	`+ * @uses \SebastianBergmann\Diff\TimeEfficientLongestCommonSubsequenceCalculator`
`20`	`20`	`*/`
`21`	`21`	`final class DiffOnlyOutputBuilderTest extends TestCase`
`22`	`22`	`{`