mirror of
https://github.com/woodpecker-ci/woodpecker.git
synced 2024-11-30 05:41:12 +00:00
22dfd2ef62
- refactor pipeline parsing - do not parse the pipeline multiple times to perform filter checks, do this once and perform checks on the result directly - code deduplication - refactor forge token refreshing - move refreshing to a helper func to reduce code --------- Co-authored-by: Anbraten <anton@ju60.de>
564 lines
11 KiB
Go
564 lines
11 KiB
Go
// Copyright 2022 Woodpecker Authors
|
|
// Copyright 2018 Drone.IO Inc.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
package pipeline
|
|
|
|
import (
|
|
"fmt"
|
|
"testing"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
|
|
"github.com/woodpecker-ci/woodpecker/server/forge"
|
|
"github.com/woodpecker-ci/woodpecker/server/forge/mocks"
|
|
forge_types "github.com/woodpecker-ci/woodpecker/server/forge/types"
|
|
"github.com/woodpecker-ci/woodpecker/server/model"
|
|
)
|
|
|
|
func TestGlobalEnvsubst(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Envs: map[string]string{
|
|
"KEY_K": "VALUE_V",
|
|
"IMAGE": "scratch",
|
|
},
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{
|
|
Message: "aaa",
|
|
},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: ${IMAGE}
|
|
yyy: ${CI_COMMIT_MESSAGE}
|
|
`)},
|
|
},
|
|
}
|
|
|
|
if pipelineItems, err := b.Build(); err != nil {
|
|
t.Fatal(err)
|
|
} else {
|
|
fmt.Println(pipelineItems)
|
|
}
|
|
}
|
|
|
|
func TestMissingGlobalEnvsubst(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Envs: map[string]string{
|
|
"KEY_K": "VALUE_V",
|
|
"NO_IMAGE": "scratch",
|
|
},
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{
|
|
Message: "aaa",
|
|
},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: ${IMAGE}
|
|
yyy: ${CI_COMMIT_MESSAGE}
|
|
`)},
|
|
},
|
|
}
|
|
|
|
if _, err := b.Build(); err != nil {
|
|
fmt.Println("test rightfully failed")
|
|
} else {
|
|
t.Fatal("test erroneously succeeded")
|
|
}
|
|
}
|
|
|
|
func TestMultilineEnvsubst(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{
|
|
Message: `aaa
|
|
bbb`,
|
|
},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
xxx:
|
|
image: scratch
|
|
yyy: ${CI_COMMIT_MESSAGE}
|
|
`)},
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
yyy: ${CI_COMMIT_MESSAGE}
|
|
`)},
|
|
},
|
|
}
|
|
|
|
if pipelineItems, err := b.Build(); err != nil {
|
|
t.Fatal(err)
|
|
} else {
|
|
fmt.Println(pipelineItems)
|
|
}
|
|
}
|
|
|
|
func TestMultiPipeline(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
xxx:
|
|
image: scratch
|
|
`)},
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems) != 2 {
|
|
t.Fatal("Should have generated 2 pipelineItems")
|
|
}
|
|
}
|
|
|
|
func TestDependsOn(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Name: "lint", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
{Name: "test", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
{Data: []byte(`
|
|
steps:
|
|
deploy:
|
|
image: scratch
|
|
|
|
depends_on:
|
|
- lint
|
|
- test
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems[0].DependsOn) != 2 {
|
|
t.Fatal("Should have 3 dependencies")
|
|
}
|
|
if pipelineItems[0].DependsOn[1] != "test" {
|
|
t.Fatal("Should depend on test")
|
|
}
|
|
}
|
|
|
|
func TestRunsOn(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
deploy:
|
|
image: scratch
|
|
|
|
runs_on:
|
|
- success
|
|
- failure
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems[0].RunsOn) != 2 {
|
|
t.Fatal("Should run on success and failure")
|
|
}
|
|
if pipelineItems[0].RunsOn[1] != "failure" {
|
|
t.Fatal("Should run on failure")
|
|
}
|
|
}
|
|
|
|
func TestPipelineName(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{Config: ".woodpecker"},
|
|
Curr: &model.Pipeline{},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Name: ".woodpecker/lint.yml", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
{Name: ".woodpecker/.test.yml", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
pipelineNames := []string{pipelineItems[0].Workflow.Name, pipelineItems[1].Workflow.Name}
|
|
if !containsItemWithName("lint", pipelineItems) || !containsItemWithName("test", pipelineItems) {
|
|
t.Fatalf("Pipeline name should be 'lint' and 'test' but are '%v'", pipelineNames)
|
|
}
|
|
}
|
|
|
|
func TestRootWhenBranchFilter(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{Branch: "dev"},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
steps:
|
|
xxx:
|
|
image: scratch
|
|
when:
|
|
branch: main
|
|
`)},
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if !assert.Len(t, pipelineItems, 1) {
|
|
t.Fatal("Should have generated 1 pipeline")
|
|
}
|
|
if pipelineItems[0].Workflow.State != model.StatusPending {
|
|
t.Fatal("Should run on dev branch")
|
|
}
|
|
}
|
|
|
|
func TestRootWhenFilter(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: &model.Pipeline{Event: "tester"},
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
when:
|
|
event:
|
|
- tester
|
|
steps:
|
|
xxx:
|
|
image: scratch
|
|
`)},
|
|
{Data: []byte(`
|
|
when:
|
|
event:
|
|
- push
|
|
steps:
|
|
xxx:
|
|
image: scratch
|
|
`)},
|
|
{Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
|
|
if len(pipelineItems) != 2 {
|
|
t.Fatal("Should have generated 2 pipelineItems")
|
|
}
|
|
}
|
|
|
|
func TestZeroSteps(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
pipeline := &model.Pipeline{Branch: "dev"}
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: pipeline,
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Data: []byte(`
|
|
skip_clone: true
|
|
steps:
|
|
build:
|
|
when:
|
|
branch: notdev
|
|
image: scratch
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems) != 0 {
|
|
t.Fatal("Should not generate a pipeline item if there are no steps")
|
|
}
|
|
}
|
|
|
|
func TestZeroStepsAsMultiPipelineDeps(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
pipeline := &model.Pipeline{Branch: "dev"}
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: pipeline,
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Name: "zerostep", Data: []byte(`
|
|
skip_clone: true
|
|
steps:
|
|
build:
|
|
when:
|
|
branch: notdev
|
|
image: scratch
|
|
`)},
|
|
{Name: "justastep", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
{Name: "shouldbefiltered", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
depends_on: [ zerostep ]
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems) != 1 {
|
|
t.Fatal("Zerostep and the step that depends on it should not generate a pipeline item")
|
|
}
|
|
if pipelineItems[0].Workflow.Name != "justastep" {
|
|
t.Fatal("justastep should have been generated")
|
|
}
|
|
}
|
|
|
|
func TestZeroStepsAsMultiPipelineTransitiveDeps(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
pipeline := &model.Pipeline{Branch: "dev"}
|
|
|
|
b := StepBuilder{
|
|
Forge: getMockForge(t),
|
|
Repo: &model.Repo{},
|
|
Curr: pipeline,
|
|
Last: &model.Pipeline{},
|
|
Netrc: &model.Netrc{},
|
|
Secs: []*model.Secret{},
|
|
Regs: []*model.Registry{},
|
|
Link: "",
|
|
Yamls: []*forge_types.FileMeta{
|
|
{Name: "zerostep", Data: []byte(`
|
|
skip_clone: true
|
|
steps:
|
|
build:
|
|
when:
|
|
branch: notdev
|
|
image: scratch
|
|
`)},
|
|
{Name: "justastep", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
`)},
|
|
{Name: "shouldbefiltered", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
depends_on: [ zerostep ]
|
|
`)},
|
|
{Name: "shouldbefilteredtoo", Data: []byte(`
|
|
steps:
|
|
build:
|
|
image: scratch
|
|
depends_on: [ shouldbefiltered ]
|
|
`)},
|
|
},
|
|
}
|
|
|
|
pipelineItems, err := b.Build()
|
|
if err != nil {
|
|
t.Fatal(err)
|
|
}
|
|
if len(pipelineItems) != 1 {
|
|
t.Fatal("Zerostep and the step that depends on it, and the one depending on it should not generate a pipeline item")
|
|
}
|
|
if pipelineItems[0].Workflow.Name != "justastep" {
|
|
t.Fatal("justastep should have been generated")
|
|
}
|
|
}
|
|
|
|
func TestSanitizePath(t *testing.T) {
|
|
t.Parallel()
|
|
|
|
testTable := []struct {
|
|
path string
|
|
sanitizedPath string
|
|
}{
|
|
{
|
|
path: ".woodpecker/test.yml",
|
|
sanitizedPath: "test",
|
|
},
|
|
{
|
|
path: ".woodpecker.yml",
|
|
sanitizedPath: "woodpecker",
|
|
},
|
|
{
|
|
path: "folder/sub-folder/test.yml",
|
|
sanitizedPath: "test",
|
|
},
|
|
{
|
|
path: ".woodpecker/test.yaml",
|
|
sanitizedPath: "test",
|
|
},
|
|
{
|
|
path: ".woodpecker.yaml",
|
|
sanitizedPath: "woodpecker",
|
|
},
|
|
{
|
|
path: "folder/sub-folder/test.yaml",
|
|
sanitizedPath: "test",
|
|
},
|
|
}
|
|
|
|
for _, test := range testTable {
|
|
if test.sanitizedPath != SanitizePath(test.path) {
|
|
t.Fatal("Path hasn't been sanitized correctly")
|
|
}
|
|
}
|
|
}
|
|
|
|
func getMockForge(t *testing.T) forge.Forge {
|
|
forge := mocks.NewForge(t)
|
|
forge.On("Name").Return("mock")
|
|
forge.On("URL").Return("https://codeberg.org")
|
|
return forge
|
|
}
|