Problem/Motivation

Unit test case failure in Drupal 9

Steps to reproduce

Download the module and install it. Then run the PHPUnit test.

Proposed resolution

To remove the core: 8.x from .info.yml file if module is used in D9.

CommentFileSizeAuthor
#2 PHPUnit_testcase_issue_3208708_2.patch395 bytesrohan_katkar

Comments

rohan_katkar created an issue. See original summary.

rohan_katkar’s picture

rohan_katkar commented
StatusFileSize
new PHPUnit_testcase_issue_3208708_2.patch395 bytes
rohan_katkar’s picture

rohan_katkar commented
Assigned: rohan_katkar » Unassigned
Status: Active » Needs review
dewalt’s picture

dewalt commented

Thanks @rohan_katkar,

I should look into the issue deeper, because documentation says here:

The core: is required here because Drupal Core versions before 8.7.7 do not recognize the core_version_requirement: key.

This means that applying this change the module would be compatible with Drupal 8.8 and higher only. I'll try to find issues in Drupal core why Unit Tests are failed in this case.

rohan_katkar’s picture

rohan_katkar commented

Yes sure ... I have tried with above patch but its not working in my case.

  • dewalt committed 5a05113 on 8.x-1.x 
    Issue #3208708 by dewalt: Unit test case failure in Drupal 9
dewalt’s picture

dewalt commented
Status: Needs review » Fixed

I've made fixes and added D9 tests to the module Automated testing. They are green now.

Status: Fixed » Closed (fixed)

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