Skip to content

Commit

Permalink
DEcrease test-run timeouts.
Browse files Browse the repository at this point in the history
  • Loading branch information
wcjohns committed Jul 6, 2023
1 parent 5e14bf5 commit 04cae36
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
3 changes: 2 additions & 1 deletion .github/workflows/build_app.yml
Original file line number Diff line number Diff line change
Expand Up @@ -112,7 +112,7 @@ jobs:
run: |
$p = Start-Process .\InterSpec.exe -ArgumentList @("--test-load", "--mult-instance", "--no-restore") -PassThru
$errTimedOut = $null
$p | Wait-Process -Timeout 240 -ErrorAction SilentlyContinue -ErrorVariable errTimedOut
$p | Wait-Process -Timeout 120 -ErrorAction SilentlyContinue -ErrorVariable errTimedOut
if ($errTimedOut) { Stop-Process -InputObject $p -Force }
echo "Exit code of InterSpec test load was was" $p.ExitCode
$rval = $p.ExitCode
Expand Down Expand Up @@ -140,6 +140,7 @@ jobs:
replacesArtifacts: true
tag: 'latest'
commit: ${{ github.sha }}
makeLatest: true


- name: "Cleanup and finish"
Expand Down
4 changes: 2 additions & 2 deletions target/wxWidgets/InterSpecWxApp.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -749,7 +749,7 @@ InterSpecWxApp::InterSpecWxApp() :

if( sm_test_load_only )
{
// Right now we'll just wait 180 seconds, and if things havent loaded, declare failure.
// Right now we'll just wait 90 seconds, and if things havent loaded, declare failure.
// Normally shouldnt take very long, but on some test runners it can take a while.
// When things load, we'll stop this timer and clear it.
//
Expand All @@ -758,7 +758,7 @@ InterSpecWxApp::InterSpecWxApp() :
// void InterSpecApp::prepareForEndOfSession()
// void InterSpecApp::finalize()
// for possible hooks to check failure.
sm_check_load_timer.reset( new CheckLoadTimer( this, 180 ) );
sm_check_load_timer.reset( new CheckLoadTimer( this, 90 ) );
}//if( sm_test_load_only )

InterSpecWebFrame* frame = new InterSpecWebFrame(m_url, no_restore, file_to_open);
Expand Down

0 comments on commit 04cae36

Please sign in to comment.