@@ -26,76 +26,85 @@ public function _before(AcceptanceTester $I): void
2626
2727 //todo: add % to link to check all providers individually
2828 private $ providerLink =
29- '// div[contains(@class, "sign-in-providers")] ' .
29+ 'div[contains(@class, "sign-in-providers")] ' .
3030 '//div[@class="provider"] ' .
3131 '//a[contains(@href, "cl=oauth&fnc=redirect&provider=")] ' ;
3232
3333 public function testProvidersVisibilityOnHeaderLogin (AcceptanceTester $ I ): void
3434 {
35+ $ providerLinkElement = '//div[contains(@class, "dropdown-menu")]// ' . $ this ->providerLink ;
36+
3537 $ homePage = $ I ->openShop ();
3638 $ homePage ->openAccountMenu ();
37- $ I ->dontSeeElement ($ this -> providerLink );
39+ $ I ->dontSeeElement ($ providerLinkElement );
3840
3941 $ this ->setProviderState (true );
4042 $ homePage = $ I ->openShop ();
4143 $ homePage ->openAccountMenu ();
42- $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ this -> providerLink )));
44+ $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ providerLinkElement )));
4345 }
4446
4547 public function testProvidersVisibilityOnMyAccountLogin (AcceptanceTester $ I ): void
4648 {
49+ $ providerLinkElement = '//div[contains(@class, "card-body")]// ' . $ this ->providerLink ;
50+
4751 $ userLoginPage = new UserLogin ($ I );
4852 $ I ->amOnPage ($ userLoginPage ->URL );
49- $ I ->dontSeeElement ($ this -> providerLink );
53+ $ I ->dontSeeElement ($ providerLinkElement );
5054
5155 $ this ->setProviderState (true );
5256 $ userLoginPage = new UserLogin ($ I );
5357 $ I ->amOnPage ($ userLoginPage ->URL );
54- $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ this -> providerLink )));
58+ $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ providerLinkElement )));
5559 }
5660
5761 public function testProviderVisibilityOnCheckoutPage (AcceptanceTester $ I ): void
5862 {
63+ $ providerLinkElement = '//div[contains(@class, "card-body")]// ' . $ this ->providerLink ;
64+
5965 $ basket = new Basket ($ I );
6066 $ basket ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 );
61- $ I ->waitForElementNotVisible ( $ this -> providerLink );
67+ $ I ->dontSeeElement ( $ providerLinkElement );
6268
6369 $ this ->setProviderState (true );
6470 $ basket = new Basket ($ I );
6571 $ basket ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 );
66- $ I ->waitForElementNotVisible ( $ this -> providerLink );
67- $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ this -> providerLink )));
72+ $ I ->seeElement ( $ providerLinkElement );
73+ $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ providerLinkElement )));
6874 }
6975
7076 public function testProviderVisibilityOnCheckoutWithoutAccount (AcceptanceTester $ I ): void
7177 {
78+ $ providerLinkElement = '//div[contains(@class, "card-body")]// ' . $ this ->providerLink ;
79+
7280 $ basket = new Basket ($ I );
73- $ basket
74- ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 )
75- ->selectOptionNoRegistration ();
76- $ I ->waitForElementNotVisible ($ this ->providerLink );
81+ $ userCheckout = $ basket ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 );
82+ $ userCheckout ->selectOptionNoRegistration ();
83+ $ I ->dontSeeElement ($ providerLinkElement );
7784
7885 $ this ->setProviderState (true );
79- $ basket
80- ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 )
81- ->selectOptionNoRegistration ();
82- $ I ->waitForElementNotVisible ( $ this -> providerLink );
83- $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ this -> providerLink )));
86+ $ basket = new Basket ( $ I );
87+ $ userCheckout = $ basket ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 );
88+ $ userCheckout ->selectOptionNoRegistration ();
89+ $ I ->seeElement ( $ providerLinkElement );
90+ $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ providerLinkElement )));
8491 }
8592
8693 public function testProviderVisibilityOnCheckoutWithNewAccount (AcceptanceTester $ I ): void
8794 {
95+ $ providerLinkElement = '//div[contains(@class, "card-body")]// ' . $ this ->providerLink ;
96+
8897 $ basket = new Basket ($ I );
8998 $ basket
9099 ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 )
91100 ->selectOptionRegisterNewAccount ();
92- $ I ->waitForElementNotVisible ( $ this -> providerLink );
101+ $ I ->dontSeeElement ( $ providerLinkElement );
93102
94103 $ this ->setProviderState (true );
95104 $ basket
96105 ->addProductToBasketAndOpenUserCheckout ('1000 ' , 1 )
97106 ->selectOptionRegisterNewAccount ();
98- $ I ->waitForElementNotVisible ( $ this -> providerLink );
99- $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ this -> providerLink )));
107+ $ I ->seeElement ( $ providerLinkElement );
108+ $ I ->assertGreaterOrEquals (1 , count ($ I ->grabMultiple ($ providerLinkElement )));
100109 }
101110}
0 commit comments