This is a first pass of some of the base functionality for core aggregator.module. It incorporates feed adding, and feed item & feed updating and deleting. poetro has worked on the category areas and hopefully should be able to merge in those issues. Let us know if this seems to be moving in at least the right direction, and we can hopefully polish off the remainder. The more feedback you can provide the better. Thanks!

CommentFileSizeAuthor
#7 aggregator_module_test.txt8.83 KBcsevb10
aggregator_module_test.txt8.83 KBcsevb10

Comments

alex_b’s picture

alex_b commented

There are plans revamp aggregator for D7 - I will announce more soon. I just wanted to deposit this here as it might matter.

boombatower’s picture

boombatower commented
Status: Active » Needs work

I get a lot of errors when I run the test. You may want to check it over. I'm not sure if there have been changes to aggregator or what.

As a note make sure you write your tests for Drupal HEAD modules.

csevb10’s picture

csevb10 commented

@alex_b: I've always been in favor of revamping aggregator, so I was a little reluctant to do aggregator testing as a result, but it was the first one to be handed out, so until we swap out, I figure we'll keep pushing this one as far forward as possible.

@boombatower: Ok, I'll pull down the code, add the test to my local env, and see why things are in an error state, and then post a new file.

alex_b’s picture

alex_b commented

csevb10: I agree. For the record, Aron's D7 proposal: http://groups.drupal.org/node/9857

csevb10’s picture

csevb10 commented

alex_b: Yep. I read it already. No choice but to keep up on the aggregation world. :-)

csevb10’s picture

csevb10 commented

boombatower: Ok, we did it for the latest DRUPAL-6 branch release of Drupal core. We'll revise/rewrite for Drupal HEAD (i.e. D7).

csevb10’s picture

csevb10 commented
StatusFileSize
new aggregator_module_test.txt8.83 KB

Ok, here's an updated version that works on DRUPAL-6 & HEAD

boombatower’s picture

boombatower commented
Status: Needs work » Fixed

Committed.

As a note it threw exceptions when trying to execute:

$this->prefix

This is due to $prefix being defined as static:

private static $prefix = 'simpletest_aggregator_';

When accessing static properties use the following syntax:

self::$prefix

For more information: http://us3.php.net/manual/en/language.oop5.static.php.

Otherwise great test!
Thanks.

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

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