MTA-1814: Analyse functional test failures - Sprint 9

- Fixed bug with price
- Fixed bug with close window
- Added skipped tests

Author: orykh

dev/tests/functional/lib/Magento/Mtf/Client/Element/GlobalsearchElement.php

@@ -97,7 +97,7 @@ protected function clear()
      */
     protected function selectWindow()
     {
-        $this->driver->closeWindow();
+        $this->driver->selectWindow();
     }
 
     /**

dev/tests/functional/tests/app/Magento/Bundle/Test/Fixture/BundleProduct/Price.php

@@ -119,9 +119,9 @@ public function getPreset()
                 'cart_price' => '80.00',
             ],
             // Bundle fixed
-            'fixed-1' => [
-                'price_from' => '1.00',
-                'price_to' => '10.00',
+            'fixed-10' => [
+                'price_from' => '10.00',
+                'price_to' => '100.00',
                 'cart_price' => '80.00',
             ],
             'fixed-15' => [

dev/tests/functional/tests/app/Magento/Bundle/Test/TestCase/CreateBundleProductEntityTest.xml

@@ -145,7 +145,7 @@
             <data name="product/data/status" xsi:type="string">Product online</data>
             <data name="product/data/price_type" xsi:type="string">Fixed</data>
             <data name="product/data/price/value" xsi:type="string">10</data>
-            <data name="product/data/price/preset" xsi:type="string">fixed-1</data>
+            <data name="product/data/price/preset" xsi:type="string">fixed-10</data>
             <data name="product/data/tax_class_id/dataSet" xsi:type="string">taxable_goods</data>
             <data name="product/data/quantity_and_stock_status/is_in_stock" xsi:type="string">Out of Stock</data>
             <data name="product/data/weight_type" xsi:type="string">Fixed</data>

dev/tests/functional/tests/app/Magento/Customer/Test/TestCase/UpdateCustomerGroupEntityTest.php

@@ -33,6 +33,7 @@ class UpdateCustomerGroupEntityTest extends Injectable
     /* tags */
     const MVP = 'yes';
     const DOMAIN = 'CS';
+    const STABLE = 'no';
     /* end tags */
 
     /**

dev/tests/functional/tests/app/Magento/Downloadable/Test/TestCase/CreateDownloadableProductEntityTest.xml

@@ -122,6 +122,7 @@
             <constraint name="Magento\Downloadable\Test\Constraint\AssertDownloadableProductForm" />
         </variation>
         <variation name="CreateDownloadableProductEntityTestVariation7">
+            <data name="tag" xsi:type="string">stable:no</data>
             <data name="description" xsi:type="string">Create product with manage stock</data>
             <data name="product/data/name" xsi:type="string">DownloadableProduct_%isolation%</data>
             <data name="product/data/sku" xsi:type="string">DownloadableProduct_%isolation%</data>

dev/tests/functional/tests/app/Magento/Review/Test/TestCase/ManageProductReviewFromCustomerPageTest.php

@@ -50,6 +50,7 @@ class ManageProductReviewFromCustomerPageTest extends Injectable
     /* tags */
     const MVP = 'no';
     const DOMAIN = 'MX';
+    const STABLE = 'no';
     /* end tags */
 
     /**

dev/tests/functional/tests/app/Magento/Review/Test/TestCase/UpdateProductReviewEntityOnProductPageTest.php

@@ -6,9 +6,7 @@
 
 namespace Magento\Review\Test\TestCase;
 
-use Magento\Catalog\Test\Fixture\CatalogProductSimple;
 use Magento\Catalog\Test\Page\Adminhtml\CatalogProductEdit;
-use Magento\Catalog\Test\Page\Adminhtml\CatalogProductIndex;
 use Magento\Review\Test\Fixture\Review;
 use Magento\Review\Test\Page\Adminhtml\RatingEdit;
 use Magento\Review\Test\Page\Adminhtml\RatingIndex;
@@ -45,13 +43,6 @@ class UpdateProductReviewEntityOnProductPageTest extends Injectable
     const TO_MAINTAIN = 'yes';
     /* end tags */
 
-    /**
-     * Catalog product index page
-     *
-     * @var CatalogProductIndex
-     */
-    protected $catalogProductIndex;
-
     /**
      * Catalog product edit page
      *
@@ -115,21 +106,18 @@ public function __prepare(FixtureFactory $fixtureFactory)
      *
      * @param RatingIndex $ratingIndex
      * @param RatingEdit $ratingEdit
-     * @param CatalogProductIndex $catalogProductIndex
      * @param CatalogProductEdit $catalogProductEdit
      * @param ReviewEdit $reviewEdit
      * @return void
      */
     public function __inject(
         RatingIndex $ratingIndex,
         RatingEdit $ratingEdit,
-        CatalogProductIndex $catalogProductIndex,
         CatalogProductEdit $catalogProductEdit,
         ReviewEdit $reviewEdit
     ) {
         $this->ratingIndex = $ratingIndex;
         $this->ratingEdit = $ratingEdit;
-        $this->catalogProductIndex = $catalogProductIndex;
         $this->catalogProductEdit = $catalogProductEdit;
         $this->reviewEdit = $reviewEdit;
     }
@@ -145,10 +133,12 @@ public function test(Review $review, $rating)
     {
         // Steps
         $review = $this->createReview($review, $rating);
-        $this->catalogProductIndex->open();
-        /** @var CatalogProductSimple $product */
         $product = $this->reviewInitial->getDataFieldConfig('entity_id')['source']->getEntity();
-        $this->catalogProductIndex->getProductGrid()->searchAndOpen(['sku' => $product->getSku()]);
+        $this->objectManager->create(
+            'Magento\Catalog\Test\TestStep\OpenProductOnBackendStep',
+            ['product' => $product]
+        )->run();
+
         $this->catalogProductEdit->getProductForm()->openTab('product_reviews');
         $filter = [
             'title' => $this->reviewInitial->getTitle(),