htwkmooc/poseidon
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Defuse data race condition of TestWithSeparateStderrReturnsCommandError.

Browse Source
This commit is contained in:
Maximilian Paß
2023-09-11 11:07:00 +02:00
parent 6159f2a045
commit 68cd8f43b4
1 changed files with 2 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
internal/nomad/nomad_test.go
View File

@ -837,6 +837,8 @@ func (s *ExecuteCommandTestSuite) TestWithSeparateStderrReturnsCommandError() {
s.Require().True(ok) s.Require().True(ok)
if isStderrCommand := strings.Contains(calledCommand, "mkfifo"); isStderrCommand { if isStderrCommand := strings.Contains(calledCommand, "mkfifo"); isStderrCommand {
// Here we defuse the data race condition of the ReturnArguments being set twice at the same time.
<-time.After(tests.ShortTimeout)
call.ReturnArguments = mock.Arguments{1, nil} call.ReturnArguments = mock.Arguments{1, nil}
} else { } else {
call.ReturnArguments = mock.Arguments{1, tests.ErrDefault} call.ReturnArguments = mock.Arguments{1, tests.ErrDefault}