Skip to content

Commit

Permalink
Breaking cli release package
Browse files Browse the repository at this point in the history
Signed-off-by: rohitagg2020 <rohit.aggarwal2020@gmail.com>
  • Loading branch information
rohitagg2020 committed Nov 14, 2023
1 parent 6d83c43 commit 0d62567
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 9 deletions.
5 changes: 3 additions & 2 deletions cli/pkg/kctrl/cmd/package/release/release.go
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ import (
cmdapprelease "github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/cmd/app/release"
cmdcore "github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/cmd/core"
cmdpkg "github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/cmd/package"
"github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/cmd/package/release/schemagenerator"
"github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/local"
buildconfigs "github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/local/buildconfigs"
"github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/logger"
Expand Down Expand Up @@ -175,9 +176,9 @@ func generateValuesSchema(pkgBuild buildconfigs.PackageBuild) (*kcdatav1alpha1.V
templateStage := pkgBuild.Spec.Template.Spec.App.Spec.Template[0]
switch {
case templateStage.HelmTemplate != nil:
return NewHelmValuesSchemaGen(templateStage.HelmTemplate.Path).Schema()
return schemagenerator.NewHelmValuesSchemaGen(templateStage.HelmTemplate.Path).Schema()
case templateStage.Ytt != nil:
return NewValuesSchemaGen(templateStage.Ytt.Paths).Schema()
return schemagenerator.NewValuesSchemaGen(templateStage.Ytt.Paths).Schema()
}
}
return nil, nil
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright 2022 VMware, Inc.
// SPDX-License-Identifier: Apache-2.0

package release
package schemagenerator

import (
"fmt"
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright 2022 VMware, Inc.
// SPDX-License-Identifier: Apache-2.0

package release_test
package schemagenerator

import (
"io/fs"
Expand All @@ -10,7 +10,6 @@ import (
"testing"

"github.com/stretchr/testify/require"
"github.com/vmware-tanzu/carvel-kapp-controller/cli/pkg/kctrl/cmd/package/release"
"sigs.k8s.io/yaml"
)

Expand Down Expand Up @@ -225,7 +224,7 @@ type: object
fileName := "values.yaml"
err = os.WriteFile(filepath.Join(dirName, fileName), []byte(test.input), fs.ModePerm)
require.NoError(t, err)
valuesSchema, err := release.NewHelmValuesSchemaGen("tmp").Schema()
valuesSchema, err := NewHelmValuesSchemaGen("tmp").Schema()
output, err := yaml.JSONToYAML(valuesSchema.OpenAPIv3.Raw)
require.NoError(t, err)
require.Equal(t, test.want, string(output), "Expected valuesSchema to match")
Expand All @@ -242,7 +241,7 @@ func TestHelmValuesSchemaGen_Schema_EmptyFile(t *testing.T) {
fileName := "values.yaml"
err = os.WriteFile(filepath.Join(dirName, fileName), []byte(""), fs.ModePerm)
require.NoError(t, err)
valuesSchema, err := release.NewHelmValuesSchemaGen("tmp").Schema()
valuesSchema, err := NewHelmValuesSchemaGen("tmp").Schema()
require.NoError(t, err)
require.Equal(t, 0, len(valuesSchema.OpenAPIv3.Raw), "Expected valuesSchema.OpenAPIv3.Raw to be empty")
}
Expand All @@ -254,7 +253,7 @@ func TestHelmValuesSchemaGen_Schema_File_Not_Present(t *testing.T) {
dirName := "tmp"
err := os.Mkdir(dirName, fs.ModePerm)
require.NoError(t, err)
valuesSchema, err := release.NewHelmValuesSchemaGen("tmp").Schema()
valuesSchema, err := NewHelmValuesSchemaGen("tmp").Schema()
require.NoError(t, err)
require.Equal(t, 0, len(valuesSchema.OpenAPIv3.Raw), "Expected valuesSchema.OpenAPIv3.Raw to be empty")
}
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
// Copyright 2022 VMware, Inc.
// SPDX-License-Identifier: Apache-2.0

package release
package schemagenerator

import (
"bytes"
Expand Down

0 comments on commit 0d62567

Please sign in to comment.