fix(api): do not check empty size for artifact (#5801)

sguiheux · web-flow · commit 368caafec582 · 2021-04-26T15:54:56.000+02:00
diff --git a/sdk/workflow_run_result.go b/sdk/workflow_run_result.go
@@ -52,9 +52,6 @@ func (a *WorkflowRunResultArtifact) IsValid() error {
 	if a.Name == "" {
 		return WrapError(ErrInvalidData, "missing artifact name")
 	}
-	if a.Size == 0 {
-		return WrapError(ErrInvalidData, "missing artifact size")
-	}
 	if a.MD5 == "" {
 		return WrapError(ErrInvalidData, "missing md5Sum")
 	}
@@ -79,9 +76,6 @@ func (a *WorkflowRunResultCoverage) IsValid() error {
 	if a.Name == "" {
 		return WrapError(ErrInvalidData, "missing file name")
 	}
-	if a.Size == 0 {
-		return WrapError(ErrInvalidData, "missing file size")
-	}
 	if a.MD5 == "" {
 		return WrapError(ErrInvalidData, "missing md5Sum")
 	}

Original file line number	Diff line number	Diff line change
`@@ -52,9 +52,6 @@ func (a *WorkflowRunResultArtifact) IsValid() error {`
`52`	`52`	`if a.Name == "" {`
`53`	`53`	`return WrapError(ErrInvalidData, "missing artifact name")`
`54`	`54`	`}`
`55`		`- if a.Size == 0 {`
`56`		`- return WrapError(ErrInvalidData, "missing artifact size")`
`57`		`- }`
`58`	`55`	`if a.MD5 == "" {`
`59`	`56`	`return WrapError(ErrInvalidData, "missing md5Sum")`
`60`	`57`	`}`
`@@ -79,9 +76,6 @@ func (a *WorkflowRunResultCoverage) IsValid() error {`
`79`	`76`	`if a.Name == "" {`
`80`	`77`	`return WrapError(ErrInvalidData, "missing file name")`
`81`	`78`	`}`
`82`		`- if a.Size == 0 {`
`83`		`- return WrapError(ErrInvalidData, "missing file size")`
`84`		`- }`
`85`	`79`	`if a.MD5 == "" {`
`86`	`80`	`return WrapError(ErrInvalidData, "missing md5Sum")`
`87`	`81`	`}`