Problem/Motivation

In a drupal 9 project, the tests fail.

Steps to reproduce

Run the tests on a Drupal 9 environment.

CommentFileSizeAuthor
#2 3244924-2.patch1.87 KBpaulocs

Issue fork redirect-3244924

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

paulocs created an issue. See original summary.

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Assigned: paulocs » Unassigned
Status: Active » Needs review
StatusFileSize
new 3244924-2.patch1.87 KB
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Priority: Normal » Major

Status: Needs review » Needs work

The last submitted patch, 2: 3244924-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

paulocs opened merge request !21

paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Assigned: Unassigned » paulocs
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented
Assigned: paulocs » Unassigned
Status: Needs work » Needs review
paulocs’s picture

paulocs
Primary language Portuguese, Brazil
Location Belo Horizonte
commented

One of the tests triggered did not pass because I set the wrong MySql version.

gabrielda’s picture

gabrielda
He/Him
commented
Assigned: Unassigned » gabrielda

Reviewing

gabrielda’s picture

gabrielda
He/Him
commented
Assigned: gabrielda » Unassigned
Status: Needs review » Reviewed & tested by the community

As test bots are showing, all tests are passing in Drupal 9 and Drupal 8. Hence I am moving this to RTBC.

Berdir made their first commit to this issue’s fork.

  • Berdir committed b69d754 on 8.x-1.x authored by paulocs 
    Issue #3244924 by paulocs: Fix tests on Drupal 9
berdir’s picture

berdir
Primary language German
Location Switzerland
commented
Status: Reviewed & tested by the community » Fixed

Merged, thanks.

Status: Fixed » Closed (fixed)

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