Use helmignore for test directories

[bschwedler]

Both helm unittest and chart-testing now support using .helmignore 🙌🏻

Let's move the test files back into the charts directory so that we no longer have to symlink them.

[bschwedler]

One of the installation tests is failing:

Installing chart with values file "charts/rstudio-workbench/ci/basic-ingress-values.yaml"...
Error: INSTALLATION FAILED: 1 error occurred:
	* Ingress.networking.k8s.io "rstudio-workbench-mz6b6a9nzw" is invalid: spec.rules[0].http.paths[0].backend: Required value: port name or number is required

The output of helm template using this values file looks correct to me, with the exception of the vertical whitespace, which I don't think should be an issue
https://kubernetes.io/docs/reference/kubernetes-api/service-resources/ingress-v1/#IngressBackend

---
# Source: rstudio-workbench/templates/ingress.yaml
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: release-name-rstudio-workbench
  labels:
    helm.sh/chart: rstudio-workbench-0.9.1
    app.kubernetes.io/name: rstudio-workbench
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "2025.05.0"
    app.kubernetes.io/managed-by: Helm
  annotations:
    kubernetes.io/ingress.class: traefik
spec:
  rules:
    - host: "workbench.rstudio.com"
      http:
        paths:
          -            
            path: /test/
            pathType: Prefix
            backend:
              service:
                name: release-name-rstudio-workbench
                port:
                  number: 80

The output parses correctly with yq

$ yq '.spec.rules[0].http.paths[0].backend.service.port.number' < ingress.yaml
80

[bschwedler]

The new chart-testing release has resolved the above issue!

[ianpittwood]

LGTM

[jforest]

I love the cleanup! Simpler and easier to understand, thanks for taking this on!