From b1e518ca649a862abca2856db0c99d385ba964a3 Mon Sep 17 00:00:00 2001
From: Sergey Dovbenko <sdovbenko@magecom.us>
Date: Fri, 6 Dec 2019 18:16:25 +0000
Subject: [PATCH] Corrected Code Styles

---
 .../Magento/AsynchronousOperations/Model/Operation.php    | 8 ++++----
 .../Model/OperationStatusValidator.php                    | 6 +++---
 .../Test/Unit/Model/OperationStatusValidatorTest.php      | 4 ++--
 3 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/app/code/Magento/AsynchronousOperations/Model/Operation.php b/app/code/Magento/AsynchronousOperations/Model/Operation.php
index de2b56d0a0dd7..00f33d10a1e1b 100644
--- a/app/code/Magento/AsynchronousOperations/Model/Operation.php
+++ b/app/code/Magento/AsynchronousOperations/Model/Operation.php
@@ -10,7 +10,7 @@
 use Magento\Framework\DataObject;
 
 /**
- * Class Operation
+ * Class Operation encapsulates methods for Operation Model Object
  */
 class Operation extends DataObject implements OperationInterface
 {
@@ -22,12 +22,12 @@ class Operation extends DataObject implements OperationInterface
     /**
      * Operation constructor.
      *
-     * @param array $data
      * @param OperationStatusValidator $operationStatusValidator
+     * @param array $data
      */
     public function __construct(
-        array $data = [],
-        OperationStatusValidator $operationStatusValidator
+        OperationStatusValidator $operationStatusValidator,
+        array $data = []
     ) {
         $this->operationStatusValidator = $operationStatusValidator;
         parent::__construct($data);
diff --git a/app/code/Magento/AsynchronousOperations/Model/OperationStatusValidator.php b/app/code/Magento/AsynchronousOperations/Model/OperationStatusValidator.php
index ae154edfb50f7..f2ae135e4303c 100644
--- a/app/code/Magento/AsynchronousOperations/Model/OperationStatusValidator.php
+++ b/app/code/Magento/AsynchronousOperations/Model/OperationStatusValidator.php
@@ -32,7 +32,9 @@ public function __construct(OperationStatusPool $operationStatusPool)
     /**
      * Validate method
      *
-     * @param $status
+     * @param int $status
+     * @throws \InvalidArgumentException
+     * @return void
      */
     public function validate($status)
     {
@@ -41,7 +43,5 @@ public function validate($status)
         if (!in_array($status, $statuses)) {
             throw new \InvalidArgumentException('Invalid Operation Status.');
         }
-
-        return;
     }
 }
diff --git a/app/code/Magento/AsynchronousOperations/Test/Unit/Model/OperationStatusValidatorTest.php b/app/code/Magento/AsynchronousOperations/Test/Unit/Model/OperationStatusValidatorTest.php
index 2209564d03aee..b93d9701a7a69 100644
--- a/app/code/Magento/AsynchronousOperations/Test/Unit/Model/OperationStatusValidatorTest.php
+++ b/app/code/Magento/AsynchronousOperations/Test/Unit/Model/OperationStatusValidatorTest.php
@@ -12,7 +12,7 @@
 use PHPUnit\Framework\TestCase;
 
 /**
- * Class OperationStatusValidatorTest
+ * Class OperationStatusValidatorTest implements logic for testing Operation::setStatus() method
  */
 class OperationStatusValidatorTest extends TestCase
 {
@@ -60,7 +60,7 @@ protected function setUp()
      * @param string $expectedResult
      * @dataProvider dataProviderForTestSetStatus
      */
-    public function testSetStatus (
+    public function testSetStatus(
         string $status,
         array $statusPool,
         string $expectedResult