Skip to content

Commit

Permalink
Move check for test directories (#23)
Browse files Browse the repository at this point in the history
  • Loading branch information
elcasteel authored Nov 28, 2022
1 parent 9d9042d commit ed991ec
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 3 deletions.
5 changes: 5 additions & 0 deletions changelog/v0.1.3/directory_check.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
changelog:
- type: FIX
issueLink: https://github.com/solo-io/solo-projects/issues/4191
resolvesIssue: false
description: Move check for _test and _build directories so that it only occurs when installing from locally built versions.
6 changes: 3 additions & 3 deletions testutils/helper/install.go
Original file line number Diff line number Diff line change
Expand Up @@ -81,12 +81,12 @@ func NewSoloTestHelper(configFunc TestConfigFunc) (*SoloTestHelper, error) {
if configFunc != nil {
testConfig = configFunc(defaults)
}
if err := validateConfig(testConfig); err != nil {
return nil, errors.Wrapf(err, "test config validation failed")
}

// Get chart version
if testConfig.ReleasedVersion == "" {
if err := validateConfig(testConfig); err != nil {
return nil, errors.Wrapf(err, "test config validation failed")
}
version, err := getChartVersion(testConfig)
if err != nil {
return nil, errors.Wrapf(err, "getting Helm chart version")
Expand Down

0 comments on commit ed991ec

Please sign in to comment.