Don't lose test configuration when running skaffold fix

pkg/skaffold/runner/runner.go

@@ -145,7 +145,7 @@ func getBuilder(cfg *latest.BuildConfig, kubeContext string) (build.Builder, err
 	}
 }
 
-func getTester(cfg *[]*latest.TestCase) (test.Tester, error) {
+func getTester(cfg *latest.TestConfig) (test.Tester, error) {
 	return test.NewTester(cfg)
 }
 

pkg/skaffold/schema/latest/config.go

@@ -34,7 +34,7 @@ type SkaffoldPipeline struct {
 	Kind       string `yaml:"kind"`
 
 	Build    BuildConfig  `yaml:"build,omitempty"`
-	Test     []*TestCase  `yaml:"test,omitempty"`
+	Test     TestConfig   `yaml:"test,omitempty"`
 	Deploy   DeployConfig `yaml:"deploy,omitempty"`
 	Profiles []Profile    `yaml:"profiles,omitempty"`
 }
@@ -133,6 +133,8 @@ type AzureContainerBuild struct {
 	TenantID       string `yaml:"tenantId,omitempty"`
 }
 
+type TestConfig []*TestCase
+
 // TestCase is a struct containing all the specified test
 // configuration for an image.
 type TestCase struct {
@@ -237,7 +239,7 @@ type Artifact struct {
 type Profile struct {
 	Name   string       `yaml:"name,omitempty"`
 	Build  BuildConfig  `yaml:"build,omitempty"`
-	Test   []*TestCase  `yaml:"test,omitempty"`
+	Test   TestConfig   `yaml:"test,omitempty"`
 	Deploy DeployConfig `yaml:"deploy,omitempty"`
 }
 

pkg/skaffold/schema/profiles.go

@@ -55,7 +55,7 @@ func applyProfile(config *latest.SkaffoldPipeline, profile latest.Profile) {
 		Kind:       config.Kind,
 		Build:      overlayProfileField(config.Build, profile.Build).(latest.BuildConfig),
 		Deploy:     overlayProfileField(config.Deploy, profile.Deploy).(latest.DeployConfig),
-		Test:       overlayProfileField(config.Test, profile.Test).([]*latest.TestCase),
+		Test:       overlayProfileField(config.Test, profile.Test).(latest.TestConfig),
 	}
 }
 

pkg/skaffold/schema/v1alpha4/config.go

@@ -34,11 +34,13 @@ type SkaffoldPipeline struct {
 	Kind       string `yaml:"kind"`
 
 	Build    BuildConfig  `yaml:"build,omitempty"`
-	Test     []TestCase   `yaml:"test,omitempty"`
+	Test     TestConfig   `yaml:"test,omitempty"`
 	Deploy   DeployConfig `yaml:"deploy,omitempty"`
 	Profiles []Profile    `yaml:"profiles,omitempty"`
 }
 
+type TestConfig []TestCase
+
 func (c *SkaffoldPipeline) GetVersion() string {
 	return c.APIVersion
 }

pkg/skaffold/schema/v1alpha4/upgrade.go

@@ -46,11 +46,18 @@ func (config *SkaffoldPipeline) Upgrade() (util.VersionedConfig, error) {
 		return nil, errors.Wrap(err, "converting new build")
 	}
 
+	// convert Test (should be the same)
+	var newTest next.TestConfig
+	if err := convert(config.Test, &newTest); err != nil {
+		return nil, errors.Wrap(err, "converting new test")
+	}
+
 	return &next.SkaffoldPipeline{
 		APIVersion: next.Version,
 		Kind:       config.Kind,
-		Deploy:     newDeploy,
 		Build:      newBuild,
+		Test:       newTest,
+		Deploy:     newDeploy,
 		Profiles:   newProfiles,
 	}, nil
 }

pkg/skaffold/test/test.go

@@ -32,7 +32,7 @@ import (
 // NewTester parses the provided test cases from the Skaffold config,
 // and returns a Tester instance with all the necessary test runners
 // to run all specified tests.
-func NewTester(testCases *[]*latest.TestCase) (Tester, error) {
+func NewTester(testCases *latest.TestConfig) (Tester, error) {
 	// TODO(nkubala): copied this from runner.getDeployer(), this should be moved somewhere else
 	cwd, err := os.Getwd()
 	if err != nil {

pkg/skaffold/test/types.go

@@ -42,7 +42,7 @@ type Tester interface {
 // FullTester should always be the ONLY implementation of the Tester interface;
 // newly added testing implementations should implement the Runner interface.
 type FullTester struct {
-	testCases  *[]*latest.TestCase
+	testCases  *latest.TestConfig
 	workingDir string
 }