Fix gitlab and test next major

Additional errors:
```
------ ---------------------------------------------------------------------
Line src/ZfExtensionManagerSfContainer.php
------ ---------------------------------------------------------------------
13 Class Drupal\aggregator\ZfExtensionManagerSfContainer implements
deprecated interface Laminas\Feed\Reader\ExtensionManagerInterface:
this interface is no longer needed, and shouldn't be relied upon
13 Class Drupal\aggregator\ZfExtensionManagerSfContainer implements
deprecated interface Laminas\Feed\Writer\ExtensionManagerInterface:
this interface is no longer needed, and shouldn't be relied upon
124 Fetching class constant class of deprecated class
Laminas\Feed\Reader\ExtensionManagerInterface:
this interface is no longer needed, and shouldn't be relied upon
124 Fetching class constant class of deprecated class
Laminas\Feed\Writer\ExtensionManagerInterface:
this interface is no longer needed, and shouldn't be relied upon
------ ---------------------------------------------------------------------
------ -----------------------------------------
Line tests/src/Functional/ImportOpmlTest.php
------ -----------------------------------------
138 Variable $title might not be defined.
139 Variable $url might not be defined.
------ -----------------------------------------
------ --------------------------------------------------------------------------------------------
Line tests/src/Functional/Plugin/aggregator/processor/DefaultProcessorTest.php
------ --------------------------------------------------------------------------------------------
23 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::deleteFeedHashes()
references an invalid method.
23 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::submitConfigurationForm()
references an invalid method.
------ --------------------------------------------------------------------------------------------
------ -----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateBrokenRssFeedViewTest.php
------ -----------------------------------------------------------------------
12 @covers value aggregator_update_8606() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
------ -----------------------------------------------------------------------
------ ----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateDeleteQueueTest.php
------ ----------------------------------------------------------------------
14 @covers value aggregator_post_update_delete_queue_items() references
an invalid class or function.
💡 The @covers annotation requires a fully qualified name.
------ ----------------------------------------------------------------------
------ -----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateFeedsTest.php
------ -----------------------------------------------------------------------
28 @covers value aggregator_update_8605() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
------ -----------------------------------------------------------------------
------ -----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateItemsTest.php
------ -----------------------------------------------------------------------
29 @covers value aggregator_update_8604() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
------ -----------------------------------------------------------------------
------ -----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateRssFeedViewTest.php
------ -----------------------------------------------------------------------
12 @covers value aggregator_update_8601() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
------ -----------------------------------------------------------------------
------ -----------------------------------------------------------------------
Line tests/src/Functional/Update/AggregatorUpdateSettingsTest.php
------ -----------------------------------------------------------------------
31 @covers value aggregator_update_8602() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
31 @covers value aggregator_update_8603() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
31 @covers value aggregator_update_8607() references an invalid class or
function.
💡 The @covers annotation requires a fully qualified name.
------ -----------------------------------------------------------------------
------ -----------------------------------------------------------
Line tests/src/Functional/UpdateFeedItemTest.php
------ -----------------------------------------------------------
29 Variable $feed in empty() always exists and is not falsy.
------ -----------------------------------------------------------
------ ----------------------------------------------------------------------------
Line tests/src/Kernel/Plugin/aggregator/processor/DefaultProcessorTest.php
------ ----------------------------------------------------------------------------
59 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::process()
references an invalid method.
160 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::process()
references an invalid method.
185 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::process()
references an invalid method.
213 @covers value
\Drupal\aggregator\Plugin\aggregator\processor\DefaultProcessor::process()
references an invalid method.
------ ----------------------------------------------------------------------------
```

Still gotta deal with these:

 ------ --------------------------------------------------------------------- 
  Line   src/ZfExtensionManagerSfContainer.php                                
 ------ --------------------------------------------------------------------- 
  13     Class Drupal\aggregator\ZfExtensionManagerSfContainer implements     
         deprecated interface Laminas\Feed\Reader\ExtensionManagerInterface:  
         this interface is no longer needed, and shouldn't be relied upon     
  13     Class Drupal\aggregator\ZfExtensionManagerSfContainer implements     
         deprecated interface Laminas\Feed\Writer\ExtensionManagerInterface:  
         this interface is no longer needed, and shouldn't be relied upon     
  124    Fetching class constant class of deprecated class                    
         Laminas\Feed\Reader\ExtensionManagerInterface:                       
         this interface is no longer needed, and shouldn't be relied upon     
  124    Fetching class constant class of deprecated class                    
         Laminas\Feed\Writer\ExtensionManagerInterface:                       
         this interface is no longer needed, and shouldn't be relied upon     
 ------ --------------------------------------------------------------------- 

@dcam @larowlan Should issues reported by PHPCS CI job also addressed in this issue? If yes I could work down some or maybe all of them.

@IT-Cru Yes, I've only been looking at the output of GitLab CI. I'm not running it manually.

The project update bot tried to fix the same deprecations that we're dealing with in this issue. I think it's a good idea if we could finalize this one before proceeding with the D11 upgrade.

Can we consider the $this->container->get('datetime.time') to avoid direct drupal call in the test cases also (One go sweep would be helpful for D11 upgrade)

I'll defer to @larowlan on that decision because I'm certain he's more aware of the implications than I am. But as I understand it, using $this->container can be fraught with problems. #2066993: Use \Drupal consistently in tests → has lingered in the Core queue for a long time, but the trend in that discussion was to standardize on \Drupal in tests because of the multiple issues that the container has caused. I'm inclined to follow that recommendation.

$this->container should be avoided in functional tests where there's a site under test and the runner. Changes to the container in the site under test don't propogate to $this->container but do to \Drupal

Okh but my intention for this suggestion specifically for Kernel Test that I have seen in the core too.
Lets take a example of Drupal\KernelTests\Core\DrupalKernel or AutowireTest
cc: @larowlan

Happy with use of it in kernel tests 👍️

@dcam - i think we should create a separate ticket to fix PHPCS issue as we have many changes in it and that might become complex to review and hence created a separate ticket to fix those - https://www.drupal.org/project/aggregator/issues/3458515 📌 Fix PHPCS issue reported by gitlab CI Needs work

Yay finally

Also cutting a d11 release

And added a scheduled task so we can catch this sooner when core breaks our stuff next time

Automatically closed - issue fixed for 2 weeks with no activity.