Problem/Motivation

Follow-up from #2848508: Reverting a revision causes validation error on edit form submit .

\Drupal\Tests\node\Functional\NodeRevisionsAllTest::testRevisions does not test updating the node again via the node form after reverting the revision. We should extend the test to do so.

Tagging novice, see the issue linked above for a close code example to what we need to add to that test.

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

catch created an issue. See original summary.

timmillwood’s picture

Issue tags: +Workflow Initiative

Although not strictly a workflow initiative issue I'll tag it for visibility.

Manuel Garcia’s picture

Assigned: Unassigned » Manuel Garcia

Having a go at this...

Manuel Garcia’s picture

Assigned: Manuel Garcia » Unassigned
Status: Active » Needs review
FileSize
1.21 KB

Status: Needs review » Needs work

The last submitted patch, 4: 2876630-4.patch, failed testing.

Manuel Garcia’s picture

Status: Needs work » Needs review
timmillwood’s picture

Status: Needs review » Reviewed & tested by the community
Issue tags: -Needs tests

Looks good @Manuel Garcia, think that's all we need.

  • catch committed 11b40d7 on 8.4.x
    Issue #2876630 by Manuel Garcia: Add test coverage for saving a node...

  • catch committed f124e03 on 8.3.x
    Issue #2876630 by Manuel Garcia: Add test coverage for saving a node...
catch’s picture

Version: 8.4.x-dev » 8.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.4.x and cherry-picked to 8.3.x, thanks!

Status: Fixed » Closed (fixed)

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