 bb04699b17
			
		
	
	
		bb04699b17
		
	
	
	
	
		
			
			Depending on the order the unit tests are executed, the searx.search module is
initalized or not, issue reported in [1]::
    Traceback (most recent call last):
      File "searxng/tests/unit/test_results.py", line 72, in test_result_merge_by_title
        self.container.extend('stract', [fake_result(engine='stract', title='short title')])
      File "searxng/searx/results.py", line 243, in extend
        histogram_observe(standard_result_count, 'engine', engine_name, 'result', 'count')
      File "searxng/searx/metrics/__init__.py", line 49, in histogram_observe
        histogram_storage.get(*args).observe(duration)
        ^^^^^^^^^^^^^^^^^^^^^
      AttributeError: 'NoneType' object has no attribute 'get'
To ensure that the searx.search module is initialized, the
- searx.engines.load_engines is replace by
- searx.search.initialize
[1] https://github.com/searxng/searxng/pull/3932#discussion_r1822406569
Signed-off-by: Markus Heiser <markus.heiser@darmarit.de>
		
	
			
		
			
				
	
	
		
			101 lines
		
	
	
		
			3.8 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
			
		
		
	
	
			101 lines
		
	
	
		
			3.8 KiB
		
	
	
	
		
			Python
		
	
	
	
	
	
| # SPDX-License-Identifier: AGPL-3.0-or-later
 | |
| # pylint: disable=missing-module-docstring
 | |
| 
 | |
| import logging
 | |
| from datetime import datetime
 | |
| from unittest.mock import Mock
 | |
| from requests import HTTPError
 | |
| from parameterized import parameterized
 | |
| import searx.search
 | |
| import searx.engines
 | |
| from tests import SearxTestCase
 | |
| 
 | |
| 
 | |
| class TinEyeTests(SearxTestCase):  # pylint: disable=missing-class-docstring
 | |
| 
 | |
|     def setUp(self):
 | |
|         searx.search.initialize(
 | |
|             [{'name': 'tineye', 'engine': 'tineye', 'shortcut': 'tin', 'timeout': 9.0, 'disabled': True}]
 | |
|         )
 | |
| 
 | |
|         self.tineye = searx.engines.engines['tineye']
 | |
|         self.tineye.logger.setLevel(logging.CRITICAL)
 | |
| 
 | |
|     def tearDown(self):
 | |
|         searx.search.load_engines([])
 | |
| 
 | |
|     def test_status_code_raises(self):
 | |
|         response = Mock()
 | |
|         response.status_code = 401
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
|         self.assertRaises(HTTPError, lambda: self.tineye.response(response))
 | |
| 
 | |
|     @parameterized.expand([(400), (422)])
 | |
|     def test_returns_empty_list(self, status_code):
 | |
|         response = Mock()
 | |
|         response.json.return_value = {}
 | |
|         response.status_code = status_code
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
|         results = self.tineye.response(response)
 | |
|         self.assertEqual(0, len(results))
 | |
| 
 | |
|     def test_logs_format_for_422(self):
 | |
|         response = Mock()
 | |
|         response.json.return_value = {"suggestions": {"key": "Invalid image URL"}}
 | |
|         response.status_code = 422
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
| 
 | |
|         with self.assertLogs(self.tineye.logger) as assert_logs_context:
 | |
|             self.tineye.response(response)
 | |
|             self.assertIn(self.tineye.FORMAT_NOT_SUPPORTED, ','.join(assert_logs_context.output))
 | |
| 
 | |
|     def test_logs_signature_for_422(self):
 | |
|         response = Mock()
 | |
|         response.json.return_value = {"suggestions": {"key": "NO_SIGNATURE_ERROR"}}
 | |
|         response.status_code = 422
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
| 
 | |
|         with self.assertLogs(self.tineye.logger) as assert_logs_context:
 | |
|             self.tineye.response(response)
 | |
|             self.assertIn(self.tineye.NO_SIGNATURE_ERROR, ','.join(assert_logs_context.output))
 | |
| 
 | |
|     def test_logs_download_for_422(self):
 | |
|         response = Mock()
 | |
|         response.json.return_value = {"suggestions": {"key": "Download Error"}}
 | |
|         response.status_code = 422
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
| 
 | |
|         with self.assertLogs(self.tineye.logger) as assert_logs_context:
 | |
|             self.tineye.response(response)
 | |
|             self.assertIn(self.tineye.DOWNLOAD_ERROR, ','.join(assert_logs_context.output))
 | |
| 
 | |
|     def test_logs_description_for_400(self):
 | |
|         description = 'There was a problem with that request. Error ID: ad5fc955-a934-43c1-8187-f9a61d301645'
 | |
|         response = Mock()
 | |
|         response.json.return_value = {"suggestions": {"description": [description], "title": "Oops! We're sorry!"}}
 | |
|         response.status_code = 400
 | |
|         response.raise_for_status.side_effect = HTTPError()
 | |
| 
 | |
|         with self.assertLogs(self.tineye.logger) as assert_logs_context:
 | |
|             self.tineye.response(response)
 | |
|             self.assertIn(description, ','.join(assert_logs_context.output))
 | |
| 
 | |
|     def test_crawl_date_parses(self):
 | |
|         date_str = '2020-05-25'
 | |
|         date = datetime.strptime(date_str, '%Y-%m-%d')
 | |
|         response = Mock()
 | |
|         response.json.return_value = {
 | |
|             'matches': [
 | |
|                 {
 | |
|                     'backlinks': [
 | |
|                         {
 | |
|                             'crawl_date': date_str,
 | |
|                         }
 | |
|                     ]
 | |
|                 }
 | |
|             ]
 | |
|         }
 | |
|         response.status_code = 200
 | |
|         results = self.tineye.response(response)
 | |
|         self.assertEqual(date, results[0]['publishedDate'])
 |