Add test for updating services pipeline yaml

docs/commands/data.json

@@ -493,13 +493,13 @@
         "arg": "-a, --personal-access-token <personal-access-token>",
         "description": "Personal Access Token",
         "required": true,
-        "inherit": "azureDevops.access_token"
+        "inherit": "azure_devops.access_token"
       },
       {
         "arg": "-o, --org-name <organization-name>",
         "description": "Organization Name for Azure DevOps",
         "required": true,
-        "inherit": "azureDevops.org"
+        "inherit": "azure_devops.org"
       },
       {
         "arg": "-u, --repo-url <repo-url>",
@@ -510,7 +510,7 @@
         "arg": "-d, --devops-project <devops-project>",
         "description": "Azure DevOps Project name",
         "required": true,
-        "inherit": "azureDevops.project"
+        "inherit": "azure_devops.project"
       },
       {
         "arg": "-b, --build-script-url <build-script-url>",

src/commands/project/append-variable-group.test.ts

@@ -35,10 +35,16 @@ describe("Test execute function", () => {
       .spyOn(appendVariableGrp, "validateValues")
       .mockReturnValueOnce(mockValues as ConfigValues);
     jest.spyOn(bedrockYaml, "addVariableGroup").mockReturnValue();
+    jest
+      .spyOn(fileutils, "appendVariableGroupToPipelineYaml")
+      .mockReturnValue();
 
     expect(bedrockFile.variableGroups?.length).toBe(0);
     await execute("my-path", "my-vg", mockValues, exitFn);
     expect(exitFn).toBeCalledTimes(1);
     expect(exitFn.mock.calls).toEqual([[0]]);
+    expect(fileutils.appendVariableGroupToPipelineYaml).toHaveBeenCalledTimes(
+      3
+    );
   });
 });

src/commands/project/append-variable-group.ts

@@ -15,10 +15,12 @@ import { logger } from "../../logger";
 import { build as buildError, log as logError } from "../../lib/errorBuilder";
 import { errorStatusCode } from "../../lib/errorStatusCode";
 import decorator from "./append-variable-group.decorator.json";
-import { BedrockFileInfo } from "../../types";
+import { BedrockFileInfo, BedrockFile } from "../../types";
 import * as bedrockYaml from "../../lib/bedrockYaml";
 import { hasVariableGroup } from "../../lib/pipelines/variableGroup";
 import { AzureDevOpsOpts } from "../../lib/git";
+import { appendVariableGroupToPipelineYaml } from "../../lib/fileutils";
+import { SERVICE_PIPELINE_FILENAME } from "../../lib/constants";
 
 // Values that need to be pulled out from the command operator
 export interface CommandOptions {
@@ -87,6 +89,24 @@ export const variableGroupExists = async (
   return await hasVariableGroup(accessOpts, variableGroupName);
 };
 
+/**
+ * Update the variable groups for the project services
+ * @param bedrockFile The bedrock.yaml file
+ */
+export const updateServicesVariableGroups = (
+  bedrockFile: BedrockFile,
+  variableGroupName: string
+): void => {
+  bedrockFile.services.forEach((service) => {
+    const path = service.path;
+    appendVariableGroupToPipelineYaml(
+      path,
+      SERVICE_PIPELINE_FILENAME,
+      variableGroupName
+    );
+  });
+};
+
 /**
  * Executes the command.
  *
@@ -118,6 +138,7 @@ export const execute = async (
 
     const bedrockFile = Bedrock(projectPath);
     bedrockYaml.addVariableGroup(bedrockFile, projectPath, variableGroupName);
+    updateServicesVariableGroups(bedrockFile, variableGroupName);
     await exitFn(0);
   } catch (err) {
     logError(