feature #37840 [VarDumper] Support PHPUnit --colors option (ogizanagi)

fabpot · fabpot · commit 8b3df379debd · 2020-08-16T09:44:17.000+02:00
This PR was merged into the 5.2-dev branch. Discussion ---------- [VarDumper] Support PHPUnit --colors option | Q | A | ------------- | --- | Branch? | master  | Bug fix? | no | New feature? | yes  | Deprecations? | no  | Tickets | N/A  | License | MIT | Doc PR | N/A Commits ------- ac0a3fc [VarDumper] Support PHPUnit --colors option
diff --git a/src/Symfony/Component/VarDumper/CHANGELOG.md b/src/Symfony/Component/VarDumper/CHANGELOG.md
@@ -4,6 +4,7 @@ CHANGELOG
 5.2.0
 -----
 
+ * added support for PHPUnit `--colors` option
  * added `VAR_DUMPER_FORMAT=server` env var value support
  * prevent replacing the handler when the `VAR_DUMPER_FORMAT` env var is set
 
diff --git a/src/Symfony/Component/VarDumper/Dumper/CliDumper.php b/src/Symfony/Component/VarDumper/Dumper/CliDumper.php
@@ -522,12 +522,14 @@ protected function supportsColors()
                         case '--color=yes':
                         case '--color=force':
                         case '--color=always':
+                        case '--colors=always':
                             return static::$defaultColors = true;
 
                         case '--no-ansi':
                         case '--color=no':
                         case '--color=none':
                         case '--color=never':
+                        case '--colors=never':
                             return static::$defaultColors = false;
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -522,12 +522,14 @@ protected function supportsColors()`
`522`	`522`	`case '--color=yes':`
`523`	`523`	`case '--color=force':`
`524`	`524`	`case '--color=always':`
	`525`	`+ case '--colors=always':`
`525`	`526`	`return static::$defaultColors = true;`
`526`	`527`
`527`	`528`	`case '--no-ansi':`
`528`	`529`	`case '--color=no':`
`529`	`530`	`case '--color=none':`
`530`	`531`	`case '--color=never':`
	`532`	`+ case '--colors=never':`
`531`	`533`	`return static::$defaultColors = false;`
`532`	`534`	`}`
`533`	`535`	`}`