Merge pull request #6364 from nanasess/fix-codeception

頻繁に落ちるE2Eテストに wait を入れる

Author: ji-eunsoo

codeception/_support/Page/Admin/OrderManagePage.php

@@ -222,6 +222,7 @@ public function 一括メール送信()
 
     public function 一括メール送信_キャンセル()
     {
+        $this->tester->wait(1);
         $this->tester->click(['id' => 'bulkSendMail']);
         $this->tester->waitForElementVisible(['id' => 'sentUpdateModal']);
         $this->tester->wait(1);

codeception/_support/Page/Front/ProductListPage.php

@@ -18,6 +18,7 @@ class ProductListPage extends AbstractFrontPage
     public static function at($I)
     {
         $page = new self($I);
+        $page->tester->wait(1);
         $page->tester->seeInCurrentUrl('/products/list');
 
         return $page;
@@ -63,6 +64,7 @@ public function カートに入れる($index, $num = 1, $category1 = null, $cate
             }
         }
         $this->tester->click(['class' => 'add-cart']);
+        $this->tester->wait(1);
         $this->tester->waitForElementVisible(['css' => 'div.ec-modal-box']);
 
         return $this;

codeception/acceptance/EA07BasicinfoCest.php

@@ -160,6 +160,7 @@ public function basicinfo_会員設定_マイページ注文状況(AcceptanceTes
             ->登録();
 
         MyPage::go($I)->注文履歴();
+        $I->wait(1);
         $I->waitForText('ご注文状況', 10, '.ec-historyRole');
     }
 
@@ -352,7 +353,7 @@ public function basicinfo_ポイント設定_有効(AcceptanceTester $I)
         OrderManagePage::go($I)
             ->検索($customer->getEmail())
             ->一覧_編集(1);
-        $I->see((string)$expected_point, OrderEditPage::$加算ポイント);
+        $I->see((string) $expected_point, OrderEditPage::$加算ポイント);
 
         $I->amGoingTo('発送済みにする (ポイントが付与される)');
         OrderEditPage::at($I)
@@ -364,7 +365,7 @@ public function basicinfo_ポイント設定_有効(AcceptanceTester $I)
         CustomerManagePage::go($I)
             ->検索($customer->getEmail())
             ->一覧_編集(1);
-        $I->seeInField(CustomerManagePage::$ポイント, (string)$customerPoint);
+        $I->seeInField(CustomerManagePage::$ポイント, (string) $customerPoint);
 
         $I->expect('マイベージにて、ポイントが付与されていること');
         MyPage::go($I);
@@ -402,8 +403,8 @@ public function basicinfo_ポイント設定_有効(AcceptanceTester $I)
             ->検索($customer->getEmail())
             ->一覧_編集(1);
         $I->see($expected_discount, OrderEditPage::$ポイント値引き額);
-        $I->seeInField(OrderEditPage::$利用ポイント, (string)$expected_point);
-        $I->see((string)($expected_point - round(($point_conversion_rate * $expected_point) * ($point_rate / 100))), OrderEditPage::$加算ポイント);
+        $I->seeInField(OrderEditPage::$利用ポイント, (string) $expected_point);
+        $I->see((string) ($expected_point - round(($point_conversion_rate * $expected_point) * ($point_rate / 100))), OrderEditPage::$加算ポイント);
 
         $I->expect('ポイント付与率を変更しても, 注文のポイントに影響無いことを確認します');
         // see https://github.com/EC-CUBE/ec-cube/pull/5571
@@ -417,14 +418,14 @@ public function basicinfo_ポイント設定_有効(AcceptanceTester $I)
             ->受注情報登録();
 
         $I->see($expected_discount, OrderEditPage::$ポイント値引き額);
-        $I->seeInField(OrderEditPage::$利用ポイント, (string)$expected_point);
-        $I->see((string)($expected_point - round(($point_conversion_rate * $expected_point) * ($point_rate / 100))), OrderEditPage::$加算ポイント);
+        $I->seeInField(OrderEditPage::$利用ポイント, (string) $expected_point);
+        $I->see((string) ($expected_point - round(($point_conversion_rate * $expected_point) * ($point_rate / 100))), OrderEditPage::$加算ポイント);
 
         $I->expect('管理画面・会員管理にて、ポイントが減少していること');
         CustomerManagePage::go($I)
             ->検索($customer->getEmail())
             ->一覧_編集(1);
-        $I->seeInField(CustomerManagePage::$ポイント, (string)$customerPoint);
+        $I->seeInField(CustomerManagePage::$ポイント, (string) $customerPoint);
 
         $I->amGoingTo('マイベージ 注文詳細にて、利用ポイントが計算されていること');
         MyPage::go($I)->注文履歴詳細(0);
@@ -875,7 +876,7 @@ public function basicinfo_メール設定_テンプレート新規作成_削除(
         $text = 'text'.$id;
         $html = '<p>HTML</p>'.$id;
 
-        /** テンプレート作成 */
+        /* テンプレート作成 */
         MailSettingsPage::go($I)
             ->入力_新規テンプレート名($template_name)
             ->入力_ファイル名($file_name)
@@ -886,7 +887,7 @@ public function basicinfo_メール設定_テンプレート新規作成_削除(
 
         $I->waitForText('保存しました', 10, MailSettingsPage::$登録完了メッセージ);
 
-        /** 受注メールから送信して確認する */
+        /* 受注メールから送信して確認する */
         $I->resetEmails();
         OrderManagePage::go($I)->検索()
             ->一覧_編集(1)
@@ -903,9 +904,9 @@ public function basicinfo_メール設定_テンプレート新規作成_削除(
         $baseinfo = Fixtures::get('baseinfo');
         $I->seeInLastEmailSubjectTo('admin@example.com', $title);
 
-        /** メールテンプレート削除 */
+        /* メールテンプレート削除 */
         MailSettingsPage::go($I)->入力_テンプレート($template_name)->削除_テンプレート();
-        $I->waitForText('削除しました',10,'#page_admin_setting_shop_mail > div.c-container > div.c-contentsArea > div.alert.alert-success.alert-dismissible.fade.show.m-3 > span');
+        $I->waitForText('削除しました', 10, '#page_admin_setting_shop_mail > div.c-container > div.c-contentsArea > div.alert.alert-success.alert-dismissible.fade.show.m-3 > span');
     }
 
     /**
@@ -1085,6 +1086,5 @@ public function basicinfo_GAタグ設定(AcceptanceTester $I)
         // GAタグが設定されているか
         $I->amOnPage('/');
         $I->seeInPageSource('https://www.googletagmanager.com/gtag/js?id=UA-12345678-1');
-
     }
 }

codeception/acceptance/EF05MypageCest.php

@@ -14,14 +14,14 @@
 use Codeception\Util\Fixtures;
 use Doctrine\DBAL\Connection;
 use Doctrine\ORM\EntityManager;
+use Eccube\Repository\CustomerAddressRepository;
+use Page\Front\CartPage;
 use Page\Front\CustomerAddressEditPage;
 use Page\Front\CustomerAddressListPage;
 use Page\Front\HistoryPage;
 use Page\Front\MyPage;
 use Page\Front\ProductDetailPage;
 use Page\Front\ShoppingPage;
-use Page\Front\CartPage;
-use Eccube\Repository\CustomerAddressRepository;
 
 /**
  * @group front
@@ -68,7 +68,7 @@ public function mypage_ご注文履歴_(AcceptanceTester $I)
         $createCustomer = Fixtures::get('createCustomer');
         $customer = $createCustomer();
         $createOrders = Fixtures::get('createOrders');
-        $Orders = $createOrders($customer, 5, [], \Eccube\Entity\Master\OrderStatus::NEW);
+        $Orders = $createOrders($customer, 5, [], Eccube\Entity\Master\OrderStatus::NEW);
 
         $I->loginAsMember($customer->getEmail(), 'password');
 
@@ -90,7 +90,7 @@ public function mypage_ご注文履歴詳細(AcceptanceTester $I)
         $createCustomer = Fixtures::get('createCustomer');
         $customer = $createCustomer();
         $createOrders = Fixtures::get('createOrders');
-        $createOrders($customer, 5, [], \Eccube\Entity\Master\OrderStatus::NEW);
+        $createOrders($customer, 5, [], Eccube\Entity\Master\OrderStatus::NEW);
 
         $I->loginAsMember($customer->getEmail(), 'password');
 
@@ -319,13 +319,13 @@ public function mypage_お届け先上限確認(AcceptanceTester $I)
         $I->loginAsMember($customer->getEmail(), 'password');
 
         // 19件のお届け先を登録
-        /** @var \Doctrine\ORM\EntityManager $em */
+        /** @var EntityManager $em */
         $em = Fixtures::get('entityManager');
-        
-        $this->customerAddressRepository = $em->getRepository(\Eccube\Entity\CustomerAddress::class);
-        
+
+        $this->customerAddressRepository = $em->getRepository(Eccube\Entity\CustomerAddress::class);
+
         for ($i = 0; $i < $max; $i++) {
-            $customerAddress = new \Eccube\Entity\CustomerAddress();
+            $customerAddress = new Eccube\Entity\CustomerAddress();
             $customerAddress
                 ->setCustomer($customer)
                 ->setName01($customer->getName01())
@@ -341,7 +341,7 @@ public function mypage_お届け先上限確認(AcceptanceTester $I)
 
             $em->persist($customerAddress);
         }
-        
+
         $em->flush();
 
         // TOPページ>マイページ>お届け先一覧で上限に達していることを確認
@@ -362,7 +362,7 @@ public function mypage_お届け先上限確認(AcceptanceTester $I)
         ShoppingPage::at($I)->お届け先変更();
 
         $I->wait(1);
-        
+
         $I->see(sprintf('お届け先登録の上限の%s件に達しています。お届け先を入力したい場合は、削除か変更を行ってください。', 20), 'div.ec-registerRole > div > div > div ');
 
         // 受注に紐づくidに直接アクセスしても登録されないことを確認
@@ -390,7 +390,7 @@ public function mypage_退会手続き未実施(AcceptanceTester $I)
 
         // 会員退会手続きへ
         $I->click('div.ec-withdrawRole form button');
-
+        $I->wait(1);
         // 未実施
         $I->click('div.ec-withdrawConfirmRole form a.ec-withdrawConfirmRole__cancel');
 

codeception/acceptance/EF09ThrottlingCest.php

@@ -199,6 +199,7 @@ private function contactConfirm(AcceptanceTester $I)
 
     private function contactComplete(AcceptanceTester $I)
     {
+        $I->wait(1);
         $I->click('div.ec-contactConfirmRole div.ec-RegisterRole__actions button.ec-blockBtn--action');
     }