SimpleTest for Aggregator.module
csevb10 - March 8, 2008 - 01:19
| Project: | SimpleTest |
| Version: | 6.x-1.x-dev |
| Component: | Code |
| Category: | task |
| Priority: | normal |
| Assigned: | Unassigned |
| Status: | closed |
Description
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!
| Attachment | Size |
|---|---|
| aggregator_module_test.txt | 8.83 KB |
#1
There are plans revamp aggregator for D7 - I will announce more soon. I just wanted to deposit this here as it might matter.
#2
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.
#3
@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.
#4
csevb10: I agree. For the record, Aron's D7 proposal: http://groups.drupal.org/node/9857
#5
alex_b: Yep. I read it already. No choice but to keep up on the aggregation world. :-)
#6
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).
#7
Ok, here's an updated version that works on DRUPAL-6 & HEAD
#8
Committed.
As a note it threw exceptions when trying to execute:
$this->prefixThis is due to $prefix being defined as static:
private static $prefix = 'simpletest_aggregator_';When accessing static properties use the following syntax:
self::$prefixFor more information: http://us3.php.net/manual/en/language.oop5.static.php.
Otherwise great test!
Thanks.
#9
Automatically closed -- issue fixed for two weeks with no activity.