A good module needs some tests to ensure it works correctly. Components/actions to test could be:
- CRUDing indexes and servers via the UI.
- Indexing, clearing, reindexing content on an index (also via UI).
- Query parsing.
- Db search indexing, searching, and administration tasks (adding/removing indexes, …).

CommentFileSizeAuthor
#5 872912-5--testing.patch647 bytesdrunken monkey
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

drunken monkey’s picture

Status: Active » Needs work

All tests for the API/framework are written and cover the thing pretty well, I think. Also, everything seems to be alright, I'm just waiting for some results in the following issues for two specific problems:
#883264: Testing objects with unit tests
#883286: Messages (via drupal_set_message()) don't appear for one single page requests
However, the test cases themselves show no (more) errors in the module.

Still needed are tests for the search_api_db module.

drunken monkey’s picture

Update: This is heavily out of sync with the rest of the module, tons of tests fail. Will have to change that before considering a final 1.0 release. Already worked a bit on it, but am now stuck on some database search test, which might be a real bug in the database search module.

Also, all four "new" sub-modules still need tests.

drunken monkey’s picture

I finally got down to at least updating and fixing the tests that are already there, according to all the API and UI changes.
Some tests still fail, though, due to the following reasons:
- #946286: Multi-term fulltext filter results in exception
- #1026496: status doesn't get set properly when creating entities
- #1007830: Nested transactions throw exceptions when they got out of scope
- Seemingly some bug in the database backend that I haven't completely fleshed out, yet.
Probably only two of these are my fault, at least …

Still "needs work" because there are still three sub-modules without any tests.

drunken monkey’s picture

Since #1026496: status doesn't get set properly when creating entities got fixed (and I quickly fixed a bug that only showed itself as a result of that fix), now only some of the database backend tests fail. Hooray!

drunken monkey’s picture

Component: Code » Framework
Status: Needs work » Needs review
FileSize
647 bytes

#1860028-3: SQLSTATE[42703]: Undefined column: ### ERROR: column t.word does not exist at character ###. might very well finally make all DB backend tests pass! Hooray!

I even expanded the tests a bit (and, admittedly, had to fix one problem). See attached patch.

drunken monkey’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)
Issue tags: -gsoc, -gsoc2010, -gsoc2010-drunken_monkey

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