Add behat scenario to test no generation invoice for order without number (#396)

| Q               | A
|-----------------|-----
| Branch?         | 2.1
| Bug fix?        | no
| New feature?    | yes
| BC breaks?      | no
| Deprecations?   | no
| Related tickets | #159
| License         | MIT

Author: NoResponseMate

features/managing_invoices/generating_invoices_for_previously_placed_orders.feature

@@ -4,20 +4,26 @@ Feature: Generating invoices for previously placed orders
     As a Shop Owner
     I want to easily generate invoices for orders placed before installation of the plugin
 
-    Background:
-        Given the store operates on a single channel in "United States"
-        And the store has "VAT" tax rate of 10% for "Clothes" within the "US" zone
-        And the store has a product "Angel T-Shirt" priced at "$60.00"
-        And it belongs to "Clothes" tax category
-        And the store has "UPS" shipping method with "$10.00" fee
-        And the store allows paying with "Cash on Delivery"
-        And there is a customer "lucy@teamlucifer.com" that placed an order "#00000777"
-        And the customer bought 2 "Angel T-Shirt" products
-        And the customer "Lucifer Morningstar" addressed it to "Seaside Fwy", "90802" "Los Angeles" in the "United States"
-        And for the billing address of "Mazikeen Lilim" in the "Pacific Coast Hwy", "90806" "Los Angeles", "United States"
-        And the customer chose "UPS" shipping method with "Cash on Delivery" payment
-        And the order "#00000777" has lost all of its invoices
-        And I am logged in as an administrator
+  Background:
+    Given the store operates on a single channel in "United States"
+    And the store has "VAT" tax rate of 10% for "Clothes" within the "US" zone
+    And the store has a product "Angel T-Shirt" priced at "$60.00"
+    And it belongs to "Clothes" tax category
+    And the store has "UPS" shipping method with "$10.00" fee
+    And the store allows paying with "Cash on Delivery"
+    And there is a customer "lucy@teamlucifer.com" that placed an order "#00000777"
+    And the customer bought 2 "Angel T-Shirt" products
+    And the customer "Lucifer Morningstar" addressed it to "Seaside Fwy", "90802" "Los Angeles" in the "United States"
+    And for the billing address of "Mazikeen Lilim" in the "Pacific Coast Hwy", "90806" "Los Angeles", "United States"
+    And the customer chose "UPS" shipping method with "Cash on Delivery" payment
+    And there is a customer "chloe@lapd.com" that placed an order
+    And the customer bought 1 "Angel T-Shirt" products
+    And the customer "Chloe Decker" addressed it to "Wilshire Blvd", "90010" "Los Angeles" in the "United States"
+    And for the billing address of "Chloe Decker" in the "Wilshire Blvd", "90010" "Los Angeles", "United States"
+    And the customer chose "UPS" shipping method with "Cash on Delivery" payment
+    And this order has no number assigned
+    And all invoices have been deleted
+    And I am logged in as an administrator
 
     Scenario: Having no previous invoices
         When I browse invoices
@@ -27,3 +33,8 @@ Feature: Generating invoices for previously placed orders
         When I generate invoices for previously placed orders
         And I browse invoices
         Then I should see a single invoice for order "#00000777"
+
+    Scenario: Not generating invoices for orders without number
+        When I generate invoices for previously placed orders
+        And I browse invoices
+        Then there should be 1 invoice generated

tests/Behat/Context/Domain/GeneratingInvoiceContext.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of the Sylius package.
+ *
+ * (c) Sylius Sp. z o.o.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 declare(strict_types=1);
 
 namespace Tests\Sylius\InvoicingPlugin\Behat\Context\Domain;
@@ -29,4 +38,18 @@ public function orderHasLostAllOfItsInvoices(string $orderNumber): void
 
         $this->invoiceManager->remove($invoice);
     }
+
+    /**
+     * @Given all invoices have been deleted
+     */
+    public function allInvoicesHaveBeenDeleted(): void
+    {
+        $invoices = $this->invoiceRepository->findAll();
+
+        foreach ($invoices as $invoice) {
+            $this->invoiceManager->remove($invoice);
+        }
+
+        $this->invoiceManager->flush();
+    }
 }

tests/Behat/Context/Setup/OrderContext.php

@@ -1,5 +1,14 @@
 <?php
 
+/*
+ * This file is part of the Sylius package.
+ *
+ * (c) Sylius Sp. z o.o.
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
 declare(strict_types=1);
 
 namespace Tests\Sylius\InvoicingPlugin\Behat\Context\Setup;
@@ -27,4 +36,14 @@ public function orderHasBeenPlacedInChannel(OrderInterface $order, ChannelInterf
 
         $this->orderManager->flush();
     }
+
+    /**
+     * @Given /^(this order) has no number assigned$/
+     */
+    public function orderHasNoNumberAssigned(OrderInterface $order): void
+    {
+        $order->setNumber(null);
+
+        $this->orderManager->flush();
+    }
 }