Skip to content

Commit

Permalink
Fix test
Browse files Browse the repository at this point in the history
Signed-off-by: Yoshiki Fujikane <ffjlabo@gmail.com>
  • Loading branch information
ffjlabo committed Dec 13, 2024
1 parent f303e00 commit 2d0af05
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
4 changes: 2 additions & 2 deletions pkg/app/pipedv1/controller/scheduler_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ func TestExecuteStage(t *testing.T) {
expected: model.StageStatus_STAGE_FAILURE,
},
{
name: "stage without config, should be set as failed",
name: "stage without config, should be success",
deployment: &model.Deployment{
Stages: []*model.PipelineStage{
{
Expand All @@ -202,7 +202,7 @@ func TestExecuteStage(t *testing.T) {
Stages: []config.PipelineStage{},
},
},
expected: model.StageStatus_STAGE_FAILURE,
expected: model.StageStatus_STAGE_SUCCESS,
},
}

Expand Down
5 changes: 5 additions & 0 deletions pkg/configv1/application_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -404,6 +404,7 @@ func TestGenericTriggerConfiguration(t *testing.T) {
Planner: DeploymentPlanner{
AutoRollback: newBoolPointer(true),
},
Pipeline: &DeploymentPipeline{},
},
expectedError: nil,
},
Expand Down Expand Up @@ -447,6 +448,7 @@ func TestTrueByDefaultBoolConfiguration(t *testing.T) {
Planner: DeploymentPlanner{
AutoRollback: newBoolPointer(true),
},
Pipeline: &DeploymentPipeline{},
},
expectedError: nil,
},
Expand All @@ -468,6 +470,7 @@ func TestTrueByDefaultBoolConfiguration(t *testing.T) {
Planner: DeploymentPlanner{
AutoRollback: newBoolPointer(true),
},
Pipeline: &DeploymentPipeline{},
},
expectedError: nil,
},
Expand All @@ -489,6 +492,7 @@ func TestTrueByDefaultBoolConfiguration(t *testing.T) {
Planner: DeploymentPlanner{
AutoRollback: newBoolPointer(true),
},
Pipeline: &DeploymentPipeline{},
},
expectedError: nil,
},
Expand Down Expand Up @@ -550,6 +554,7 @@ func TestGenericPostSyncConfiguration(t *testing.T) {
},
},
},
Pipeline: &DeploymentPipeline{},
},
expectedError: nil,
},
Expand Down

0 comments on commit 2d0af05

Please sign in to comment.