fix: #8264 - fully qualified string not used for types in dataloader generator

[svihpinc]

Summary of the changes (Less than 80 chars)

• To always get the correct types in generator dataloaders, use the fully qualified string for all types

Closes #8264 (in this specific format)

[CLAassistant]

All committers have signed the CLA.

[michaelstaib]

Can you add a nullable test case? for both structs and classes?

They should go here:
https://github.com/ChilliCream/graphql-platform/blob/main/src/HotChocolate/Core/test/Types.Analyzers.Tests/DataLoaderTests.cs

[Copilot]

Pull Request Overview

This PR ensures that fully qualified type names are used consistently in DataLoader generators to address issue #8264. Key changes include:

• Updating snapshot files to use fully qualified type strings for resultMap parameters.
• Adding a new extension method, ToFullyQualifiedWithNullRefQualifier, in SymbolExtensions.cs.
• Refactoring DataLoaderFileBuilder to utilize the new helper instead of a hardcoded "global::" prefix.

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

File	Description
snapshots/DataLoaderTests.*.md	Updated snapshots to include fully qualified type names.
SymbolExtensions.cs	Added an extension method for fully qualified type formatting with null ref qualifier.
DataLoaderFileBuilder.cs	Updated to use the new extension method for the resultMap parameter.

Comments suppressed due to low confidence (1)

src/HotChocolate/Core/src/Types.Analyzers/Helpers/SymbolExtensions.cs:35

• Consider adding unit tests for the new ToFullyQualifiedWithNullRefQualifier method to verify its behavior across various type scenarios.

public static string ToFullyQualifiedWithNullRefQualifier(this ITypeSymbol typeSymbol)

[svihpinc]

@michaelstaib nullable test cases are already present, eg:

graphql-platform/src/HotChocolate/Core/test/Types.Analyzers.Tests/DataLoaderTests.cs

Line 313 in 819f23c

public static Task<Dictionary<int, string?>> GetEntityByIdAsync(

But those tests are for string? type. Do you mean specifically nullable struct and class types?

[svihpinc]

@michaelstaib added a test case for #8264 issue, also covers nullable class type