on master help sign in
[fix] duckduckgo unicode url - #419
commit4184cece4a0b2d04b45105e755492bfee5fa1a12 authorAdam Tauber <asciimoo@gmail.com> dateMon 7 Sep 2015, 23:13 (11 years ago) parent362c849
2 files changed+5−4
searx/engines/duckduckgo.py+1−1 searx/tests/engines/test_duckduckgo.py+4−3
searx/engines/duckduckgo.py+1−1
@@ -72,7 +72,7 @@ def response(resp):
7272 # append result
7373 results.append({'title': title,
7474 'content': content,
75− 'url': res_url.encode('utf8')})
75+ 'url': res_url})
7676
7777 # return results
7878 return results
searx/tests/engines/test_duckduckgo.py+4−3
@@ -1,3 +1,4 @@
1+# -*- coding: utf-8 -*-
12from collections import defaultdict
23import mock
34from searx.engines import duckduckgo
@@ -30,7 +31,7 @@ class TestDuckduckgoEngine(SearxTestCase):
3031 response = mock.Mock(text='<html></html>')
3132 self.assertEqual(duckduckgo.response(response), [])
3233
33− html = """
34+ html = u"""
3435 <div class="results_links results_links_deep web-result">
3536 <div class="icon_fav" style="display: block;">
3637 <a rel="nofollow" href="https://www.test.com/">
@@ -39,7 +40,7 @@ class TestDuckduckgoEngine(SearxTestCase):
3940 </a>
4041 </div>
4142 <div class="links_main links_deep"> <!-- This is the visible part -->
42− <a rel="nofollow" class="large" href="http://this.should.be.the.link/">
43+ <a rel="nofollow" class="large" href="http://this.should.be.the.link/ű">
4344 This <b>is</b> <b>the</b> title
4445 </a>
4546 <div class="snippet"><b>This</b> should be the content.</div>
@@ -54,7 +55,7 @@ class TestDuckduckgoEngine(SearxTestCase):
5455 self.assertEqual(type(results), list)
5556 self.assertEqual(len(results), 1)
5657 self.assertEqual(results[0]['title'], 'This is the title')
57− self.assertEqual(results[0]['url'], 'http://this.should.be.the.link/')
58+ self.assertEqual(results[0]['url'], u'http://this.should.be.the.link/ű')
5859 self.assertEqual(results[0]['content'], 'This should be the content.')
5960
6061 html = """