Skip to content

test: Add test timeout to max of 60 seconds#1427

Open
ZohebShaikh wants to merge 5 commits intotests-speed-upfrom
pytest-timeout
Open

test: Add test timeout to max of 60 seconds#1427
ZohebShaikh wants to merge 5 commits intotests-speed-upfrom
pytest-timeout

Conversation

@ZohebShaikh
Copy link
Contributor

@ZohebShaikh ZohebShaikh commented Mar 2, 2026

This will be helpful in catching unit tests taking more than 3 seconds
For helm test they should ideally be not called Unit test because we generate the helm chart in a subprocess
And for the system test the max timeout is 60 seconds because its taking to actual stuff

@ZohebShaikh ZohebShaikh changed the title Pytest timeout test: Add test timeout to max of 60 seconds Mar 2, 2026
@codecov
Copy link

codecov bot commented Mar 2, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.17%. Comparing base (0744805) to head (009a398).

Additional details and impacted files
@@               Coverage Diff               @@
##           tests-speed-up    #1427   +/-   ##
===============================================
  Coverage           95.17%   95.17%           
===============================================
  Files                  43       43           
  Lines                3111     3111           
===============================================
  Hits                 2961     2961           
  Misses                150      150           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ZohebShaikh ZohebShaikh marked this pull request as ready for review March 2, 2026 12:00
@ZohebShaikh ZohebShaikh requested a review from a team as a code owner March 2, 2026 12:01
@ZohebShaikh ZohebShaikh changed the base branch from main to tests-speed-up March 2, 2026 12:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant