Drupal 9 Compatibility audit tasks

  1. Run Drupal Check on the latest 8.x branch to find deprecated code and post the results as a .txt file on this issue
  2. If there are deprecations, do a dry-run of Drupal Rector on the module to automate some D9 compatibility, and post the results as a .patch file on this issue

Drupal 9 Compatibility checklist
- [ ] An automatable test exists that can be run against the Drupal core 9.x branch to verify minimum compatibility
- [ ] No Drupal 9-deprecated code deprecated exists in the codebase per drupal-check
- [ ] The info.yml file meets Drupal 9 syntax requirements (Add core_version_requirement: ^8 || ^9, unless additional specificity is required (see https://www.drupal.org/node/3070687))
- [x] “Drupal 9 porting info” exists on the project page

Drupal 9 test coverage
For further compliance with community standards, test coverage will be added as part of this issue.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mark_fullmer created an issue. See original summary.

KapilV’s picture

$ drupal-check -d bootstrap_horizontal_tabs/
4/4 [▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓▓] 100%

[OK] No errors

KapilV’s picture

Status: Active » Needs review
lreynaga’s picture

Status: Needs review » Fixed

lreynaga’s picture

Issue summary: View changes
lreynaga’s picture

Status: Fixed » Needs work
lreynaga’s picture

Assigned: Unassigned » lreynaga
lreynaga’s picture

Status: Needs work » Needs review
FileSize
8.41 KB

Adding test coverage and also making sure there are no deprecations showing up for tests.

mark_fullmer’s picture

Queueing the tests!

Status: Needs review » Needs work
lreynaga’s picture

mark_fullmer’s picture

Status: Needs review » Reviewed & tested by the community

Test coverage is a great start for demonstrating the basic working of this module! Thanks, @lreynaga! RTBC!!

  • lreynaga committed d5a543a on 8.x-1.x
    Issue #3130321 by lreynaga: Drupal 9 compatible test coverage
    
lreynaga’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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