GraphQl-93: Implement support for variables in query

[Aquarvin]

-- Variables may be Input type
-- Query now accepts variables
-- added type property for Output/Input Type element
-- functional test was added

Description (*)

Currently running the following Graph Ql query

query GetProductsQuery($page: Int, $filterInput: ProductFilterInput){
  products(
    pageSize: 10
    currentPage: $page
    filter: $filterInput
    sort: {}
  ) {
    items {
    	name
    }
  }
}

with variables:

{
  "page": 1,
  "filterInput": {
    "price": {
    	"gt": "10"
    }
  }
}

Will return filtered products list, since ProductFilterInput is an input type, according to scheme

Fixed Issues (if relevant)

1. Implement support for variables in query #93: Implement support for variables in query

Manual testing scenarios (*)

1. ...
2. ...

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 on Travis CI are green)

[Aquarvin]

Request Sample:

query GetProductsQuery($page: Int, $filterInput: ProductFilterInput){
  products(
    pageSize: 10
    currentPage: $page
    filter: $filterInput
    sort: {}
  ) {
    items {
    	name
    }
  }
}

With variables:

{
  "page": 1,
  "filterInput": {
    "price": {
    	"gt": "10"
    }
  }
}

[paliarush]

Do we have enum used here?

[paliarush]

Is this common style for documenting array structure? Usually I see
['name' => 'example value', 'type' = 'example value']

[paliarush]

Can we rename this method, to getDeclaredTypes, for example

[paliarush]

Suggested change

	* Create type object based off array of configured GraphQL InputType data.
	* Create input type object based off array of configured GraphQL InputType data.

[paliarush]

Suggested change

	* Class representing 'type' GraphQL config element.
	* Class representing 'type' GraphQL config elements.

[paliarush]

Suggested change

	private function getVariables(array $variables): array
	private function formatVariables(array $variables): array

or maybe extractVariables

[paliarush]

Please create a ticket not to load all types, but just the ones which do not have references from Query/Mutation types

[paliarush]

Suggested change

	new Phrase("Type for '{$configElementClass}' has not declared.")
	new Phrase("No mapping to Webonyx type is declared for '{$configElementClass}' config element type.")

[magento-engcom-team]

@Aquarvin thank you for contributing. Please accept Community Contributors team invitation here to gain extended permissions for this repository.

[ghost]

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