MAGETWO-36514: Fix builds

 - added phpdoc
 - removed unused variables in tests

Author: Cari Spruiell

app/code/Magento/Quote/Model/Webapi/ParamOverriderCartId.php

@@ -26,6 +26,12 @@ class ParamOverriderCartId implements ParamOverriderInterface
      */
     private $cartManagement;
 
+    /**
+     * Constructs an object to override the cart ID parameter on a request.
+     *
+     * @param UserContextInterface $userContext
+     * @param CartManagementInterface $cartManagement
+     */
     public function __construct(
         UserContextInterface $userContext,
         CartManagementInterface $cartManagement
@@ -34,6 +40,9 @@ public function __construct(
         $this->cartManagement = $cartManagement;
     }
 
+    /**
+     * {@inheritDoc}
+     */
     public function getOverridenValue()
     {
         try {

app/code/Magento/Quote/Test/Unit/Model/Webapi/ParamOverriderCartIdTest.php

@@ -69,7 +69,6 @@ public function testGetOverridenValueIsCustomerAndCartExists()
 
     public function testGetOverridenValueIsCustomerAndCartDoesNotExist()
     {
-        $retValue = 'retValue';
         $customerId = 1;
 
         $this->userContext->expects($this->once())
@@ -79,8 +78,6 @@ public function testGetOverridenValueIsCustomerAndCartDoesNotExist()
             ->method('getUserId')
             ->will($this->returnValue($customerId));
 
-        $cart = $this->getMockBuilder('Magento\Quote\Api\Data\CartInterface')
-            ->getMockForAbstractClass();
         $this->cartManagement->expects($this->once())
             ->method('getCartForCustomer')
             ->with($customerId)
@@ -91,7 +88,6 @@ public function testGetOverridenValueIsCustomerAndCartDoesNotExist()
 
     public function testGetOverridenValueIsCustomerAndCartIsNull()
     {
-        $retValue = 'retValue';
         $customerId = 1;
 
         $this->userContext->expects($this->once())
@@ -101,8 +97,6 @@ public function testGetOverridenValueIsCustomerAndCartIsNull()
             ->method('getUserId')
             ->will($this->returnValue($customerId));
 
-        $cart = $this->getMockBuilder('Magento\Quote\Api\Data\CartInterface')
-            ->getMockForAbstractClass();
         $this->cartManagement->expects($this->once())
             ->method('getCartForCustomer')
             ->with($customerId)

app/code/Magento/Webapi/Controller/Rest/ParamOverriderCustomerId.php

@@ -19,11 +19,19 @@ class ParamOverriderCustomerId implements ParamOverriderInterface
      */
     private $userContext;
 
+    /**
+     * Constructs an object to override the customer ID parameter on a request.
+     *
+     * @param UserContextInterface $userContext
+     */
     public function __construct(UserContextInterface $userContext)
     {
         $this->userContext = $userContext;
     }
 
+    /**
+     * {@inheritDoc}
+     */
     public function getOverridenValue()
     {
         if ($this->userContext->getUserType() === UserContextInterface::USER_TYPE_CUSTOMER) {

dev/tests/api-functional/testsuite/Magento/Quote/Api/CartTotalRepositoryTest.php

@@ -189,7 +189,6 @@ public function testGetMyTotals()
         /** @var \Magento\Quote\Model\Quote $quote */
         $quote = $this->objectManager->create('Magento\Quote\Model\Quote');
         $quote->load('test_order_1', 'reserved_order_id');
-        $cartId = $quote->getId();
 
         $serviceInfo = [
             'rest' => [