mirror of
https://github.com/helm/chart-testing.git
synced 2026-02-05 09:45:14 +01:00
5b4fb1a4cf
The existing rule for validating dependency import-values worked well for either of these two cases:
**1. Importing exports**
```yaml
dependencies:
- name: dep1
version: 0.14.1
repository: oci://myrepo
import-values:
- data
```
**2. Importing child values into the parent**
```yaml
dependencies:
- name: dep1
version: 0.14.1
repository: oci://myrepo
import-values:
- child: default.data
parent: myimports
```
However, it failed on the following valid usecase:
```yaml
dependencies:
- name: dep1
version: 0.14.1
repository: oci://myrepo
import-values:
- data
- child: default.data
parent: myimports
```
Giving the following error:
```
dependencies.0.import-values.1: '{'parent': 'myimports', 'child': 'default.data'}' is not a str.
dependencies.0.import-values.0 : 'data' is not a map
```
This is because the Yamale is validating on a list of strings *or* a list of `import-value`, where a list containing both is also valid.
This commit changes the rule to allow both methods to be in the `import-values` list.
Signed-off-by: Ben Lavery-Griffiths <ben@lavery-griffiths.com>
38 lines
1011 B
YAML
38 lines
1011 B
YAML
name: str()
|
|
home: str(required=False)
|
|
version: str()
|
|
apiVersion: str()
|
|
appVersion: any(str(), num(), required=False)
|
|
description: str(required=False)
|
|
keywords: list(str(), required=False)
|
|
sources: list(str(), required=False)
|
|
maintainers: list(include('maintainer'), required=False)
|
|
dependencies: list(include('dependency'), required=False)
|
|
icon: str(required=False)
|
|
engine: str(required=False)
|
|
condition: str(required=False)
|
|
tags: str(required=False)
|
|
deprecated: bool(required=False)
|
|
kubeVersion: str(required=False)
|
|
annotations: map(str(), str(), required=False)
|
|
type: str(required=False)
|
|
---
|
|
maintainer:
|
|
name: str()
|
|
email: str(required=False)
|
|
url: str(required=False)
|
|
---
|
|
dependency:
|
|
name: str()
|
|
version: str()
|
|
repository: str(required=False)
|
|
condition: str(required=False)
|
|
tags: list(str(), required=False)
|
|
enabled: bool(required=False)
|
|
import-values: list(any(str(), include('import-value')), required=False)
|
|
alias: str(required=False)
|
|
---
|
|
import-value:
|
|
child: str()
|
|
parent: str()
|