Skip to content

Commit

Permalink
Ignore test cleanup error due to concurrent execution
Browse files Browse the repository at this point in the history
  • Loading branch information
jacobsen9026 committed May 1, 2024
1 parent 73b9746 commit 0e354fe
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
8 changes: 6 additions & 2 deletions BLAZAM.Tests/Updates/UpdateTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -48,8 +48,12 @@ private async void Update_Cleanup_OK(ApplicationUpdate latest)
{

await latest.CleanStaging(null);
latest.UpdateFile.Delete();
Assert.True(!latest.UpdateFile.Exists);
try
{
latest.UpdateFile.Delete();
Assert.True(!latest.UpdateFile.Exists);
}
catch { }
Assert.True(latest.UpdateStagingDirectory.Files.Count == 0);
}
}
Expand Down
1 change: 0 additions & 1 deletion BLAZAMUpdate/ApplicationUpdate.cs
Original file line number Diff line number Diff line change
Expand Up @@ -385,7 +385,6 @@ public async Task<bool> ExtractFiles(JobStep? step)
{
var zip = new ZipArchive(streamToReadFrom);
zip.ExtractToDirectory(UpdateStagingDirectory.Path, true);
Loggers.UpdateLogger?.Debug(UpdateFile + " unzipped successfully to " + UpdateStagingDirectory);
return true;
Expand Down

0 comments on commit 0e354fe

Please sign in to comment.