Minor optimization on context manager

Author: bearaujus

internal/ctx_manager.go

@@ -20,9 +20,11 @@ func (cm *CtxManager) Ctx() context.Context {
 func (cm *CtxManager) Cancel() bool {
 	cm.rwMu.Lock()
 	defer cm.rwMu.Unlock()
-	cancelled := cm.c.Err() == nil
+	if cm.c.Err() != nil {
+		return false
+	}
 	cm.cl()
-	return cancelled
+	return true
 }
 
 func (cm *CtxManager) IsDead() bool {

internal/ctx_manager_test.go

@@ -20,6 +20,8 @@ func TestCtxManager(t *testing.T) {
 
 				assert.NotNil(t, cm.Ctx().Err())
 				assert.True(t, cm.IsDead())
+
+				assert.False(t, cm.Cancel())
 			},
 		},
 	}