Merge branch '6.0' into 6.1

xabbuh · xabbuh · commit e17f437fb06c · 2022-09-03T16:24:42.000+02:00
* 6.0:
  tweak the factory class description for inlined factory services
  skip a transient test on Windows
  Make bash completion run in non interactive mode
diff --git a/Console/Descriptor/JsonDescriptor.php b/Console/Descriptor/JsonDescriptor.php
@@ -245,7 +245,7 @@ private function getContainerDefinitionData(Definition $definition, bool $omitTa
                 if ($factory[0] instanceof Reference) {
                     $data['factory_service'] = (string) $factory[0];
                 } elseif ($factory[0] instanceof Definition) {
-                    $data['factory_class'] = $factory[0]->getClass() ?? 'not configured';
+                    $data['factory_service'] = sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'class not configured');
                 } else {
                     $data['factory_class'] = $factory[0];
                 }
diff --git a/Console/Descriptor/MarkdownDescriptor.php b/Console/Descriptor/MarkdownDescriptor.php
@@ -224,7 +224,7 @@ protected function describeContainerDefinition(Definition $definition, array $op
                 if ($factory[0] instanceof Reference) {
                     $output .= "\n".'- Factory Service: `'.$factory[0].'`';
                 } elseif ($factory[0] instanceof Definition) {
-                    $output .= "\n".'- Factory Class: `'.($factory[0]->getClass() ?? 'not configured').'`';
+                    $output .= "\n".sprintf('- Factory Service: inline factory service (%s)', $factory[0]->getClass() ? sprintf('`%s`', $factory[0]->getClass()) : 'not configured');
                 } else {
                     $output .= "\n".'- Factory Class: `'.$factory[0].'`';
                 }
diff --git a/Console/Descriptor/TextDescriptor.php b/Console/Descriptor/TextDescriptor.php
@@ -315,7 +315,7 @@ protected function describeContainerDefinition(Definition $definition, array $op
                 if ($factory[0] instanceof Reference) {
                     $tableRows[] = ['Factory Service', $factory[0]];
                 } elseif ($factory[0] instanceof Definition) {
-                    $tableRows[] = ['Factory Class', $factory[0]->getClass() ?? 'not configured'];
+                    $tableRows[] = ['Factory Service', sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'class not configured')];
                 } else {
                     $tableRows[] = ['Factory Class', $factory[0]];
                 }
diff --git a/Console/Descriptor/XmlDescriptor.php b/Console/Descriptor/XmlDescriptor.php
@@ -324,7 +324,7 @@ private function getContainerDefinitionDocument(Definition $definition, string $
                 if ($factory[0] instanceof Reference) {
                     $factoryXML->setAttribute('service', (string) $factory[0]);
                 } elseif ($factory[0] instanceof Definition) {
-                    $factoryXML->setAttribute('class', $factory[0]->getClass() ?? 'not configured');
+                    $factoryXML->setAttribute('service', sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'not configured'));
                 } else {
                     $factoryXML->setAttribute('class', $factory[0]);
                 }
diff --git a/Tests/Fixtures/Descriptor/builder_1_services.json b/Tests/Fixtures/Descriptor/builder_1_services.json
@@ -45,7 +45,7 @@
             "autowire": false,
             "autoconfigure": false,
             "file": "\/path\/to\/file",
-            "factory_class": "Full\\Qualified\\FactoryClass",
+            "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",
             "factory_method": "get",
             "tags": []
         }
diff --git a/Tests/Fixtures/Descriptor/builder_1_services.md b/Tests/Fixtures/Descriptor/builder_1_services.md
@@ -36,7 +36,7 @@ Definitions
 - Autowired: no
 - Autoconfigured: no
 - File: `/path/to/file`
-- Factory Class: `Full\Qualified\FactoryClass`
+- Factory Service: inline factory service (`Full\Qualified\FactoryClass`)
 - Factory Method: `get`
 
 
diff --git a/Tests/Fixtures/Descriptor/builder_1_services.xml b/Tests/Fixtures/Descriptor/builder_1_services.xml
@@ -18,6 +18,6 @@
     </tags>
   </definition>
   <definition id=".definition_3" class="Full\Qualified\Class3" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="/path/to/file">
-    <factory class="Full\Qualified\FactoryClass" method="get"/>
+    <factory service="inline factory service (Full\Qualified\FactoryClass)" method="get"/>
   </definition>
 </container>
diff --git a/Tests/Fixtures/Descriptor/definition_3.json b/Tests/Fixtures/Descriptor/definition_3.json
@@ -8,7 +8,7 @@
     "autowire": false,
     "autoconfigure": false,
     "file": "\/path\/to\/file",
