aiondemand / AIOD-rest-api

A containerized application using FastAPI and SQLAlchemy connected to a MySQL database.
MIT License
10 stars 7 forks source link

Review Time Assertions in Tests: test_router_dataset_generic_fields #309

Open jsmatias opened 2 months ago

jsmatias commented 2 months ago

Description: The current time assertion in the test_router_dataset_generic_fields, specifically the time set as 0.2, was increased due to intermittent failures observed during pre-commit locally. While the adjustment temporarily mitigated the failures, it's uncertain the purpose of initially setting the time to 0.1.

To ensure consistency in time-related assertions across tests, it is necessary to review and standardise the time assertions.

Proposed Solution: The proposed solution involves the following steps:

  1. Review the purpose and significance of the time assertion set initially to 0.1.
  2. Standardise time assertions across tests to ensure consistency.

_Originally from discussion in https://github.com/aiondemand/AIOD-rest-api/pull/298#discussion_r1587306870_