After #2613654: Parse image fields for image URLs was added, we need to add tests to make it clear how image fields should work.

CommentFileSizeAuthor
#6 metatag-n2628472-6.patch19.76 KBDamienMcKenna
#4 metatag-n2628472-4.patch19.87 KBDamienMcKenna
#2 metatag-n2628472-2.patch20.42 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
metatag-n2628472-2.patch20.42 KB

Moving the tests over from #2761909: og:image does not parse url from image field properly when there is a comma in the alt field. I'm expecting this to not work.

Status: Needs review » Needs work

The last submitted patch, 2: metatag-n2628472-2.patch, failed testing.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
metatag-n2628472-4.patch19.87 KB

Updated to be a BrowserTest.

Status: Needs review » Needs work

The last submitted patch, 4: metatag-n2628472-4.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented
Status: Needs work » Needs review
FileSize
metatag-n2628472-6.patch19.76 KB

WIP.

Status: Needs review » Needs work

The last submitted patch, 6: metatag-n2628472-6.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

DamienMcKenna’s picture

DamienMcKenna
Location NH, USA
CreditAttribution: DamienMcKenna at Mediacurrent commented

Also expand the test for multiple items to confirm what happens when there's a blank item.