-    "factory_class": "Full\\Qualified\\FactoryClass",
+    "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",
     "factory_method": "get",
     "tags": []
 }
diff --git a/Tests/Fixtures/Descriptor/definition_3.md b/Tests/Fixtures/Descriptor/definition_3.md
@@ -7,5 +7,5 @@
 - Autowired: no
 - Autoconfigured: no
 - File: `/path/to/file`
-- Factory Class: `Full\Qualified\FactoryClass`
+- Factory Service: inline factory service (`Full\Qualified\FactoryClass`)
 - Factory Method: `get`
diff --git a/Tests/Fixtures/Descriptor/definition_3.txt b/Tests/Fixtures/Descriptor/definition_3.txt
@@ -1,18 +1,18 @@
- ---------------- ----------------------------- 
- [32m Option         [39m [32m Value                       [39m 
- ---------------- ----------------------------- 
-  Service ID       -                            
-  Class            Full\Qualified\Class3        
-  Tags             -                            
-  Public           no                           
-  Synthetic        no                           
-  Lazy             no                           
-  Shared           yes                          
-  Abstract         no                           
-  Autowired        no                           
-  Autoconfigured   no                           
-  Required File    /path/to/file                
-  Factory Class    Full\Qualified\FactoryClass  
-  Factory Method   get                          
- ---------------- ----------------------------- 
+ ----------------- ------------------------------------------------------ 
+ [32m Option          [39m [32m Value                                                [39m 
+ ----------------- ------------------------------------------------------ 
+  Service ID        -                                                     
+  Class             Full\Qualified\Class3                                 
+  Tags              -                                                     
+  Public            no                                                    
+  Synthetic         no                                                    
+  Lazy              no                                                    
+  Shared            yes                                                   
+  Abstract          no                                                    
+  Autowired         no                                                    
+  Autoconfigured    no                                                    
+  Required File     /path/to/file                                         
+  Factory Service   inline factory service (Full\Qualified\FactoryClass)  
+  Factory Method    get                                                   
+ ----------------- ------------------------------------------------------ 
 
diff --git a/Tests/Fixtures/Descriptor/definition_3.xml b/Tests/Fixtures/Descriptor/definition_3.xml
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <definition class="Full\Qualified\Class3" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="/path/to/file">
-  <factory class="Full\Qualified\FactoryClass" method="get"/>
+  <factory service="inline factory service (Full\Qualified\FactoryClass)" method="get"/>
 </definition>
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_3.json b/Tests/Fixtures/Descriptor/definition_arguments_3.json
@@ -9,7 +9,7 @@
     "autoconfigure": false,
     "arguments": [],
     "file": "\/path\/to\/file",
-    "factory_class": "Full\\Qualified\\FactoryClass",
+    "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",
     "factory_method": "get",
     "tags": []
 }
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_3.md b/Tests/Fixtures/Descriptor/definition_arguments_3.md
@@ -8,5 +8,5 @@
 - Autoconfigured: no
 - Arguments: no
 - File: `/path/to/file`
-- Factory Class: `Full\Qualified\FactoryClass`
+- Factory Service: inline factory service (`Full\Qualified\FactoryClass`)
 - Factory Method: `get`
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_3.txt b/Tests/Fixtures/Descriptor/definition_arguments_3.txt
@@ -1,18 +1,18 @@
- ---------------- ----------------------------- 
- [32m Option         [39m [32m Value                       [39m 
- ---------------- ----------------------------- 
-  Service ID       -                            
-  Class            Full\Qualified\Class3        
-  Tags             -                            
-  Public           no                           
-  Synthetic        no                           
-  Lazy             no                           
-  Shared           yes                          
-  Abstract         no                           
-  Autowired        no                           
-  Autoconfigured   no                           
-  Required File    /path/to/file                
-  Factory Class    Full\Qualified\FactoryClass  
-  Factory Method   get                          
- ---------------- -----------------------------
+ ----------------- ------------------------------------------------------ 
+ [32m Option          [39m [32m Value                                                [39m 
+ ----------------- ------------------------------------------------------ 
+  Service ID        -                                                     
+  Class             Full\Qualified\Class3                                 
+  Tags              -                                                     
+  Public            no                                                    
+  Synthetic         no                                                    
+  Lazy              no                                                    
+  Shared            yes                                                   
+  Abstract          no                                                    
+  Autowired         no                                                    
+  Autoconfigured    no                                                    
+  Required File     /path/to/file                                         
+  Factory Service   inline factory service (Full\Qualified\FactoryClass)  
+  Factory Method    get                                                   
+ ----------------- ------------------------------------------------------
 
diff --git a/Tests/Fixtures/Descriptor/definition_arguments_3.xml b/Tests/Fixtures/Descriptor/definition_arguments_3.xml
@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <definition class="Full\Qualified\Class3" public="false" synthetic="false" lazy="false" shared="true" abstract="false" autowired="false" autoconfigured="false" file="/path/to/file">
-  <factory class="Full\Qualified\FactoryClass" method="get"/>
+  <factory service="inline factory service (Full\Qualified\FactoryClass)" method="get"/>
 </definition>

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,7 @@ private function getContainerDefinitionData(Definition $definition, bool $omitTa`
`245`	`245`	`if ($factory[0] instanceof Reference) {`
`246`	`246`	`$data['factory_service'] = (string) $factory[0];`
`247`	`247`	`} elseif ($factory[0] instanceof Definition) {`
`248`		`- $data['factory_class'] = $factory[0]->getClass() ?? 'not configured';`
	`248`	`+ $data['factory_service'] = sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'class not configured');`
`249`	`249`	`} else {`
`250`	`250`	`$data['factory_class'] = $factory[0];`
`251`	`251`	`}`
Original file line number	Diff line number	Diff line change
`@@ -224,7 +224,7 @@ protected function describeContainerDefinition(Definition $definition, array $op`
`224`	`224`	`if ($factory[0] instanceof Reference) {`
`225`	`225`	$output .= "\n".'- Factory Service: `'.$factory[0].'`';
`226`	`226`	`} elseif ($factory[0] instanceof Definition) {`
`227`		- $output .= "\n".'- Factory Class: `'.($factory[0]->getClass() ?? 'not configured').'`';
	`227`	+ $output .= "\n".sprintf('- Factory Service: inline factory service (%s)', $factory[0]->getClass() ? sprintf('`%s`', $factory[0]->getClass()) : 'not configured');
`228`	`228`	`} else {`
`229`	`229`	$output .= "\n".'- Factory Class: `'.$factory[0].'`';
`230`	`230`	`}`
Original file line number	Diff line number	Diff line change
`@@ -315,7 +315,7 @@ protected function describeContainerDefinition(Definition $definition, array $op`
`315`	`315`	`if ($factory[0] instanceof Reference) {`
`316`	`316`	`$tableRows[] = ['Factory Service', $factory[0]];`
`317`	`317`	`} elseif ($factory[0] instanceof Definition) {`
`318`		`- $tableRows[] = ['Factory Class', $factory[0]->getClass() ?? 'not configured'];`
	`318`	`+ $tableRows[] = ['Factory Service', sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'class not configured')];`
`319`	`319`	`} else {`
`320`	`320`	`$tableRows[] = ['Factory Class', $factory[0]];`
`321`	`321`	`}`
Original file line number	Diff line number	Diff line change
`@@ -324,7 +324,7 @@ private function getContainerDefinitionDocument(Definition $definition, string $`
`324`	`324`	`if ($factory[0] instanceof Reference) {`
`325`	`325`	`$factoryXML->setAttribute('service', (string) $factory[0]);`
`326`	`326`	`} elseif ($factory[0] instanceof Definition) {`
`327`		`- $factoryXML->setAttribute('class', $factory[0]->getClass() ?? 'not configured');`
	`327`	`+ $factoryXML->setAttribute('service', sprintf('inline factory service (%s)', $factory[0]->getClass() ?? 'not configured'));`
`328`	`328`	`} else {`
`329`	`329`	`$factoryXML->setAttribute('class', $factory[0]);`
`330`	`330`	`}`
Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@`
`45`	`45`	`"autowire": false,`
`46`	`46`	`"autoconfigure": false,`
`47`	`47`	`"file": "\/path\/to\/file",`
`48`		`- "factory_class": "Full\\Qualified\\FactoryClass",`
	`48`	`+ "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",`
`49`	`49`	`"factory_method": "get",`
`50`	`50`	`"tags": []`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`"autowire": false,`
`9`	`9`	`"autoconfigure": false,`
`10`	`10`	`"file": "\/path\/to\/file",`
`11`		`- "factory_class": "Full\\Qualified\\FactoryClass",`
	`11`	`+ "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",`
`12`	`12`	`"factory_method": "get",`
`13`	`13`	`"tags": []`
`14`	`14`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@`
`9`	`9`	`"autoconfigure": false,`
`10`	`10`	`"arguments": [],`
`11`	`11`	`"file": "\/path\/to\/file",`
`12`		`- "factory_class": "Full\\Qualified\\FactoryClass",`
	`12`	`+ "factory_service": "inline factory service (Full\\Qualified\\FactoryClass)",`
`13`	`13`	`"factory_method": "get",`
`14`	`14`	`"tags": []`
`15`	`15`	`}`