https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=42540 --- Comment #23 from David Cook <dcook@prosentient.com.au> --- (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... https://www.elastic.co/docs/reference/elasticsearch/configuration-reference/... 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.