Add appropriate return typehints to all @dataProvider methods

Will add missing typehint to dataProviders method.

Added a typehint for dataProviders as array and \Generator wherever yield is used as return.

MR has some failures.

sorry it was my bad, Now pipelines are green, hence moving into review.

Added a typehint for dataProviders as array and \Generator wherever yield is used as return.

The issue summary suggests that generators should have the iterable typehint. So either we should update the issue summary or change \Generator to iterable.

Also found a few missing:

• All tests that extend \Drupal\Tests\migrate\Kernel\MigrateSqlSourceTestBase have a providerSource method
• \Drupal\Tests\layout_builder\Unit\SectionTest::providerTestGetThirdPartySettings

I think I have all missing return types for the @dataProviders.
@mstrelan for iterable or generator should be as a typehint , I'll left this for a discussion.
And Isn't the all iterable is a generator? So bit confused here as most of the place \Generator is mentioned the comment block of the provider method.

cc: @xjm @mondrake @mustgrave

I think it’s more accurate to typehint \Generator where yield is used.

iterable is an alias for array|\Traversable which means it’s the most generic typehint possible. See https://www.php.net/manual/en/language.types.iterable.php

Yes, All iterable's is a generator.
With the @mondrake comment#11 I'm moving this issue into review.
as per @mstrelan comment#9 summary needs to be updated.

If summary needs to be updated then it should be NW

Given #11 maybe all data providers should be iterable? It makes no difference for the purposes of the test whether it returns an array or uses a generator, and iterable allows us to use either and swap between them freely.