improvement: set default include on Helm modules

BREAKING CHANGE:

If not set by the user, the `include` field on Helm modules now defaults to:

```javascript
["*", "charts/**/*", "templates/**/*"]
```

if the module has local chart sources, otherwise to:

```javascript
["*", "charts/**/*", "templates/**/*"]
```

Previously, Helm modules would simply include all content under the
module path.

If your Helm modules doesn't have `include` set and depends
on content that's not captured with the default include, you will need
to update the relevant `garden.yml` file and set the includes manually.

garden-service/src/plugins/kubernetes/helm/config.ts

@@ -352,5 +352,10 @@ export async function configureHelmModule({
     "release-name": getReleaseName(moduleConfig),
   }
 
+  // Automatically set the include if not explicitly set
+  if (!moduleConfig.include) {
+    moduleConfig.include = containsSources ? ["*", "charts/**/*", "templates/**/*"] : []
+  }
+
   return { moduleConfig }
 }

garden-service/test/integ/src/plugins/kubernetes/helm/config.ts

@@ -46,7 +46,7 @@ describe("validateHelmModule", () => {
       },
       configPath: resolve(ctx.projectRoot, "api", "garden.yml"),
       description: "The API backend for the voting UI",
-      include: undefined,
+      include: ["*", "charts/**/*", "templates/**/*"],
       exclude: undefined,
       name: "api",
       outputs: {
@@ -122,6 +122,19 @@ describe("validateHelmModule", () => {
     })
   })
 
+  it("should not set default includes if they have already been explicitly set", async () => {
+    patchModuleConfig("api", { include: ["foo"] })
+    const config = await garden.resolveModuleConfig(garden.log, "api")
+    expect(config.include).to.eql(["foo"])
+  })
+
+  it("should set include to empty if module does not have local chart sources", async () => {
+    // So that Chart.yaml isn't found
+    patchModuleConfig("api", { spec: { chartPath: "invalid-path" } })
+    const config = await garden.resolveModuleConfig(garden.log, "api")
+    expect(config.include).to.eql([])
+  })
+
   it("should not return a serviceConfig if skipDeploy=true", async () => {
     patchModuleConfig("api", { spec: { skipDeploy: true } })
     const config = await garden.resolveModuleConfig(garden.log, "api")