Fill in the tests to confirm that the node revision handling works properly.

CommentFileSizeAuthor
#7 panelizer-n2742609-7.interdiff.txt2.14 KBDamienMcKenna
#7 panelizer-n2742609-7.patch14.14 KBDamienMcKenna
#6 panelizer-n2742609-6.patch13.27 KBDamienMcKenna
#4 panelizer-n2742609-4.patch4.11 KBDamienMcKenna
#3 panelizer-n2742609-3.patch10.52 KBDamienMcKenna
#2 panelizer-n2742609-2.patch10.39 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna created an issue. See original summary.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
panelizer-n2742609-2.patch10.39 KB

Started by using the page_manager view mode for the tests.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2742609-3.patch10.52 KB

Added assertions for the default display.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2742609-4.patch4.11 KB

This adds some assertions for checking the revisions when revisions are deleted or reverted, but it needs to be further checked to verify the {panelizer_entity} records.

Status: Needs review » Needs work

The last submitted patch, 4: panelizer-n2742609-4.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
panelizer-n2742609-6.patch13.27 KB

#4 was actually an interdiff, here's the full test.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
panelizer-n2742609-7.patch14.14 KB
panelizer-n2742609-7.interdiff.txt2.14 KB

Confirm the number of records exist when revisions are deleted or reverted.

  • DamienMcKenna committed 7623014 on 7.x-3.x 
    Issue #2742609 by DamienMcKenna: Added tests for core node revision...
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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