Merge pull request #55 from magento-commerce/imported-fredden-magento…

…-semver-72 [Imported] Detect when virtualTypes move elsewhere
magento · Mar 7, 2024 · de42815 · de42815
2 parents d7feca7 + 962967d
commit de42815
Show file tree

Hide file tree

Showing 7 changed files with 96 additions and 0 deletions.
diff --git a/dev/tests/Unit/Console/Command/CompareSourceCommandDiXmlTest.php b/dev/tests/Unit/Console/Command/CompareSourceCommandDiXmlTest.php
@@ -55,6 +55,22 @@ public function changesDataProvider()
                 ],
                 ''
             ],
+            'moved-to-app-etc' => [
+                $pathToFixtures . '/moved-to-app-etc/source-code-before',
+                $pathToFixtures . '/moved-to-app-etc/source-code-after',
+                [
+                    '#Suggested semantic versioning change: NONE#',
+                ],
+                'Patch change is detected.',
+            ],
+            'moved-to-another-module' => [
+                $pathToFixtures . '/moved-to-another-module/source-code-before',
+                $pathToFixtures . '/moved-to-another-module/source-code-after',
+                [
+                    '#Suggested semantic versioning change: NONE#',
+                ],
+                'Patch change is detected.',
+            ],
             'moved-to-global' => [
                 $pathToFixtures . '/moved-to-global/source-code-before',
                 $pathToFixtures . '/moved-to-global/source-code-after',

diff --git a/...les/di_xml/moved-to-another-module/source-code-after/Magento/AnotherTestModule/etc/di.xml b/...les/di_xml/moved-to-another-module/source-code-after/Magento/AnotherTestModule/etc/di.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<!--
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+-->
+<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
+    <!-- This file is in Magento_AnotherTestModule's etc/ directory -->
+
+    <!-- @api -->
+    <virtualType name="customCacheInstance" type="Magento\Framework\App\Cache" shared="true"/>
+    <!-- @api -->
+    <virtualType name="customCacheInstance2" type="Magento\Test" />
+    <!-- @api -->
+    <virtualType name="customCacheInstance3" type="Magento\Framework\Test2" shared="false"/>
+</config>
diff --git a/...st/_files/di_xml/moved-to-another-module/source-code-before/Magento/TestModule/etc/di.xml b/...st/_files/di_xml/moved-to-another-module/source-code-before/Magento/TestModule/etc/di.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<!--
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+-->
+<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
+    <!-- This file is in Magento_TestModule's etc/ directory -->
+
+    <!-- @api -->
+    <virtualType name="customCacheInstance" type="Magento\Framework\App\Cache" shared="true"/>
+    <!-- @api -->
+    <virtualType name="customCacheInstance2" type="Magento\Test" />
+    <!-- @api -->
+    <virtualType name="customCacheInstance3" type="Magento\Framework\Test2" shared="false"/>
+</config>
diff --git a/.../CompareSourceCommandTest/_files/di_xml/moved-to-app-etc/source-code-after/app/etc/di.xml b/.../CompareSourceCommandTest/_files/di_xml/moved-to-app-etc/source-code-after/app/etc/di.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<!--
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+-->
+<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
+    <!-- This file is in a the global app/etc/ directory -->
+
+    <!-- @api -->
+    <virtualType name="customCacheInstance" type="Magento\Framework\App\Cache" shared="true"/>
+    <!-- @api -->
+    <virtualType name="customCacheInstance2" type="Magento\Test" />
+    <!-- @api -->
+    <virtualType name="customCacheInstance3" type="Magento\Framework\Test2" shared="false"/>
+</config>
diff --git a/...mmandTest/_files/di_xml/moved-to-app-etc/source-code-before/Magento/TestModule/etc/di.xml b/...mmandTest/_files/di_xml/moved-to-app-etc/source-code-before/Magento/TestModule/etc/di.xml
@@ -0,0 +1,17 @@
+<?xml version="1.0"?>
+<!--
+/**
+ * Copyright © Magento, Inc. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+-->
+<config xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:framework:ObjectManager/etc/config.xsd">
+    <!-- This file is in a module's etc/ directory -->
+
+    <!-- @api -->
+    <virtualType name="customCacheInstance" type="Magento\Framework\App\Cache" shared="true"/>
+    <!-- @api -->
+    <virtualType name="customCacheInstance2" type="Magento\Test" />
+    <!-- @api -->
+    <virtualType name="customCacheInstance3" type="Magento\Framework\Test2" shared="false"/>
+</config>
diff --git a/src/Analyzer/DiXml/VirtualTypeAnalyzer.php b/src/Analyzer/DiXml/VirtualTypeAnalyzer.php
@@ -67,6 +67,17 @@ public function analyze($registryBefore, $registryAfter)
                     continue;
                 }
 
+                foreach ($nodesAfter as $newModuleNodes) {
+                    foreach ($newModuleNodes as $nodeAfter) {
+                        if ($nodeBefore->getName() !== $nodeAfter->getName()) {
+                            continue;
+                        }
+
+                        $this->triggerNodeChange($nodeBefore, $nodeAfter, $fileBefore);
+                        continue 3;
+                    }
+                }
+
                 $operation = new VirtualTypeRemoved($fileBefore, $name);
                 $this->report->add('di', $operation);
             }

diff --git a/src/resources/application_includes.txt b/src/resources/application_includes.txt
@@ -1,4 +1,5 @@
 app/code
+app/etc
 lib/internal/Magento
 Inventory*
 Adobe*