DAOS-13496 control: Extend timeout for SystemJoin unit test

There were some cases where the external timeout was being hit in
CI.

Also updated the mock InvokeUnaryRPC() method to allow the response-
sending goroutine to abort during the response delay, if there is one.

Required-githooks: true

Signed-off-by: Kris Jacque <kris.jacque@intel.com>

src/control/lib/control/mocks.go

@@ -162,16 +162,16 @@ func (mi *MockInvoker) InvokeUnaryRPCAsync(ctx context.Context, uReq UnaryReques
  }
  if delay > 0 {
  mi.log.Debugf("delaying mock response for %s", delay)
- time.Sleep(delay)
+ select {
+ case <-time.After(delay):
+ case <-ctx.Done():
+ mi.log.Debugf("context canceled on iteration %d (error=%s)", idx, ctx.Err().Error())
+ return
+ }
  }
 
- select {
- case <-ctx.Done():
- mi.log.Debugf("context canceled on iteration %d (error=%s)", idx, ctx.Err().Error())
- return
- case responses <- hr:
- mi.log.Debug("sending mock response")
- }
+ mi.log.Debug("sending mock response")
+ responses <- hr
  }
  close(responses)
  }(invokeCount)

src/control/lib/control/system_test.go

@@ -1455,7 +1455,7 @@ func TestControl_SystemJoin_Timeouts(t *testing.T) {
  },
  "inner context is canceled; request is retried": {
  mic: &MockInvokerConfig{
- ReqTimeout: 100 * time.Millisecond, // outer timeout
+ ReqTimeout: 500 * time.Millisecond, // outer timeout
  RetryTimeout: 10 * time.Millisecond, // inner timeout
  UnaryResponseSet: []*UnaryResponse{
  {