TDisc / FrisPy / 15740422848

84%

push

github

areo coefficients and smoke tests (#8)

* The test suite has been improved with several changes:

1.  **Reorganized Service API Tests**:
    The tests formerly in `tests/test_service.py` have been reorganized
    into more focused files:
    - `tests/test_aero_coefficients_endpoint.py`
    - `tests/test_flight_path_endpoints.py`
    - `tests/test_root_endpoint.py`
    A `tests/conftest.py` was added for shared fixtures (e.g., Flask client).

2.  **Fixed Failing Tests in `tests/test_disc.py`**:
    I addressed failures in `tests/test_disc.py` that arose after
    dependency updates. This involved:
    - Aligning test initial conditions with the quaternion-based system.
    - Updating assertions related to initial conditions and trajectory results.
    - Correcting test logic for specific scenarios.
    All tests in `tests/test_disc.py` now pass.

3.  **Added Landing Point Smoke Tests**:
    I introduced new smoke tests in `tests/test_flight_smoke.py`. These
    tests execute several predefined, realistic throw scenarios through the
    flight path API endpoints and assert that the calculated landing
    positions are within acceptable tolerances of pre-established baselines.
    This provides a higher-level check on the integrity of the physics
    simulation.

All existing and new tests pass, ensuring the stability and correctness
of the application after these changes.

* chore: Update GitHub Actions workflow trigger to develop branch

Modifies the `.github/workflows/python-app.yml` to trigger
on push and pull_request events to the `develop` branch,
instead of `master`.

This aligns the CI process with the project's default development branch.

* fix: Use typing.List for Python 3.8 compatibility

I've updated the type hints in `frispy/throw_data.py` to use `typing.List`
instead of the `list[...]]` syntax for generic types.

This change is necessary for compatibility with Python 3.8, which
does not support the `list[...]]` syntax (introduced in Python... (continued)

13 of 16 new or added lines in 1 file covered. (81.25%)

525 of 622 relevant lines covered (84.41%)

0.84 hits per line