[GraphQl] Gift Message coverage for customer order

[Usik2203]

Description (*)

This PR extends PR #27956 (Parent PR)
add coverage for customer order

This PR was mentioned for fixing failed static test related with composer.json file
#253

Related Pull Requests

#27956

Fixed Issues (if relevant)

1. GraphQL Checkout :: Add support for Gift Wrapping / Gift Message during checkout.

Manual testing scenarios (*)

1. For getting messages for customer order (note: You should be logged as customer)

{
  customerOrders {
  items {
      order_number
      status
      gift_message {
        to
        from
	message
      }
    }
  }
}

[m2-assistant]

Hi @Usik2203. Thank you for your contribution
Here is some useful tips how you can test your changes using Magento test environment.
Add the comment under your pull request to deploy test or vanilla Magento instance:

• @magento give me test instance - deploy test instance based on PR changes
• @magento give me 2.4-develop instance - deploy vanilla Magento instance

For more details, please, review the Magento Contributor Guide documentation.

[Usik2203]

Hi @rogyar .
I confused about implementing this point.
I can't understand which GraphQl query should be for testing this one.
Maybe you could clarify it.
Thanks a lot.

[rogyar]

Currently, we have 2 different resolvers in the directory called GiftMessage and GiftMessageCustomerOrder.
The naming is not 100% consistent. So I would propose moving resolvers to the separate directories, so we have:

• Model/Resolver/Order/GiftMessage.php
• Model/Resolver/Cart/GiftMessage.php
• Model/Resolver/Order/Item/GiftMessage (in the future)

The same applies to the tests.

[Usik2203]

Done for all three PRs
Thanks

[Usik2203]

@magento run all tests

[Usik2203]

@magento run WebAPI Tests

[Usik2203]

@magento run all tests

[paliarush]

Verified that no new @api or interfaces were introduced in scope of this PR, so approval by the architecture team is not required.

[engcom-Bravo]

✔️ QA Passed

query{
  customerOrders{
    items{
      gift_message{
        from
        message
        to
      }
    }
  }
}
response: 
{
  "data": {
    "customerOrders": {
      "items": [
        {
          "gift_message": {
            "from": "User2",
            "message": "Gift message",
            "to": "User1"
          }
        }
      ]
    }
  }
}

If user has two orders the response returns two messages:

{
  "data": {
    "customerOrders": {
      "items": [
        {
          "gift_message": {
            "from": "User2",
            "message": "Gift message",
            "to": "User1"
          }
        },
        {
          "gift_message": {
            "from": "User1",
            "message": "Thank you message",
            "to": "User2"
          }
        }
      ]
    }
  }
}

If incorrect Authorization token is provided:

{
  "errors": [
    {
      "message": "The current customer isn't authorized.",
      "extensions": {
        "category": "graphql-authorization"
      },
      "locations": [
        {
          "line": 2,
          "column": 3
        }
      ],
      "path": [
        "customerOrders"
      ]
    }
  ],
  "data": {
    "customerOrders": null
  }
}

If there is an order but with no gift message

{
  "errors": [
    {
      "message": "Can't load gift message for order",
      "extensions": {
        "category": "graphql-input"
      },
      "locations": [
        {
          "line": 4,
          "column": 7
        }
      ],
      "path": [
        "customerOrders",
        "items",
        0,
        "gift_message"
      ]
    }
  ],
  "data": {
    "customerOrders": {
      "items": [
        {
          "gift_message": null
        }
      ]
    }
  }
}

[engcom-Kilo]

@magento run all tests

[engcom-Kilo]

@magento run Magento Health Index

[engcom-Kilo]

@magento run all tests

[m2-assistant]

Hi @Usik2203, 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.