Closed (fixed)
Project:
Feeds
Version:
8.x-3.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
19 Apr 2020 at 18:35 UTC
Updated:
19 Jun 2020 at 12:34 UTC
Jump to comment: Most recent
With #3042774: Drupal 9 Deprecated Code Report committed, the work to get the module fully compatible with Drupal 9 is not done yet.
There are currently 106 test failures on D9:
https://www.drupal.org/pift-ci-job/1655810
I've scanned through the report, and I found the following kind of errors:
Exception: Drupal\Tests\BrowserTestBase::$defaultTheme is required.Drupal\Tests\feeds\Unit\FeedExpireHandlerTest::testPostExpire
Drupal\Core\DependencyInjection\ContainerNotInitializedException: \Drupal::$container is not initialized yet. \Drupal::setContainer() must be called with a real container.Drupal\Tests\feeds\Kernel\Entity\FeedTest::testPushImport
Error: Class 'Doctrine\Common\Inflector\Inflector' not foundDrupal\Tests\feeds\Unit\Component\CsvParserTest::testInvalidFilePath
The @expectedException, @expectedExceptionCode, @expectedExceptionMessage, and @expectedExceptionMessageRegExp annotations are deprecated. They will be removed in PHPUnit 9. Refactor your test to use expectException(), expectExceptionCode(), expectExceptionMessage(), or expectExceptionMessageRegExp() instead.See child issues for each identified error.
Hopefully none.
Hopefully none.
Hopefully none.
Comments
Comment #2
megachrizListed all child issues in "Remaining tasks".
Comment #3
megachrizComment #4
megachrizComment #5
megachrizComment #6
jcnventuraAdded some more deprecations in #3136594: Drupal 9 Deprecated Code Report Strikes Back
Comment #7
jcnventuraComment #8
megachrizThere is a D9 compatible release now!
https://www.drupal.org/project/feeds/releases/8.x-3.0-alpha8