makary/phpbb
1
0
Fork 0
mirror of https://github.com/phpbb/phpbb.git synced 2025-06-10 13:28:55 +00:00
Code Issues Projects Releases Packages Wiki Activity

[ticket/17365] Add a test for max search keywords

Browse source 
PHPBB-17365
This commit is contained in:
Marc Alexander 2024-07-10 21:32:15 +02:00
parent 30144052da
commit c0b81a1a48
No known key found for this signature in database
GPG key ID: 50E0D2423696F995
1 changed files with 80 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

81
tests/search/native_test.php
View file

@ -25,19 +25,24 @@ class phpbb_search_native_test extends phpbb_search_test_case
protected function setUp(): void
{
global $phpbb_root_path, $phpEx, $config, $user, $cache;
global $phpbb_root_path, $phpEx, $config, $cache;
parent::setUp();
// dbal uses cache
$cache = new phpbb_mock_cache();
$lang_loader = new \phpbb\language\language_file_loader($phpbb_root_path, $phpEx);
$lang = new \phpbb\language\language($lang_loader);
$user = new \phpbb\user($lang, '\phpbb\datetime');;
$this->db = $this->new_dbal();
$phpbb_dispatcher = new phpbb_mock_event_dispatcher();
$error = null;
$class = self::get_search_wrapper('\phpbb\search\fulltext_native');
$config['fulltext_native_min_chars'] = 2;
$config['fulltext_native_max_chars'] = 14;
$config['max_num_search_keywords'] = 10;
$this->search = new $class($error, $phpbb_root_path, $phpEx, null, $config, $this->db, $user, $phpbb_dispatcher);
}
@ -259,4 +264,78 @@ class phpbb_search_native_test extends phpbb_search_test_case
}
$this->assert_array_content_equals($common, $this->search->get_common_words());
}
public function data_split_keywords_max(): array
{
return [
'character count within limits separated by more spaces' => [
'foo bar baz boo far faz roo rar raz zoo',
'all',
false,
],
'character count within limits separated by spaces' => [
'foo bar baz boo far faz roo rar raz zoo',
'all',
false,
],
'character count within limits separated by +, spaces after +' => [
'foo+ bar+ baz+ boo+ far+ faz+ roo+ rar+ raz+ zoo',
'all',
false,
],
'character count within limits separated by +, no spaces' => [
'foo+bar+baz+boo+far+faz+roo+rar+raz+zoo',
'all',
false,
],
'character count outside limits separated by +, no spaces' => [
'foo+bar+baz+boo+far+faz+roo+rar+raz+zoo+zar',
'all',
true,
],
'character count outside limits separated by + and spaces' => [
'foo +bar +baz +boo +far +faz +roo +rar +raz +zoo +zar',
'all',
true,
],
'character count outside limits separated by spaces' => [
'foo bar baz boo far faz roo rar raz zoo zar',
'all',
true,
],
'character count outside limits separated by -, no spaces' => [
'foo-bar-baz-boo-far-faz-roo-rar-raz-zoo-zar',
'all',
true,
],
'character count outside limits separated by - and spaces' => [
'foo -bar -baz -boo -far -faz -roo -rar -raz -zoo -zar',
'all',
true,
],
'character count outside limits separated by |, no spaces' => [
'foo|bar|baz|boo|far|faz|roo|rar|raz|zoo|zar',
'all',
true,
],
'character count outside limits separated by | and spaces' => [
'foo |bar |baz |boo |far |faz |roo |rar |raz |zoo |zar',
'all',
true,
],
];
}
/**
* @dataProvider data_split_keywords_max
*/
public function test_split_max_keywords($keywords, $terms, $expect_error)
{
if ($expect_error)
{
$this->setExpectedTriggerError(E_USER_NOTICE, 'MAX_NUM_SEARCH_KEYWORDS_REFINE');
}
$this->assertTrue($this->search->split_keywords($keywords, $terms));
}
}