#2179081: [meta] Inventory search.module tests. File issues to address test holes. identified a missing test that needs to be created for the Search module API:

We need a test that the search_reindex() function can successfully be used to clear the index entry for a single node.

We could test some other combinations of inputs too. The "clear all" functionality is implicitly tested already via a UI test in SearchConfigSettingsFormTest.

Comments

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon

I think this can best be done in the Multilingual test (which is testing other aspects of cron/index/throttle and has some convenient nodes for testing this) so I'll take care of it in conjunction with #2178643: Indexing status can show a negative percentage of content as having been indexed, which I'm working on now.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Active » Postponed

Those tests were added to the other issue, so postponing this one until that one is decided (if it is rejected we still need tests for this issue added).

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented

We also need to postpone this on #2211241: Refactor search_reindex() into separate functions which will most likely take care of the remaining necessary tests.

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Status: Postponed » Closed (duplicate)

Definitely taking care of this on the other issue.