Merge branch 'MAGETWO-83834' of https://github.com/magento-helix/magento2ce into pr-tests

Author: vgoncharenko

dev/tests/static/testsuite/Magento/Test/Integrity/ComposerTest.php

@@ -5,6 +5,7 @@
  */
 namespace Magento\Test\Integrity;
 
+use Composer\Semver\VersionParser;
 use Magento\Framework\App\Bootstrap;
 use Magento\Framework\Component\ComponentRegistrar;
 use Magento\Framework\Composer\MagentoComponent;
@@ -20,6 +21,11 @@ class ComposerTest extends \PHPUnit\Framework\TestCase
      */
     private static $root;
 
+    /**
+     * @var array
+     */
+    private static $mainComposerModules;
+
     /**
      * @var \stdClass
      */
@@ -39,6 +45,11 @@ public static function setUpBeforeClass()
     {
         self::$root = BP;
         self::$rootJson = json_decode(file_get_contents(self::$root . '/composer.json'), true);
+        $availableSections = ['require', 'require-dev', 'replace'];
+        self::$mainComposerModules = [];
+        foreach ($availableSections as $availableSection) {
+            self::$mainComposerModules = array_merge(self::$mainComposerModules, self::$rootJson[$availableSection]);
+        }
         self::$dependencies = [];
         self::$objectManager = Bootstrap::create(BP, $_SERVER)->getObjectManager();
     }
@@ -174,6 +185,8 @@ private function assertMagentoConventions($dir, $packageType, \StdClass $json)
             default:
                 throw new \InvalidArgumentException("Unknown package type {$packageType}");
         }
+
+        $this->assertPackageVersions($json);
     }
 
     /**
@@ -290,11 +303,11 @@ private function assertRequireInSync(\StdClass $json)
                     // Magento Composer Installer is not needed for already existing components
                     continue;
                 }
-                if (!isset(self::$rootJson['require-dev'][$depName]) && !isset(self::$rootJson['require'][$depName])
-                    && !isset(self::$rootJson['replace'][$depName])) {
+                if (!isset(self::$mainComposerModules[$depName])) {
                     $errors[] = "'$name' depends on '$depName'";
                 }
             }
+
             if (!empty($errors)) {
                 $this->fail(
                     "The following dependencies are missing in root 'composer.json',"
@@ -308,6 +321,52 @@ private function assertRequireInSync(\StdClass $json)
         }
     }
 
+    /**
+     *
+     *
+     * @param \StdClass $json
+     */
+    private function assertPackageVersions(\StdClass $json)
+    {
+        $name = $json->name;
+        if (preg_match('/magento\/project-*/', self::$rootJson['name']) == 1) {
+            return;
+        }
+        if (isset($json->require)) {
+            $errors = [];
+            $errorTemplate = "root composer.json has dependency '%s:%s' BUT '%s' composer.json has dependency '%s:%s'";
+            foreach (array_keys((array)$json->require) as $depName) {
+                if ($this->checkDiscrepancy($json, $depName)) {
+                    $errors[] = sprintf(
+                        $errorTemplate,
+                        $depName,
+                        self::$mainComposerModules[$depName],
+                        $name,
+                        $depName,
+                        $json->require->$depName
+                    );
+                }
+            }
+
+            if (!empty($errors)) {
+                $this->fail(join("\n", $errors));
+            }
+        }
+    }
+
+    /**
+     * @param $componentConfig
+     * @param $packageName
+     * @return bool
+     */
+    private function checkDiscrepancy($componentConfig, $packageName)
+    {
+        $rootConstraint = (new VersionParser())->parseConstraints(self::$mainComposerModules[$packageName]);
+        $componentConstraint = (new VersionParser())->parseConstraints($componentConfig->require->$packageName);
+
+        return !$rootConstraint->matches($componentConstraint);
+    }
+
     /**
      * Convert a fully qualified module name to a composer package name according to conventions
      *