Support from Acquia helps fund testing for Drupal Acquia logo

Comments

becw created an issue. See original summary.

becw’s picture

Assigned: Unassigned » becw

  • becw committed ef1e44c on save-destination-2645606
    refs #2645606: redirect to latest_version tab after saving a forward...
becw’s picture

  • becw committed 971b7af on save-destination-2645606
    refs #2645606: add a test for edit form destinations.
    

The last submitted patch, 6: ux_for_saving_drafts-2645606-6.patch, failed testing.

The last submitted patch, 6: ux_for_saving_drafts-2645606-6.patch, failed testing.

The last submitted patch, 6: ux_for_saving_drafts-2645606-6.patch, failed testing.

Status: Needs review » Needs work

The last submitted patch, 6: ux_for_saving_drafts-2645606-6-test-only.patch, failed testing.

The last submitted patch, 6: ux_for_saving_drafts-2645606-6-test-only.patch, failed testing.

The last submitted patch, 6: ux_for_saving_drafts-2645606-6-test-only.patch, failed testing.

becw’s picture

I am stumped as to why, after submitting the node add form, the test still thinks it is at the path 'user/2'; at this point in manual testing, the form redirects to the node view page.

fail: [Other] Line 73 of modules/workbench_moderation/src/Tests/ModerationStateNodeTest.php:
Expected 'http://localhost/checkout/node/1' matches current URL (http://localhost/checkout/user/2).
Value 'http://localhost/checkout/user/2' is equal to value 'http://localhost/checkout/node/1'.

fail: [Other] Line 74 of modules/workbench_moderation/src/Tests/ModerationStateNodeTest.php:
"First version of the content." found

  • becw committed 031304f on save-destination-2645606
    refs #2645606: redirect to latest_version tab after saving a forward...
  • becw committed 27c8d76 on save-destination-2645606
    refs #2645606: add a test for edit form destinations.
    
becw’s picture

Status: Needs review » Needs work

The last submitted patch, 15: ux_for_saving_drafts-2645606-14-test-only.patch, failed testing.

The last submitted patch, 15: ux_for_saving_drafts-2645606-14-test-only.patch, failed testing.

The last submitted patch, 15: ux_for_saving_drafts-2645606-14-test-only.patch, failed testing.

becw’s picture

Disabling the test-only patch so that the issue shows as needs review.

Crell’s picture

Merged. Thanks!

  • becw committed 031304f on 8.x-1.x
    refs #2645606: redirect to latest_version tab after saving a forward...
  • becw committed 27c8d76 on 8.x-1.x
    refs #2645606: add a test for edit form destinations.
    
Crell’s picture

Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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