[TASK] Reduce and finetune the scope of @covers annotations (#1188)

oliverklee · web-flow · commit e89a90cfe6a5 · 2025-03-18T10:41:39.000Z
The legacy tests are not very focused. Until we have split them
up, try to avoid false positives for code coverage.

Also add `@covers` annotations for the parent classes of the
tested classes.
diff --git a/tests/CSSList/AtRuleBlockListTest.php b/tests/CSSList/AtRuleBlockListTest.php
@@ -10,6 +10,8 @@
 
 /**
  * @covers \Sabberworm\CSS\CSSList\AtRuleBlockList
+ * @covers \Sabberworm\CSS\CSSList\CSSBlockList
+ * @covers \Sabberworm\CSS\CSSList\CSSList
  */
 final class AtRuleBlockListTest extends TestCase
 {
diff --git a/tests/Comment/CommentTest.php b/tests/Comment/CommentTest.php
@@ -9,9 +9,7 @@
 use Sabberworm\CSS\Tests\ParserTest as TestsParserTest;
 
 /**
- * @covers \Sabberworm\CSS\Comment\Comment
- * @covers \Sabberworm\CSS\OutputFormat
- * @covers \Sabberworm\CSS\OutputFormatter
+ * @coversNothing
  */
 final class CommentTest extends TestCase
 {
diff --git a/tests/OutputFormatTest.php b/tests/OutputFormatTest.php
@@ -11,7 +11,7 @@
 use Sabberworm\CSS\Parsing\OutputException;
 
 /**
- * @covers \Sabberworm\CSS\OutputFormat
+ * @coversNothing
  */
 final class OutputFormatTest extends TestCase
 {
diff --git a/tests/ParserTest.php b/tests/ParserTest.php
@@ -28,17 +28,7 @@
 use Sabberworm\CSS\Value\ValueList;
 
 /**
- * @covers \Sabberworm\CSS\CSSList\Document
  * @covers \Sabberworm\CSS\Parser
- * @covers \Sabberworm\CSS\RuleSet\DeclarationBlock
- * @covers \Sabberworm\CSS\Rule\Rule
- * @covers \Sabberworm\CSS\Value\CSSString
- * @covers \Sabberworm\CSS\Value\CalcFunction
- * @covers \Sabberworm\CSS\Value\Color
- * @covers \Sabberworm\CSS\Value\LineName
- * @covers \Sabberworm\CSS\Value\Size
- * @covers \Sabberworm\CSS\Value\URL
- * @covers \Sabberworm\CSS\Value\Value
  */
 final class ParserTest extends TestCase
 {
diff --git a/tests/RuleSet/DeclarationBlockTest.php b/tests/RuleSet/DeclarationBlockTest.php
@@ -13,6 +13,7 @@
 
 /**
  * @covers \Sabberworm\CSS\RuleSet\DeclarationBlock
+ * @covers \Sabberworm\CSS\RuleSet\RuleSet
  */
 final class DeclarationBlockTest extends TestCase
 {
diff --git a/tests/RuleSet/LenientParsingTest.php b/tests/RuleSet/LenientParsingTest.php
@@ -11,16 +11,7 @@
 use Sabberworm\CSS\Settings;
 
 /**
- * @covers \Sabberworm\CSS\CSSList\Document
- * @covers \Sabberworm\CSS\Parser
- * @covers \Sabberworm\CSS\RuleSet\DeclarationBlock
- * @covers \Sabberworm\CSS\Rule\Rule
- * @covers \Sabberworm\CSS\Value\CSSString
- * @covers \Sabberworm\CSS\Value\CalcFunction
- * @covers \Sabberworm\CSS\Value\Color
- * @covers \Sabberworm\CSS\Value\LineName
- * @covers \Sabberworm\CSS\Value\Size
- * @covers \Sabberworm\CSS\Value\URL
+ * @coversNothing
  */
 final class LenientParsingTest extends TestCase
 {

Original file line number	Diff line number	Diff line change
`@@ -10,6 +10,8 @@`
`10`	`10`
`11`	`11`	`/**`
`12`	`12`	`* @covers \Sabberworm\CSS\CSSList\AtRuleBlockList`
	`13`	`+ * @covers \Sabberworm\CSS\CSSList\CSSBlockList`
	`14`	`+ * @covers \Sabberworm\CSS\CSSList\CSSList`
`13`	`15`	`*/`
`14`	`16`	`final class AtRuleBlockListTest extends TestCase`
`15`	`17`	`{`
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`use Sabberworm\CSS\Parsing\OutputException;`
`12`	`12`
`13`	`13`	`/**`
`14`		`- * @covers \Sabberworm\CSS\OutputFormat`
	`14`	`+ * @coversNothing`
`15`	`15`	`*/`
`16`	`16`	`final class OutputFormatTest extends TestCase`
`17`	`17`	`{`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,7 @@`
`13`	`13`
`14`	`14`	`/**`
`15`	`15`	`* @covers \Sabberworm\CSS\RuleSet\DeclarationBlock`
	`16`	`+ * @covers \Sabberworm\CSS\RuleSet\RuleSet`
`16`	`17`	`*/`
`17`	`18`	`final class DeclarationBlockTest extends TestCase`
`18`	`19`	`{`