DEVX-13: Enhance check_test_speed with per-test timing quality gate #24

2026-06-23T16:10:51Z

emil commented

2026-06-23 16:10:51 +00:00

Summary

Add --max-single-seconds option to devx.tools.check_test_speed for per-test timing validation. This enforces that no individual test exceeds a configurable time limit (default 0.5s), in addition to the existing total suite time check.

Changes

Enhanced check_test_speed.py with parse_per_test_durations() and check_per_test_speed() functions
Uses PYTEST_ADDOPTS=--durations=0 to collect per-test timings from make test-unit
Updated CI to use --max-seconds 60 --max-single-seconds 2.0 (generous for CI overhead)
Updated hooks to use --max-seconds 10 --max-single-seconds 0.5 (strict for local dev)
Updated CLI docs
Full test coverage for new functionality

Closes DEVX-13

## Summary Add --max-single-seconds option to devx.tools.check_test_speed for per-test timing validation. This enforces that no individual test exceeds a configurable time limit (default 0.5s), in addition to the existing total suite time check. ### Changes - Enhanced check_test_speed.py with parse_per_test_durations() and check_per_test_speed() functions - Uses PYTEST_ADDOPTS=--durations=0 to collect per-test timings from make test-unit - Updated CI to use --max-seconds 60 --max-single-seconds 2.0 (generous for CI overhead) - Updated hooks to use --max-seconds 10 --max-single-seconds 0.5 (strict for local dev) - Updated CLI docs - Full test coverage for new functionality Closes DEVX-13

emil reviewed 2026-06-23 16:12:00 +00:00

emil left a comment

Automated PR Review

Architecture compliance: OK
Best practices: OK
Security: OK
i18n: OK
Resource management: OK
Documentation: OK
Tests: OK
Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.

emil reviewed 2026-06-23 16:21:41 +00:00

emil left a comment

Automated PR Review

Architecture compliance: OK
Best practices: OK
Security: OK
i18n: OK
Resource management: OK
Documentation: OK
Tests: OK
Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.

emil reviewed 2026-06-23 16:25:36 +00:00

emil left a comment

Automated PR Review

Architecture compliance: OK
Best practices: OK
Security: OK
i18n: OK
Resource management: OK
Documentation: OK
Tests: OK
Commit conventions: OK

No issues found by automated checks.

Auto-merge: If all CI checks pass, this PR will be merged automatically.

## Automated PR Review - Architecture compliance: OK - Best practices: OK - Security: OK - i18n: OK - Resource management: OK - Documentation: OK - Tests: OK - Commit conventions: OK No issues found by automated checks. --- **Auto-merge:** If all CI checks pass, this PR will be merged automatically.

emil merged commit c20dfd185a into master

2026-06-23 16:25:51 +00:00

emil deleted branch DEVX-13-per-test-speed-gate

2026-06-23 16:25:51 +00:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: oblachno-oss/devx#24