I went to run the SimpleTest automated tests on FeedAPI 1.4 (specifically for FeedAPI Node) and they do not appear, so cannot be run. Makes it impossible to aid in debugging this issue:
http://drupal.org/node/260670#comment-969353

Other people mentioning this:
http://drupal.org/node/260670#comment-970296
http://drupal.org/node/260670#comment-983806

Looks like a bug! Drupal 6.6 and SimpleTest 6.x-2.5. Other modules provide tests without issue.

CommentFileSizeAuthor
#9 326637_feedapi_node.module.test.simpletest2x.patch7.81 KBaron novak
#6 feedapi_node.module.test.simpletest2x.patch1.25 KBlyricnz
#5 feedapi_node.module.test.simpletest2x.patch1.3 KBlyricnz

Comments

Arto’s picture

Arto commented
Title: SimpleTest tests are not available in admin/build/testing » Support SimpleTest 2.x
Category: bug » feature
Priority: Critical » Normal

SimpleTest 2.x broke compatibility with many older unit tests, hence why this is not showing up. Somebody needs to refactor the tests and submit a patch.

Arto’s picture

Arto commented

(In other words, if you downgrade to SimpleTest 6.x-1.x, the FeedAPI node unit tests should show up just fine.)

Arto’s picture

Arto commented
Title: Support SimpleTest 2.x » Support Simpletest 2.x
greg.harvey’s picture

greg.harvey
Primary language English
Location Occitanie, France
commented

Thanks - I don't think I'll have time to write the patch, I'm afraid... too much on at the moment. But I'll use SimpleTest 1.x to test the above issue. =)

lyricnz’s picture

lyricnz commented
Status: Active » Needs review
StatusFileSize
new feedapi_node.module.test.simpletest2x.patch1.3 KB

Here's a patch for feedapi_node.module.test that updates for SimpleTest 6.x-2.x. Right now it delivers

110 passes, 0 fails, and 2 exceptions

Those two exceptions are caused by #449198: SimpleTest: Clean up test loading and related API which is waiting for a backport from D7.

lyricnz’s picture

lyricnz commented
StatusFileSize
new feedapi_node.module.test.simpletest2x.patch1.25 KB

Update patch to remove legacy get_info()

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Needs review » Needs work

I received this:

Notice: Undefined variable: classes in simpletest_get_all_tests() (line 504 of /var/www/html/drupal-6.9/sites/all/modules/simpletest/simpletest.module).
No test cases found.
No test(s) selected.

I tested with SimpleTest 6.x-2.8 and 6.x-2.x-dev also.
Any guess what's going on?

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Version: 6.x-1.4 » 6.x-1.x-dev

I renamed the class to FeedAPITestsCase and other, more serious errors appeared (now the processing of the test starts)

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Needs work » Needs review
StatusFileSize
new 326637_feedapi_node.module.test.simpletest2x.patch7.81 KB

lyricnz, can you confirm that the patch below works sane at your test site as well?
There are some ugly exceptions, but overall, all the tests should pass.

lyricnz’s picture

lyricnz commented

Reply to #7; That warning is the one I refer to above : fixed in HEAD waiting for a backport to D6 simpletest. I don't know why it didn't find your tests, my "110 passes, 0 fails, and 2 exceptions" was cut'n'paste from the testing output.

lyricnz’s picture

lyricnz commented
Status: Needs review » Reviewed & tested by the community

Reply to #9: 339 passes, 0 fails, and 0 exceptions
(simpletest 6.x-2.8)

aron novak’s picture

aron novak
Primary language Hungarian
Location Hungary, Budapest
commented
Status: Reviewed & tested by the community » Fixed

lyricnz: thank you for the test, i committed the patch.

Status: Fixed » Closed (fixed)

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