From 295014c9d604a97ffb5b19de9e3cc7f30dc38842 Mon Sep 17 00:00:00 2001
From: Olivier ALLAIN <olivier.allain@synolia.com>
Date: Sat, 9 Jul 2022 22:23:26 +0200
Subject: [PATCH] Add color on phpcsfixer

---
 src/Task/PhpCsFixer.php           |  3 ++-
 test/Unit/Task/PhpCsFixerTest.php | 39 ++++++++++++++++++++-----------
 2 files changed, 28 insertions(+), 14 deletions(-)

diff --git a/src/Task/PhpCsFixer.php b/src/Task/PhpCsFixer.php
index dd9c1c0aa..a223f866b 100644
--- a/src/Task/PhpCsFixer.php
+++ b/src/Task/PhpCsFixer.php
@@ -71,11 +71,12 @@ public function run(ContextInterface $context): TaskResultInterface
         $this->formatter->resetCounter();
 
         $arguments = $this->processBuilder->createArgumentsForCommand('php-cs-fixer');
-        $arguments->add('--format=json');
+        $arguments->add('--format=txt');
         $arguments->add('--dry-run');
         $arguments->addOptionalBooleanArgument('--allow-risky=%s', $config['allow_risky'], 'yes', 'no');
         $arguments->addOptionalArgument('--cache-file=%s', $config['cache_file']);
         $arguments->addOptionalArgument('--config=%s', $config['config']);
+        $arguments->addOptionalArgument('--ansi', true);
 
         if ($rules = $config['rules']) {
             $arguments->add(sprintf(
diff --git a/test/Unit/Task/PhpCsFixerTest.php b/test/Unit/Task/PhpCsFixerTest.php
index bcf86cffb..209a699e4 100644
--- a/test/Unit/Task/PhpCsFixerTest.php
+++ b/test/Unit/Task/PhpCsFixerTest.php
@@ -115,8 +115,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--verbose',
                 'fix',
             ]
@@ -128,9 +129,10 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
                 '--allow-risky=yes',
+                '--ansi',
                 '--verbose',
                 'fix',
             ]
@@ -142,9 +144,10 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
                 '--cache-file=cachefile',
+                '--ansi',
                 '--verbose',
                 'fix',
             ]
@@ -156,9 +159,10 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
                 '--config=config.php',
+                '--ansi',
                 '--verbose',
                 'fix',
             ]
@@ -170,8 +174,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--rules=foo,bar',
                 '--verbose',
                 'fix',
@@ -188,8 +193,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--rules='.json_encode($rules),
                 '--verbose',
                 'fix',
@@ -202,8 +208,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--using-cache=yes',
                 '--verbose',
                 'fix',
@@ -216,8 +223,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 'fix',
             ]
         ];
@@ -228,8 +236,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--verbose',
                 '--diff',
                 'fix',
@@ -242,8 +251,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--verbose',
                 'fix',
             ]
@@ -255,8 +265,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(RunContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--verbose',
                 'fix',
                 'hello.php',
@@ -270,8 +281,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(GitPreCommitContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--verbose',
                 'fix',
                 'hello.php',
@@ -285,8 +297,9 @@ public function provideExternalTaskRuns(): iterable
             $this->mockContext(GitPreCommitContext::class, ['hello.php', 'hello2.php']),
             'php-cs-fixer',
             [
-                '--format=json',
+                '--format=txt',
                 '--dry-run',
+                '--ansi',
                 '--path-mode=intersection',
                 '--verbose',
                 'fix',