Adjust DataProvider test code

[phil-davis]

Adjust the DataProvider test code to use Attribute rather than Annotation.

This happened in other sabre-io repos because phpstan is at a higher level, and I think that it adjusted this stuff.

phpstan is still at a low level in Vobject, so I used Rector rules:

    ->withRules([
        AddParamArrayDocblockFromDataProviderRector::class,
        DataProviderAnnotationToAttributeRector::class,
    ])

Note: when I tried to use Rector:

    ->withSets([
        // Includes sets leading up to and including PHPUnit 11
        PHPUnitSetList::PHPUNIT_100,
        PHPUnitSetList::PHPUNIT_110,
        PHPUnitSetList::PHPUNIT_CODE_QUALITY,
    ])

It did things like putting the PHPunit assert calls back from static to dynamic calls. And then phpstan complained about that and wanted to put them back to static calls. So there is some conflict between what the Rector and phpstan rule sets think is "a good thing".

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.66%. Comparing base (55cb62c) to head (6afc39c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##             master     #772   +/-   ##
=========================================
  Coverage     98.66%   98.66%           
  Complexity     1757     1757           
=========================================
  Files            71       71           
  Lines          5163     5163           
=========================================
  Hits           5094     5094           
  Misses           69       69           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.