Skip to content

Commit

Permalink
Add timeouts tests for v1beta1 PipelineRun Defaults
Browse files Browse the repository at this point in the history
This commit adds the timeouts defaults for v1beta1 PipelineRun
defaults. It aims to increase the coverage of the pipelineRun timeouts
defaults in v1beta1.
  • Loading branch information
JeromeJu authored and tekton-robot committed Apr 18, 2023
1 parent 9b30f08 commit a941df2
Showing 1 changed file with 49 additions and 0 deletions.
49 changes: 49 additions & 0 deletions pkg/apis/pipeline/v1beta1/pipelinerun_defaults_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,6 +55,55 @@ func TestPipelineRunSpec_SetDefaults(t *testing.T) {
ServiceAccountName: config.DefaultServiceAccountValue,
Timeout: &metav1.Duration{Duration: 500 * time.Millisecond},
},
}, {
desc: "timeouts is nil",
prs: &v1beta1.PipelineRunSpec{},
want: &v1beta1.PipelineRunSpec{
ServiceAccountName: config.DefaultServiceAccountValue,
Timeout: &metav1.Duration{Duration: config.DefaultTimeoutMinutes * time.Minute},
},
},
{
desc: "timeouts is not nil",
prs: &v1beta1.PipelineRunSpec{
Timeouts: &v1beta1.TimeoutFields{},
},
want: &v1beta1.PipelineRunSpec{
ServiceAccountName: config.DefaultServiceAccountValue,
Timeouts: &v1beta1.TimeoutFields{
Pipeline: &metav1.Duration{Duration: config.DefaultTimeoutMinutes * time.Minute},
},
},
},
{
desc: "timeouts.pipeline is not nil",
prs: &v1beta1.PipelineRunSpec{
Timeouts: &v1beta1.TimeoutFields{
Pipeline: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
},
},
want: &v1beta1.PipelineRunSpec{
ServiceAccountName: config.DefaultServiceAccountValue,
Timeouts: &v1beta1.TimeoutFields{
Pipeline: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
},
},
}, {
desc: "timeouts.pipeline is nil with timeouts.tasks and timeouts.finally",
prs: &v1beta1.PipelineRunSpec{
Timeouts: &v1beta1.TimeoutFields{
Tasks: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
Finally: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
},
},
want: &v1beta1.PipelineRunSpec{
ServiceAccountName: config.DefaultServiceAccountValue,
Timeouts: &v1beta1.TimeoutFields{
Pipeline: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes) * time.Minute},
Tasks: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
Finally: &metav1.Duration{Duration: (config.DefaultTimeoutMinutes + 1) * time.Minute},
},
},
},
{
desc: "pod template is nil",
Expand Down

0 comments on commit a941df2

Please sign in to comment.