aiondemand / aiondemand/AIOD-rest-api

Review Time Assertions in Tests: test_router_dataset_generic_fields

Open
#309 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
36
Forks
77
PR merge metrics
No merged PRs in 30d

Description

**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_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.