Merge pull request #153 from magento-firedrakes/MAGETWO-31368

[Firedrakes] Unit Test Coverage

Author: sivaschenko

app/code/Magento/Bundle/view/base/web/js/price-bundle.js

@@ -22,7 +22,8 @@ define([
         ' +<%- finalPrice.formatted %>' +
         '<% } %>',
         controlContainer: 'dd', // should be eliminated
-        priceFormat: {}
+        priceFormat: {},
+        isFixedPrice: false
     };
 
     $.widget('mage.priceBundle', {
@@ -116,17 +117,17 @@ define([
          */
         _applyQtyFix: function applyQtyFix() {
             var config = this.options.optionConfig;
-            _.each(config.options, function (option) {
-                _.each(option.selections, function (item) {
-                    if (item.priceType === '0') {
+            if (config.isFixedPrice) {
+                _.each(config.options, function (option) {
+                    _.each(option.selections, function (item) {
                         if (item.qty && item.qty !== 1) {
                             _.each(item.prices, function (price) {
                                 price.amount = price.amount / item.qty;
                             });
                         }
-                    }
+                    });
                 });
-            });
+            }
         },
 
         /**

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

@@ -253,7 +253,7 @@ public function getComments();
      * @param \Magento\Sales\Api\Data\ShipmentCommentInterface[] $comments
      * @return $this
      */
-    public function setComments(array $comments = null);
+    public function setComments($comments = null);
 
     /**
      * Sets the store ID for the shipment.

app/code/Magento/Sales/Model/Order/Creditmemo/Item.php

@@ -167,7 +167,7 @@ public function setQty($qty)
     /**
      * Applying qty to order item
      *
-     * @return \Magento\Sales\Model\Order\Shipment\Item
+     * @return \Magento\Sales\Model\Order\Creditmemo\Item
      */
     public function register()
     {

app/code/Magento/Sales/Model/Order/Shipment.php

@@ -733,7 +733,7 @@ public function getComments()
      * @param \Magento\Sales\Api\Data\ShipmentCommentInterface[] $comments
      * @return $this
      */
-    public function setComments(array $comments = null)
+    public function setComments($comments = null)
     {
         return $this->setData(ShipmentInterface::COMMENTS, $comments);
     }

app/code/Magento/Sales/Test/Unit/Model/Order/Admin/ItemTest.php

@@ -0,0 +1,59 @@
+<?php
+/**
+ * Copyright © 2015 Magento. All rights reserved.
+ * See COPYING.txt for license details.
+ */
+namespace Magento\Sales\Test\Unit\Model\Order\Admin;
+
+/**
+ * Class ValidatorTest
+ */
+class ItemTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * @var \PHPUnit_Framework_MockObject_MockObject
+     */
+    protected $orderItemMock;
+
+    /** @var \Magento\Sales\Model\Order\Admin\Item */
+    protected $item;
+
+
+    public function setUp()
+    {
+        $this->orderItemMock = $this->getMockBuilder('Magento\Sales\Model\Order\Item')
+            ->disableOriginalConstructor()
+            ->getMock();
+        $this->item = new \Magento\Sales\Model\Order\Admin\Item();
+    }
+
+    public function testGetSku()
+    {
+        $sku = 'sku';
+        $this->orderItemMock->expects($this->once())
+            ->method('getSku')
+            ->willReturn($sku);
+        $result = $this->item->getSku($this->orderItemMock);
+        $this->assertEquals($sku, $result);
+    }
+
+    public function testGetName()
+    {
+        $name = 'name';
+        $this->orderItemMock->expects($this->once())
+            ->method('getName')
+            ->willReturn($name);
+        $result = $this->item->getName($this->orderItemMock);
+        $this->assertEquals($name, $result);
+    }
+
+    public function testGetProductId()
+    {
+        $productId = 1;
+        $this->orderItemMock->expects($this->once())
+            ->method('getProductId')
+            ->willReturn($productId);
+        $result = $this->item->getProductId($this->orderItemMock);
+        $this->assertEquals($productId, $result);
+    }
+}