forked from zaclys/searxng
A little fix and modified the testing for content highlight
This commit is contained in:
parent
6c421110b5
commit
99b5272d9a
|
@ -124,7 +124,8 @@ def contains_cjko(s: str) -> bool:
|
|||
Returns:
|
||||
bool: True if the input s contains the characters and False otherwise.
|
||||
"""
|
||||
unicode_ranges = ('\u4e00-\u9fff' # Chinese characters
|
||||
unicode_ranges = (
|
||||
'\u4e00-\u9fff' # Chinese characters
|
||||
'\u3040-\u309f' # Japanese hiragana
|
||||
'\u30a0-\u30ff' # Japanese katakana
|
||||
'\u4e00-\u9faf' # Japanese kanji
|
||||
|
@ -168,6 +169,8 @@ def highlight_content(content, query):
|
|||
querysplit = query.split()
|
||||
queries = []
|
||||
for qs in querysplit:
|
||||
qs = qs.replace("'", "").replace('"', '').replace(" ", "")
|
||||
if len(qs) > 0:
|
||||
queries.extend(re.findall(regex_highlight_cjk(qs), content, flags=re.I | re.U))
|
||||
if len(queries) > 0:
|
||||
for q in set(queries):
|
||||
|
|
|
@ -28,32 +28,33 @@ class TestWebUtils(SearxTestCase):
|
|||
|
||||
content = 'a'
|
||||
query = 'test'
|
||||
self.assertEqual(webutils.highlight_content(content, query), content)
|
||||
self.assertEqual(webutils.highlight_content(content, query), 'a')
|
||||
query = 'a test'
|
||||
self.assertEqual(webutils.highlight_content(content, query), content)
|
||||
self.assertEqual(webutils.highlight_content(content, query), '<span class="highlight">a</span>')
|
||||
|
||||
data = (
|
||||
('" test "', 'a test string', 'a <span class="highlight">test</span> string'),
|
||||
('"a"', 'this is a test string', 'this is<span class="highlight"> a </span>test string'),
|
||||
('"a"', 'this is a test string', 'this is <span class="highlight">a</span> test string'),
|
||||
(
|
||||
'a test',
|
||||
'this is a test string that matches entire query',
|
||||
'this is <span class="highlight">a test</span> string that matches entire query',
|
||||
'this is <span class="highlight">a</span> <span class="highlight">test</span> string that matches entire query',
|
||||
),
|
||||
(
|
||||
'this a test',
|
||||
'this is a string to test.',
|
||||
(
|
||||
'<span class="highlight">this</span> is<span class="highlight"> a </span>'
|
||||
'string to <span class="highlight">test</span>.'
|
||||
'<span class="highlight">this</span> is <span class="highlight">a</span> string to <span class="highlight">test</span>.'
|
||||
),
|
||||
),
|
||||
(
|
||||
'match this "exact phrase"',
|
||||
'this string contains the exact phrase we want to match',
|
||||
(
|
||||
'<span class="highlight">this</span> string contains the <span class="highlight">exact</span>'
|
||||
' <span class="highlight">phrase</span> we want to <span class="highlight">match</span>'
|
||||
''.join(
|
||||
[
|
||||
'<span class="highlight">this</span> string contains the <span class="highlight">exact</span> ',
|
||||
'<span class="highlight">phrase</span> we want to <span class="highlight">match</span>',
|
||||
]
|
||||
),
|
||||
),
|
||||
)
|
||||
|
|
Loading…
Reference in New Issue