Problem/Motivation

In #2850528: Source and Target are set to Current when target is downloaded we added a new test that didn't consider #2886767: Fix test failures after "Save and Publish" button on entity forms changed to checkbox

Proposed resolution

Fix the test so it passes with core 8.4.x

Remaining tasks

Fix the test.

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#2 2895425-fix-tests-2.patch1.02 KBpenyaskito
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

penyaskito created an issue. See original summary.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Active » Needs review
FileSize
2895425-fix-tests-2.patch1.02 KB

Attached patch with tests fixed.

penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Needs review » Reviewed & tested by the community

Build back to green as expected.

  • penyaskito committed 3a43a26 on 8.x-2.x 
    Issue #2895425 by penyaskito: Fix Drupal\lingotek\Tests\...
penyaskito’s picture

penyaskito
he/him
Primary language Spanish
Location 🧑🏽‍🌾Seville 💃, Andalusia 🇪🇸, UTC+2 🇪🇺
CreditAttribution: penyaskito at Lingotek commented
Status: Reviewed & tested by the community » Fixed

Committed 3a43a26 and pushed to 8.x-2.x. Thanks!

Status: Fixed » Closed (fixed)

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