MAGETWO-55657: Implement new service OrderInvoice

Author: NadiyaS

app/code/Magento/Sales/Api/Data/InvoiceCommentInterface.php

@@ -12,8 +12,8 @@
  * invoice history.
  * @api
  */
-interface InvoiceCommentInterface extends
-    \Magento\Framework\Api\ExtensibleDataInterface, \Magento\Sales\Api\Data\InvoiceCommentCreationInterface
+interface InvoiceCommentInterface extends \Magento\Framework\Api\ExtensibleDataInterface,
+    \Magento\Sales\Api\Data\InvoiceCommentCreationInterface
 {
     /**#@+
      * Constants for keys of data array. Identical to the name of the getter in snake case.

app/code/Magento/Sales/Model/OrderInvoice.php

@@ -141,7 +141,7 @@ public function execute(
         InvoiceCommentCreationInterface $comment = null,
         InvoiceCreationArgumentsInterface $arguments = null
     ) {
-        $connection = $this->resourceConnection->getConnection('sales');
+        $connection = $this->resourceConnection->getConnectionByName('sales');
         $order = $this->orderRepository->get($orderId);
         $invoice = $this->invoiceDocumentFactory->create(
             $order,

app/code/Magento/Sales/Test/Unit/Model/Order/Invoice/Sender/EmailSenderTest.php

@@ -204,7 +204,7 @@ protected function setUp()
             ->willReturn($this->storeMock);
 
         $this->senderBuilderFactoryMock = $this->getMockBuilder(
-                \Magento\Sales\Model\Order\Email\SenderBuilderFactory::class
+            \Magento\Sales\Model\Order\Email\SenderBuilderFactory::class
         )
         ->disableOriginalConstructor()
         ->setMethods(['create'])

app/code/Magento/Sales/Test/Unit/Model/OrderInvoiceTest.php

@@ -192,7 +192,7 @@ protected function setUp()
     public function testOrderInvoice($orderId, $capture, $items, $notify, $appendComment)
     {
         $this->resourceConnectionMock->expects($this->once())
-            ->method('getConnection')
+            ->method('getConnectionByName')
             ->with('sales')
             ->willReturn($this->adapterInterface);
 
@@ -284,9 +284,9 @@ public function testOrderInvoice($orderId, $capture, $items, $notify, $appendCom
     }
 
     /**
-     * @expectedException \Magento\Sales\Model\Order\SalesDocumentValidationException
+     * @expectedException \Magento\Sales\Api\DocumentValidationException
      */
-    public function testSalesDocumentValidationException()
+    public function testDocumentValidationException()
     {
         $orderId = 1;
         $capture = true;
@@ -326,17 +326,17 @@ public function testSalesDocumentValidationException()
     }
 
     /**
-     * @expectedException \Magento\Sales\Model\Order\SalesOperationFailedException
+     * @expectedException \Magento\Sales\Api\CouldNotInvoiceException
      */
-    public function testSalesOperationFailedException()
+    public function testCouldNotInvoiceException()
     {
         $orderId = 1;
         $items = [1 => 2];
         $capture = true;
         $notify = true;
         $appendComment = true;
         $this->resourceConnectionMock->expects($this->once())
-            ->method('getConnection')
+            ->method('getConnectionByName')
             ->with('sales')
             ->willReturn($this->adapterInterface);