Helm test only pod should be ignored

Description of the bug:

When using the Helm construct to load a helm chart, the generated k8s yaml file contains pod that is test only. According to helm doc https://helm.sh/docs/topics/charts_hooks/ , those are only used when run helm test. The Helm construct should ignore it.

Reproduction Steps:

One helm chart can be used to reproduce is

  - name: elasticsearch
    repository: https://helm.elastic.co
    version: 7.6.2
    condition: elasticsearch.enabled

The final k8s yaml file will contain something like this when it should not

apiVersion: "v1"
kind: "Pod"
metadata:
  annotations:
    helm.sh/hook: "test-success"
  name: "temporal-fjobr-test"
spec:
  containers:
    - command:
        - "sh"
        - "-c"
        - "#!/usr/bin/env bash -e

          curl -XGET --fail
          'elasticsearch-master:9200/_cluster/health?wait_for_status=green&time\
          out=1s'\n"
      image: "docker.elastic.co/elasticsearch/elasticsearch:7.10.1"
      name: "temporal-yuizz-test"
  restartPolicy: "Never"

Error Log:

After kubectl apply you will see this

temporal-fjobr-test                                 0/1     Error        0          33s

---

This is 🐛 Bug Report

[sunshineo]

I confirm --skip-tests works. Need to have helm version >= 3.5

[sunshineo]

@mbonig Elasticsearch is the only chart I encountered doing this but any helm chart with test pods will encounter this.

I’ll try the helmFlags property. Thank you very much!