Skip to content

Commit

Permalink
Up async wait for test, remove self contianed arg for fx-dependent build
Browse files Browse the repository at this point in the history
Work done for #196
  • Loading branch information
atruskie committed Mar 31, 2020
1 parent 074e24f commit 6665101
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 3 additions & 1 deletion build/azure-pipelines-build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -44,8 +44,10 @@ jobs:
variables:
${{ if eq('', platform.rid) }}:
runtimeArgument: ''
selfContainedArgument: ''
${{ if ne('', platform.rid) }}:
runtimeArgument: --runtime ${{ platform.rid }}
selfContainedArgument: --self-contained
steps:
# - pwsh: 'Get-ChildItem Env:'
# displayName: debug environment variables
Expand Down Expand Up @@ -175,7 +177,7 @@ jobs:
$(runtimeArgument)
--configuration ${{ configuration }}
--output $(Build.BinariesDirectory)
--self-contained
$(selfContainedArgument)
nobuild: true
publishWebProjects: false
zipAfterPublish: false # does not seem to tar - we need to keep unix permissions
Expand Down
2 changes: 2 additions & 0 deletions tests/Acoustics.Test/AnalysisBase/AnalysisCoordinatorTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -881,13 +881,15 @@ private void TestAnalysisCoordinatorPaths(SaveBehavior wav, bool unique, Directo
this.AssertFilesAreAsExpected(3, states[3], paths);

// complete
// TODO: remove this rubbish and stick in a IoC file system for testing!
dummyAnalyzer.Pump(false);
do
{
task.Wait(1.0.Seconds());
dummyAnalyzer.Pump(false);
}
while (!task.IsCompleted);
task.Wait(1.0.Seconds());

this.AssertFilesAreAsExpected(4, states[4], paths);

Expand Down

0 comments on commit 6665101

Please sign in to comment.