This file needs more tests, this is a follow-up issue of #276493: Tests needed: aggregator.module.

Comments

alex_b’s picture

alex_b commented

Specifically, these are the open issues from #276493: Tests needed: aggregator.module :

1) aggregator_element_data().
2) Update with no new syndicated content.
3) Update to a moved feed.
4) Make sure lines 607-609 use theme('image') instead of other stupid stuff.
5) Update to unparsable feeds.
6) aggregator_parse_w3cdtf()
7) Trying to use aggregator_parse_feed() to parse invalid xml.

mustafau’s picture

mustafau commented

I have uploaded some tests along with the patch here; #130344: ATOM feed items link to the current page rather than back to the original source

catch’s picture

catch
he/him
Primary language English
commented
Priority: Critical » Normal

Moving this out of the critical queue, see #607038: Meta issue: fix gaps in code coverage.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Version: 7.x-dev » 8.x-dev
jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Issue summary: View changes
Status: Active » Closed (won't fix)

aggregator.parser.inc is no more.