[Behat] Add scenario for changing an email of a customer

GSadee · GSadee · commit ae02dc50589a · 2017-09-12T08:22:51.000+02:00
diff --git a/features/user/managing_customers/changing_customer_email.feature b/features/user/managing_customers/changing_customer_email.feature
@@ -0,0 +1,18 @@
+@managing_customers
+Feature: Changing an email of an existing customer
+    In order to be able to help a customer when he wants to change his email
+    As an Administrator
+    I want to be able to edit the customer's email
+
+    Background:
+        Given the store operates on a single channel in "United States"
+        And there is a customer "Frodo Baggins" with an email "f.baggins@example.com" and a password "ring"
+        And I am logged in as an administrator
+
+    @ui
+    Scenario: Changing an email of an existing customer
+        When I want to edit this customer
+        And I change their email to "strawberry@example.com"
+        And I save my changes
+        Then I should be notified that it has been successfully edited
+        And the customer should be able to log in as "strawberry@example.com" with "ring" password
diff --git a/src/Sylius/Behat/Context/Setup/CustomerContext.php b/src/Sylius/Behat/Context/Setup/CustomerContext.php
@@ -145,6 +145,7 @@ public function theStoreHasEnabledCustomerAccountWithPassword($email, $password
 
     /**
      * @Given there is a customer :name identified by an email :email and a password :password
+     * @Given there is a customer :name with an email :email and a password :password
      */
     public function theStoreHasCustomerAccountWithEmailAndPassword($name, $email, $password)
     {
diff --git a/src/Sylius/Behat/Context/Ui/Admin/ManagingCustomersContext.php b/src/Sylius/Behat/Context/Ui/Admin/ManagingCustomersContext.php
@@ -115,6 +115,15 @@ public function iSpecifyItsEmailAs($email = null)
         $this->createPage->specifyEmail($email);
     }
 
+    /**
+     * @When I change their email to :email
+     * @When I remove its email
+     */
+    public function iChangeTheirEmailTo(?string $email = null): void
+    {
+        $this->updatePage->changeEmail($email);
+    }
+
     /**
      * @When I add them
      * @When I try to add them
@@ -160,7 +169,7 @@ public function iSpecifyItsBirthdayAs($birthday)
     }
 
     /**
-     * @Given /^I want to edit (this customer)$/
+     * @When /^I want to edit (this customer)$/
      */
     public function iWantToEditThisCustomer(CustomerInterface $customer)
     {
@@ -280,14 +289,6 @@ public function theCustomerShouldStillHaveAnEmptyLastName(CustomerInterface $cus
         Assert::eq($this->updatePage->getLastName(), '');
     }
 
-    /**
-     * @When I remove its email
-     */
-    public function iRemoveItsEmail()
-    {
-        $this->updatePage->changeEmail('');
-    }
-
     /**
      * @Then I should be notified that email is not valid
      */
diff --git a/src/Sylius/Behat/Context/Ui/Shop/LoginContext.php b/src/Sylius/Behat/Context/Ui/Shop/LoginContext.php
@@ -209,6 +209,7 @@ public function iShouldBeNotifiedThatElementIsRequired($elementName)
 
     /**
      * @Then I should be able to log in as :email with :password password
+     * @Then the customer should be able to log in as :email with :password password
      */
     public function iShouldBeAbleToLogInAsWithPassword($email, $password)
     {
diff --git a/src/Sylius/Behat/Resources/config/suites/ui/user/managing_customers.yml b/src/Sylius/Behat/Resources/config/suites/ui/user/managing_customers.yml
@@ -30,5 +30,6 @@ default:
                 - sylius.behat.context.ui.admin.dashboard
                 - sylius.behat.context.ui.admin.managing_customers
                 - sylius.behat.context.ui.admin.notification
+                - sylius.behat.context.ui.shop.login
             filters:
                 tags: "@managing_customers && @ui"

Original file line number	Diff line number	Diff line change
`@@ -145,6 +145,7 @@ public function theStoreHasEnabledCustomerAccountWithPassword($email, $password`
`145`	`145`
`146`	`146`	`/**`
`147`	`147`	`* @Given there is a customer :name identified by an email :email and a password :password`
	`148`	`+ * @Given there is a customer :name with an email :email and a password :password`
`148`	`149`	`*/`
`149`	`150`	`public function theStoreHasCustomerAccountWithEmailAndPassword($name, $email, $password)`
`150`	`151`	`{`
Original file line number	Diff line number	Diff line change
`@@ -209,6 +209,7 @@ public function iShouldBeNotifiedThatElementIsRequired($elementName)`
`209`	`209`
`210`	`210`	`/**`
`211`	`211`	`* @Then I should be able to log in as :email with :password password`
	`212`	`+ * @Then the customer should be able to log in as :email with :password password`
`212`	`213`	`*/`
`213`	`214`	`public function iShouldBeAbleToLogInAsWithPassword($email, $password)`
`214`	`215`	`{`