Page MenuHomePhabricator

Fix some integration tests
Closed, ResolvedPublic


Some cirrus integration tests are randomly failing but recently one is consistently failing.

  • clean_api_update_deleted_redirects_are_removed_from_the_index fails consistently since few days
  • clean_api_prefix_redirect_accent_squashing_accented_namespace_suggest_search_prefers_main_namespace_over_crossns_redirects fails randomly since the beginning

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

clean_api_update_deleted_redirects_are_removed_from_the_index can be explained because of

clean_api_prefix_redirect_accent_squashing_accented_namespace_suggest_search_prefers_main_namespace_over_crossns_redirects seems due to a instability in the scoring function:

[2018-07-13 14:20:41+00:00] AssertionError: expected 'V:N' to equal 'Venom'
[2018-07-13 14:20:41+00:00] Last Api:
[2018-07-13 14:20:41+00:00] Extra: ["V",["V:N","Venom"],["",""],["",""]]

Both V:N and Venom appears in the resultset but in the wrong order, obviously testing locally I can't figure out what's happening. The comp suggest scores is 11627 for V:N and Venom 1216141 (100 times more). I wonder what situation causes these scores to change so much when the test fails...

Change 446857 had a related patch set uploaded (by DCausse; owner: DCausse):
[mediawiki/extensions/CirrusSearch@master] Fix integration for V => Venom > V:N

Change 446857 merged by jenkins-bot:
[mediawiki/extensions/CirrusSearch@master] Fix integration tests for V => Venom > V:N