Problem/Motivation

PathContentModerationTest uses form submissions to change language settings.

This is expensive, and not what the test is covering.

Steps to reproduce

Proposed resolution

Change to API calls.

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#2 use_api_instead_ui-3385837-2.patch1.9 KBvbouchet

Comments

joachim created an issue. See original summary.

vbouchet’s picture

vbouchet
Location Toulouse
commented
Status: Active » Needs review
StatusFileSize
new use_api_instead_ui-3385837-2.patch1.9 KB
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Freaky literally just reviewed a ticket with the same issue.

But refactor seems good!

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Version: 11.x-dev » 10.1.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 815a709fc1 to 11.x and 4d236cf276 to 10.2.x and b9f1f9a8c6 to 10.1.x. Thanks!

  • longwave committed b9f1f9a8 on 10.1.x 
    Issue #3385837 by vbouchet: PathContentModerationTest should use API to...

  • longwave committed 4d236cf2 on 10.2.x 
    Issue #3385837 by vbouchet: PathContentModerationTest should use API to...

  • longwave committed 815a709f on 11.x 
    Issue #3385837 by vbouchet: PathContentModerationTest should use API to...
needs-review-queue-bot’s picture

needs-review-queue-bot commented
Status: Fixed » Needs work

The Needs Review Queue Bot tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request. Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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