Fix/required variables in query hook

[tomaskukk]

Description

Currently when a query has required variables, the passing of variables object isn't enforced in query hook args. This can cause runtime errors as if one doesn't specify the variables object at all, the types all are happy.

This PR introduces a proposed fix to ensure that variables need to be passed in args, when there are some required variables in the query. Similar has been done already for the component generation, but looks like this wasn't taken into account then.

Related dotansimha/graphql-code-generator#2870

Type of change

Please delete options that are not relevant.

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

There are new test cases ensuring that variables added as a required field instead of optional, when some of the query variables are required.

I ran the tests locally and saw that they passed.

Test Environment:

• OS: macOS
• @graphql-codegen/typescript-react-apollo: 4.2.0
• NodeJS: v18.19.0

Checklist:

• I have followed the
  CONTRIBUTING doc and the
  style guidelines of this project
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules

[changeset-bot]

🦋 Changeset detected

Latest commit: 1177546

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@graphql-codegen/typescript-react-apollo	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[saihaj]

Thank you! Published @graphql-codegen/typescript-react-apollo@4.3.0