Merge pull request #323 from arXiv/ARXIVCE-837-repair-test-cases

Arxivce 837 repair test cases

.github/workflows/python-app.yml

@@ -0,0 +1,39 @@
+# This workflow will install Python dependencies, run tests and lint with a single version of Python
+# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python
+
+name: Python application
+
+on:
+  push:
+    branches: [ "develop" ]
+  pull_request:
+    branches: [ "develop" ]
+
+permissions:
+  contents: read
+
+jobs:
+  build:
+
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Python 3.10
+      uses: actions/setup-python@v3
+      with:
+        python-version: "3.10.9"
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install flake8 pytest poetry
+        poetry install
+    #- name: Lint with flake8
+    #  run: |
+    #    # stop the build if there are Python syntax errors or undefined names
+    #    flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics
+    #    # exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide
+    #    flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics
+    - name: Test with pytest
+      run: |
+        poetry run pytest

README.md

@@ -234,6 +234,11 @@ E.g.
 WITH_INTEGRATION=1 pipenv run nose2 --with-coverage
 ```
 
+You can also run to avoid all intentional error messages or kinesis print statements created during the tests
+```bash
+pytest --disable-warnings search
+```
+
 ### Static checking
 Goal: zero errors/warnings.
 

search/controllers/tests.py

@@ -14,20 +14,16 @@ class TestHealthCheck(TestCase):
     def test_index_is_down(self, mock_index):
         """Test returns 'DOWN' + status 500 when index raises an exception."""
         mock_index.search.side_effect = RuntimeError
-        response, status_code, _ = health_check()
-        self.assertEqual(response, "DOWN", "Response content should be DOWN")
-        self.assertEqual(
-            status_code,
-            HTTPStatus.INTERNAL_SERVER_ERROR,
-            "Should return 500 status code.",
-        )
+
+        with self.assertRaises(RuntimeError):
+            response, status_code, _ = health_check()
 
     @mock.patch("search.controllers.index.SearchSession")
     def test_index_returns_no_result(self, mock_index):
         """Test returns 'DOWN' + status 500 when index returns no results."""
         mock_index.search.return_value = {"metadata": {}, "results": []}
         response, status_code, _ = health_check()
-        self.assertEqual(response, "DOWN", "Response content should be DOWN")
+        self.assertEqual(response, "DOWN: document_set lacked results", "Response content should be DOWN: document_set lacked results")
         self.assertEqual(
             status_code,
             HTTPStatus.INTERNAL_SERVER_ERROR,

search/factory.py

@@ -49,7 +49,8 @@ def create_ui_web_app() -> Flask:
     for filter_name, template_filter in filters.filters:
         app.template_filter(filter_name)(template_filter)
 
-    index_startup_check(app)
+    if app.config["TESTING"]:
+        index_startup_check(app)
     return app
 
 

search/tests/test_param_persistence.py

@@ -41,7 +41,7 @@ def test_request_does_not_include_params(self):
         self.assertIn("Set-Cookie", response.headers, "Should set a cookie")
         self.assertEqual(
             response.headers["Set-Cookie"],
-            'foo-cookie="{}"; Path=/',
+            'foo-cookie={}; Path=/',
             "Cookie should not contain request params",
         )
 
@@ -52,7 +52,7 @@ def test_request_includes_cookie(self, mock_simple):
         ui.PARAMS_TO_PERSIST = ["foo", "baz"]
         ui.PARAMS_COOKIE_NAME = "foo-cookie"
         self.client.set_cookie(
-            "", ui.PARAMS_COOKIE_NAME, json.dumps({"foo": "ack"})
+            ui.PARAMS_COOKIE_NAME, json.dumps({"foo": "ack"})
         )
         self.client.get("/")
         self.assertEqual(
@@ -68,7 +68,7 @@ def test_request_includes_cookie_but_also_explicit_val(self, mock_simple):
         ui.PARAMS_TO_PERSIST = ["foo", "baz"]
         ui.PARAMS_COOKIE_NAME = "foo-cookie"
         self.client.set_cookie(
-            "", ui.PARAMS_COOKIE_NAME, json.dumps({"foo": "ack"})
+            ui.PARAMS_COOKIE_NAME, json.dumps({"foo": "ack"})
         )
         self.client.get("/?foo=oof")
         self.assertEqual(

search/tests/test_searches.py

@@ -3,7 +3,7 @@
 from unittest import TestCase, mock
 
 from search.factory import create_ui_web_app
-
+from search.services.index import QueryError
 
 class TestSearchs(TestCase):
     """Test for the advanced search UI."""
@@ -13,8 +13,10 @@ def setUp(self):
         self.app = create_ui_web_app()
         self.client = self.app.test_client()
 
-    def test_bad_query(self):
+    @mock.patch("search.controllers.simple.SearchSession")
+    def test_bad_query(self, mock_index):
         """Bad query should result in a 400 not a 500. query from ARXIVNG-2437"""
+        mock_index.search.side_effect = QueryError
         response = self.client.get("/?query=%2F%3F&searchtype=all&source=header")
         self.assertEqual(
             response.status_code,