Fix FilterIdTest test names

You changed name of function testProvider() with description 'The test data provider.'. In my opinion, the description should also be made more clear.

I will change to roll patch:
- for testTransform() description from 'The test data provider.' to 'Tests transformation of filter_id plugin.' ;
- for provideFilters() from 'The test data provider.' to 'The test of filter provider.'

Were changed descriptions in accordance with the new names of the methods of the class.

+++ b/core/modules/filter/tests/src/Kernel/Plugin/migrate/process/FilterIdTest.php
@@ -78,11 +78,11 @@ public function test($value, $expected_value, $invalid_id = NULL) {
-   * The test data provider.
+   * The test of filter provider.

This doesn't make it any better I think. I'd prefer something like: Provides test filter for ::testTransform.

Or doesn't that make it better?

@borisson_ Thanks for your comment. Yes I agree with you to change description for method provideFilters() to 'Provides test filter for ::testTransform.' I Re-rolled the patch.

This looks solid!

PS: Next time, please provide an interdiff as well as a patch to make reviewing easier.

Temporary testbot hiccup.

Fixed #14.

I don't think I can move this back to RTBC because I also added a patch. I just noticed that this issue was novice, sorry about that.

Committed as 69167b5 and pushed to 8.6.x.

Cherry-picked as 75c8cdd and pushed to 8.5.x.

Can we get a follow up (novice) to add names to the test cases?

e.g from

 // The filter ID is mapped, and the plugin exists.
      [
        'foo',
        'filter_html',
      ],

to

      'filter ID mapped to plugin that exists' => [
        'foo',
        'filter_html',
      ],

The output from phpunit is much easier to understand.

Test case 'Filter ID mapped to plugin that exists' being better than Test case '0'