MQE-1962: Remove support of deprecated locations for tests

Author: jilu1

docs/mftf-tests-packaging.md

@@ -0,0 +1,58 @@
+<style>
+.mftf-dl {
+  margin-bottom: 2.5em;
+}
+dl dt{
+  font-weight:400;
+}
+</style>
+
+# MFTF functional test modules and packaging
+
+## MFTF predefined test module paths
+The Magento Functional Testing Framework can run tests from predefined paths and custom paths. The predefined paths are:
+```
+app/code/<Vendor>/<Module>/Test/Mftf
+dev/tests/acceptance/tests/functional/<Vendor>/<TestModule>
+vendor/<Vendor>/<Module>/Test/Mftf
+vendor/<Vendor>/<TestModule>
+```
+
+To support future service isolation, Test module in `dev/tests/acceptance/tests/functional/<Vendor>/<TestModule>` and 
+`vendor/<Vendor>/<TestModule>` must define the module type as `magento2-functional-test-module` in a `composer.json` file. 
+No `composer.json` file is required for tests in `app/code/<Vendor>/<Module>/Test/Mftf` and `vendor/<Vendor>/<Module>/Test/Mftf` 
+as they are part of the Magento modules.
+
+Test module for a specific Magento module can only be in one of the path.
+
+## Test module composer.json format
+
+Test module `composer.json` file should use type `magento2-functional-test-module`.
+
+Test module `composer.json` file should define Magento module dependencies in suggests block. 
+MFTF will recognize the dependency if the suggest message of a module specifies `type` using `magento2-module` and `name` 
+using module name registered with Magento.
+
+Here is an example `composer.json` file for the test module `dev/tests/acceptance/tests/functional/Magento/ConfigurableProductCatalogSearch`:
+
+```json
+{
+    "name": "magento/module-configurable-product-catalog-search-functional-test",
+    "description": "MFTF test module for Magento_ConfigurableProduct and Magento_CatalogSearch",
+    "type": "magento2-functional-test-module",
+    "config": {
+        "sort-packages": true
+    },
+    "require": {
+        "magento/magento2-functional-testing-framework": ">=2.5"
+    },
+    "suggest": {
+        "magento/module-configurable-product": "type: magento2-module, name: Magento_ConfigurableProduct, version: *",
+        "magento/module-catalog-search": "type: magento2-module, name: Magento_CatalogSearch, version: *"
+    },
+    "license": [
+        "OSL-3.0",
+        "AFL-3.0"
+    ]
+}
+```

docs/mftf-tests.md

@@ -9,8 +9,7 @@ dl dt{
 
 # MFTF functional test reference
 
-The Magento Functional Testing Framework runs tests on every Module within Magento. These files are stored within each Module folder in the Magento repo.
-This page lists all those tests so that developers can have a good sense of what is covered.
+This page lists all existing tests so that developers can have a good sense of what is covered.
 
 {% include mftf/functional_data.md %}
 

src/Magento/FunctionalTestingFramework/Util/ModuleResolver.php

@@ -54,12 +54,6 @@ class ModuleResolver
     . 'tests'
     . DIRECTORY_SEPARATOR
     . 'functional';
-    const DEPRECATED_DEV_TESTS = DIRECTORY_SEPARATOR
-        . self:: DEV_TESTS
-        . DIRECTORY_SEPARATOR
-        . "Magento"
-        . DIRECTORY_SEPARATOR
-        . "FunctionalTest";
 
     /**
      * Enabled modules.
@@ -327,8 +321,7 @@ private function aggregateTestModulePaths()
         $codePathsToPattern = [
             $modulePath => '',
             $magentoBaseCodePath . $vendorCodePath => self::TEST_MFTF_PATTERN,
-            $magentoBaseCodePath . $appCodePath => self::TEST_MFTF_PATTERN,
-            $magentoBaseCodePath . self::DEPRECATED_DEV_TESTS => ''
+            $magentoBaseCodePath . $appCodePath => self::TEST_MFTF_PATTERN
         ];
 
         foreach ($codePathsToPattern as $codePath => $pattern) {
@@ -374,22 +367,6 @@ private function globRelevantPaths($testPath, $pattern)
             }
         }
 
-        /* TODO uncomment this to show deprecation warning when we ready to fully deliver test packaging feature
-        if (strpos($testPath, self::DEPRECATED_DEV_TESTS) !== false && !empty($modulePaths)) {
-            $deprecatedPath = ltrim(self::DEPRECATED_DEV_TESTS, DIRECTORY_SEPARATOR);
-            $suggestedPath = self::DEV_TESTS . DIRECTORY_SEPARATOR . 'Magento';
-            $message = "DEPRECATION: Found MFTF test modules in the deprecated path: $deprecatedPath."
-                . " Move these test modules to $suggestedPath.";
-
-            if (MftfApplicationConfig::getConfig()->verboseEnabled()) {
-                LoggingUtil::getInstance()->getLogger(ModuleResolver::class)->warning($message);
-            }
-            // Suppress print during unit testing
-            if (MftfApplicationConfig::getConfig()->getPhase() !== MftfApplicationConfig::UNIT_TEST_PHASE) {
-                print ("\n$message\n\n");
-            }
-        }
-        */
         return $modulePaths;
     }