Skip to content

Commit

Permalink
fix
Browse files Browse the repository at this point in the history
  • Loading branch information
Yongxuanzhang committed Aug 11, 2022
1 parent f66b8c3 commit d6b9248
Show file tree
Hide file tree
Showing 4 changed files with 14 additions and 14 deletions.
8 changes: 4 additions & 4 deletions pkg/apis/pipeline/v1beta1/param_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -48,26 +48,26 @@ func (p *ParamSpec) convertFrom(ctx context.Context, source v1.ParamSpec) {

func (p Param) convertTo(ctx context.Context, sink *v1.Param) {
sink.Name = p.Name
newValue := v1.ArrayOrString{}
newValue := v1.ParamValue{}
p.Value.convertTo(ctx, &newValue)
sink.Value = newValue
}

func (p *Param) convertFrom(ctx context.Context, source v1.Param) {
p.Name = source.Name
newValue := ArrayOrString{}
newValue := ParamValue{}
newValue.convertFrom(ctx, source.Value)
p.Value = newValue
}

func (aos ArrayOrString) convertTo(ctx context.Context, sink *v1.ArrayOrString) {
func (aos ArrayOrString) convertTo(ctx context.Context, sink *v1.ParamValue) {
sink.Type = v1.ParamType(aos.Type)
sink.StringVal = aos.StringVal
sink.ArrayVal = aos.ArrayVal
sink.ObjectVal = aos.ObjectVal
}

func (aos *ArrayOrString) convertFrom(ctx context.Context, source v1.ArrayOrString) {
func (aos *ArrayOrString) convertFrom(ctx context.Context, source v1.ParamValue) {
aos.Type = ParamType(source.Type)
aos.StringVal = source.StringVal
aos.ArrayVal = source.ArrayVal
Expand Down
4 changes: 2 additions & 2 deletions pkg/apis/pipeline/v1beta1/pipeline_conversion.go
Original file line number Diff line number Diff line change
Expand Up @@ -258,7 +258,7 @@ func (pr PipelineResult) convertTo(ctx context.Context, sink *v1.PipelineResult)
sink.Name = pr.Name
sink.Type = v1.ResultsType(pr.Type)
sink.Description = pr.Description
newValue := v1.ArrayOrString{}
newValue := v1.ParamValue{}
pr.Value.convertTo(ctx, &newValue)
sink.Value = newValue
}
Expand All @@ -267,7 +267,7 @@ func (pr *PipelineResult) convertFrom(ctx context.Context, source v1.PipelineRes
pr.Name = source.Name
pr.Type = ResultsType(source.Type)
pr.Description = source.Description
newValue := ArrayOrString{}
newValue := ParamValue{}
newValue.convertFrom(ctx, source.Value)
pr.Value = newValue
}
8 changes: 4 additions & 4 deletions pkg/apis/pipeline/v1beta1/pipeline_conversion_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,13 +104,13 @@ func TestPipelineConversion(t *testing.T) {
RunAfter: []string{"task-1"},
Params: []v1beta1.Param{{
Name: "param-task-1",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
ArrayVal: []string{"value-task-1"},
},
}},
Matrix: []v1beta1.Param{{
Name: "a-param",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
Type: v1beta1.ParamTypeArray,
ArrayVal: []string{"$(params.baz)", "and", "$(params.foo-is-baz)"},
},
Expand All @@ -127,7 +127,7 @@ func TestPipelineConversion(t *testing.T) {
Type: v1beta1.ParamTypeString,
Description: "My first pipeline param",
Properties: map[string]v1beta1.PropertySpec{"foo": {Type: v1beta1.ParamTypeString}},
Default: v1beta1.NewArrayOrString("bar"),
Default: v1beta1.NewStructuredValues("bar"),
}},
Workspaces: []v1beta1.WorkspacePipelineDeclaration{{
Name: "workspace",
Expand All @@ -138,7 +138,7 @@ func TestPipelineConversion(t *testing.T) {
Name: "my-pipeline-result",
Type: v1beta1.ResultsTypeObject,
Description: "this is my pipeline result",
Value: *v1beta1.NewArrayOrString("foo.bar"),
Value: *v1beta1.NewStructuredValues("foo.bar"),
}},
Finally: []v1beta1.PipelineTask{{
Name: "final-task",
Expand Down
8 changes: 4 additions & 4 deletions pkg/apis/pipeline/v1beta1/taskrun_validation_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -97,7 +97,7 @@ func TestTaskRun_Validate(t *testing.T) {
Spec: v1beta1.TaskRunSpec{
Params: []v1beta1.Param{{
Name: "task-words",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
ArrayVal: []string{"hello", "task run"},
},
}},
Expand All @@ -119,7 +119,7 @@ func TestTaskRun_Validate(t *testing.T) {
Spec: v1beta1.TaskRunSpec{
Params: []v1beta1.Param{{
Name: "task-words",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
ArrayVal: []string{"hello", "task run"},
},
}},
Expand Down Expand Up @@ -150,7 +150,7 @@ func TestTaskRun_Validate(t *testing.T) {
Spec: v1beta1.TaskRunSpec{
Params: []v1beta1.Param{{
Name: "task-words",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
ArrayVal: []string{"hello", "task run"},
},
}},
Expand Down Expand Up @@ -184,7 +184,7 @@ func TestTaskRun_Validate(t *testing.T) {
Spec: v1beta1.TaskRunSpec{
Params: []v1beta1.Param{{
Name: "task-words",
Value: v1beta1.ArrayOrString{
Value: v1beta1.ParamValue{
ArrayVal: []string{"hello", "task run"},
},
}},
Expand Down

0 comments on commit d6b9248

Please sign in to comment.