Correct cart_item_id source for address items

[pmclain]

Description (*)

\Magento\Quote\Model\Quote\Address::getAlItems returns
\Magento\Quote\Model\Quote\Address\Item[] when the quote has multiple
shipping addresses and Magento\Quote\Model\Quote\Item[] with a single
shipping address. These objects have different methods for accessing the quote
item id and both variants need to be accommodated in the extractor.

Fixed Issues

1. Cannot return null for non-nullable field CartItemQuantity.cart_item_id. #822

Manual testing scenarios (*)

1. Create empty cart
2. Add non-virtual product to cart
3. Set shipping address on cart
4. Query cart shipping address requesting cart.shipping_addresses.cart_items.cart_item_id

query {
  cart (cart_id: "{$maskedQuoteId}") {
    shipping_addresses {
      cart_items {
        cart_item_id
        quantity
      }
      available_shipping_methods {
        amount {
          value
          currency
        }
        base_amount {
          value
          currency
        }
        carrier_code
        carrier_title
        error_message
        method_code
        method_title
        price_excl_tax {
          value
          currency
        }
        price_incl_tax {
          value
          currency
        }
      }
    }
  }
}

Questions or comments

Contribution checklist (*)

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds are green)

[pmclain]

@magento run Functional Tests build

[magento-engcom-team]

Hi @lenaorobei, thank you for the review.
ENGCOM-5699 has been created to process this Pull Request
✳️ @lenaorobei, could you please add one of the following labels to the Pull Request?

Label	Description
Auto-Tests: Covered	All changes in Pull Request is covered by auto-tests
Auto-Tests: Not Covered	Changes in Pull Request requires coverage by auto-tests
Auto-Tests: Not Required	Changes in Pull Request does not require coverage by auto-tests

[ghost]

Hi @pmclain, thank you for your contribution!
Please, complete Contribution Survey, it will take less than a minute.
Your feedback will help us to improve contribution process.