https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42540
--- Comment #23 from David Cook <[email protected]> --- (In reply to Pedro Amorim (ammopt) from comment #20) > Hi guys, Search.t fails for me running es8: > > prove t/db_dependent/Koha/SearchEngine/Elasticsearch/Search.t > > Can or should we make this test not rely on ElasticSearch version? Thanks for providing that output. I see what you mean now... in Elasticsearch 7 there was a setting indices.query.bool.max_clause_count which defaulted to 1024. But that setting was deprecated in Elasticsearch 8 and the limit gets set dynamically... https://www.elastic.co/guide/en/elasticsearch/reference/7.17/search-settings.html https://www.elastic.co/docs/reference/elasticsearch/configuration-reference/search-settings So... we'll need to mock the exception... probably using a mock for Search::Elasticsearch or a mock function for Koha::SearchEngine::Elasticsearch->get_elasticsearch so that we get an object with a search method that reliably throws the exception when running '$searcher->search($query)'. Nice one, Pedro. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list -- [email protected] To unsubscribe send an email to [email protected] website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
