Skip to content

[Bug]: Universal CI failure in test_health_check due to missing client fixture #344

@Chirantan112

Description

@Chirantan112

What happened?

The test_health_check function in tests/test_basic.py is currently failing across the repository. It is missing the client positional argument required by the test suite, causing a TypeError.

Evidence:

My current Pull Request (#263) is failing because of this universal test error, even though the logic in my new file is correct. You can see the error in my PR's check logs: test_health_check() missing 1 required positional argument: 'client'.

Fix:

I have a patch ready that updates the function signature to test_health_check(client) and removes the redundant get_client() call.
Please assign this to me under GSSoC 2026 so that I can submit the fix!

To Reproduce:

Run python -m pytest tests/
Observe the failure in test_health_check.
Expected behavior:
The test should accept the client fixture to properly interface with the Flask test client.

Expected behaviour

The pytest runner should automatically inject the client fixture into the test_health_check function. When running the test suite, the health check should return a 200 OK status and pass successfully without any TypeError or manual intervention in the codebase.

Area of the app affected

starter_code

Python version

Python 3.10+

Operating system

Windows 11

Before submitting

  • I searched existing issues and this has not been reported before.
  • I can reproduce this bug consistently with the steps above.
  • I am running the latest version of the main branch.

Metadata

Metadata

Assignees

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions