Skip to content

Mc issue 352 search endpoint #135

Mc issue 352 search endpoint

Mc issue 352 search endpoint #135

Triggered via pull request July 19, 2024 00:57
Status Failure
Total duration 3m 6s
Artifacts

pull.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

9 errors and 66 warnings
[flake8] tests/fixtures.py#L133 <302>: tests/fixtures.py#L133
expected 2 blank lines, found 1
Lint API
Process completed with exit code 1.
[flake8] utilities/enums.py#L3 <302>: utilities/enums.py#L3
expected 2 blank lines, found 1
[flake8] middleware/search_logic.py#L1 <100>: middleware/search_logic.py#L1
Missing docstring in public module
[flake8] middleware/search_logic.py#L7 <401>: middleware/search_logic.py#L7
'database_client.result_formatter.ResultFormatter' imported but unused
[flake8] middleware/search_logic.py#L11 <103>: middleware/search_logic.py#L11
Missing docstring in public function
[flake8] resources/Search.py#L1 <100>: resources/Search.py#L1
Missing docstring in public module
[flake8] resources/Search.py#L53 <205>: resources/Search.py#L53
1 blank line required between summary line and description
[flake8] resources/Search.py#L53 <400>: resources/Search.py#L53
First line should end with a period
[flake8] resources/Search.py#L65 <401>: resources/Search.py#L65
First line should be in imperative mood
[flake8] resources/resource_helpers.py#L32 <103>: resources/resource_helpers.py#L32
Missing docstring in public function
[flake8] tests/fixtures.py#L135 <200>: tests/fixtures.py#L135
One-line docstring should fit on one line with quotes
[flake8] tests/fixtures.py#L135 <401>: tests/fixtures.py#L135
First line should be in imperative mood
[flake8] tests/fixtures.py#L143 <391>: tests/fixtures.py#L143
blank line at end of file
[flake8] tests/integration/test_search.py#L2 <401>: tests/integration/test_search.py#L2
'tests.fixtures.client_with_db' imported but unused
[flake8] tests/integration/test_search.py#L2 <401>: tests/integration/test_search.py#L2
'tests.fixtures.dev_db_connection' imported but unused
[flake8] tests/integration/test_search.py#L2 <401>: tests/integration/test_search.py#L2
'tests.fixtures.bypass_api_required' imported but unused
[flake8] tests/integration/test_search.py#L4 <811>: tests/integration/test_search.py#L4
redefinition of unused 'client_with_db' from line 2
[flake8] tests/integration/test_search.py#L4 <811>: tests/integration/test_search.py#L4
redefinition of unused 'bypass_api_required' from line 2
[flake8] tests/resources/test_Search.py#L3 <401>: tests/resources/test_Search.py#L3
'unittest.mock.MagicMock' imported but unused
[flake8] tests/resources/test_Search.py#L6 <401>: tests/resources/test_Search.py#L6
'tests.fixtures.client_with_mock_db' imported but unused
[flake8] tests/resources/test_Search.py#L6 <401>: tests/resources/test_Search.py#L6
'tests.fixtures.bypass_api_required' imported but unused
[flake8] tests/resources/test_Search.py#L26 <811>: tests/resources/test_Search.py#L26
redefinition of unused 'client_with_mock_db' from line 6
[flake8] tests/resources/test_Search.py#L26 <100>: tests/resources/test_Search.py#L26
Unused argument 'bypass_api_required'
[flake8] tests/resources/test_Search.py#L26 <811>: tests/resources/test_Search.py#L26
redefinition of unused 'bypass_api_required' from line 6
[flake8] tests/resources/test_Search.py#L52 <811>: tests/resources/test_Search.py#L52
redefinition of unused 'client_with_mock_db' from line 6
[flake8] tests/resources/test_Search.py#L52 <100>: tests/resources/test_Search.py#L52
Unused argument 'bypass_api_required'
[flake8] tests/resources/test_Search.py#L52 <811>: tests/resources/test_Search.py#L52
redefinition of unused 'bypass_api_required' from line 6
[flake8] tests/test_database_client.py#L15 <401>: tests/test_database_client.py#L15
'tests.fixtures.live_database_client' imported but unused
[flake8] tests/test_database_client.py#L15 <401>: tests/test_database_client.py#L15
'tests.fixtures.dev_db_connection' imported but unused
[flake8] tests/test_database_client.py#L15 <401>: tests/test_database_client.py#L15
'tests.fixtures.db_cursor' imported but unused
[flake8] tests/test_database_client.py#L15 <401>: tests/test_database_client.py#L15
'tests.fixtures.xylonslyvania_test_data' imported but unused
[flake8] tests/test_database_client.py#L21 <401>: tests/test_database_client.py#L21
'tests.helper_scripts.test_data_generator.TestDataGenerator' imported but unused
[flake8] tests/test_database_client.py#L452 <811>: tests/test_database_client.py#L452
redefinition of unused 'live_database_client' from line 15
[flake8] tests/test_database_client.py#L453 <205>: tests/test_database_client.py#L453
1 blank line required between summary line and description
[flake8] utilities/enums.py#L1 <100>: utilities/enums.py#L1
Missing docstring in public module
[flake8] utilities/enums.py#L4 <200>: utilities/enums.py#L4
One-line docstring should fit on one line with quotes
[flake8] utilities/enums.py#L4 <204>: utilities/enums.py#L4
1 blank line required after class docstring
Test API
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-python@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Setup API
The following actions uses Node.js version which is deprecated and will be forced to run on node20: styfle/cancel-workflow-action@0.11.0, actions/setup-python@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Setup client
The following actions uses Node.js version which is deprecated and will be forced to run on node20: styfle/cancel-workflow-action@0.11.0, actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Build client
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Lint client
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-node@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Lint client: client/src/util/pageData.js#L172
Insert `·`
Lint client: client/src/util/pageData.js#L189
Replace `⏎↹↹↹{·type:·'Other'·},⏎↹↹]` with `{·type:·'Other'·}],`
Test client
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/setup-node@v3, szenius/set-timezone@v1.2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/