Add tests to ensure that the node editorial process still works, i.e. revisions & deletions.

CommentFileSizeAuthor
#6 metatag-n2514916-6.patch61.18 KBDamienMcKenna
#5 metatag-n2514916-5.patch58.5 KBDamienMcKenna
#4 metatag-n2514916-4.patch55.73 KBDamienMcKenna
#3 metatag-n2514916-3.patch55.01 KBDamienMcKenna
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna commented
Parent issue: » #2505213: Plan for Metatag 7.x-1.7 release
DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Parent issue: #2505213: Plan for Metatag 7.x-1.7 release »

I'll do this later.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Active » Needs review
FileSize
metatag-n2514916-3.patch55.01 KB

This splits out each class into its own file, renames them, and adds some extra items to the node test to have some revision testing. It needs more work, but it's a start.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
metatag-n2514916-4.patch55.73 KB

This adds some more tests to confirm that the revisions page loads when appropriate, and clarifies the old-vs-new titles.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
metatag-n2514916-5.patch58.5 KB

This gets into reverting & deleting revisions. Woot!

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
FileSize
metatag-n2514916-6.patch61.18 KB

Forgot to confirm that the meta tags had correctly changed after the revisions were fiddled with. And some other things.

  • DamienMcKenna committed 34162b5 on 7.x-1.x 
    Issue #2514916 by DamienMcKenna: Split up the tests file, moved them...
DamienMcKenna’s picture

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

Committed! Yay!

Status: Fixed » Closed (fixed)

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