feat: [KSBP-101776] - Update emoji

Author: kindredjcharters

k8s-deployer/src/dependency-resolver.ts

@@ -221,15 +221,15 @@ export const printDependencyGraph = (components: Array<Schema.DeployableComponen
   console.log("Dependency Graph")
   console.log(sep)
   levels.forEach((level, idx) =>
-    console.log(`  Stage ${idx + 1} │  ${level.map(c => c.parallel ? `${c.id} ⚡` : c.id).join("  ")}`)
+    console.log(`  Stage ${idx + 1} │  ${level.map(c => c.parallel ? `${c.id} 🔀` : c.id).join("  ")}`)
   )
   if (edges.length > 0) {
     console.log(sep)
     edges.forEach(e => console.log(e))
   }
   if (components.some(c => c.parallel)) {
     console.log(sep)
-    console.log("  ⚡ = deployed concurrently within stage")
+    console.log("  🔀 = concurrent deployment")
   }
   console.log(sep)
 }

k8s-deployer/src/pitfile/schema-v1.ts

@@ -67,12 +67,6 @@ export class DeployableComponent {
 export class Graph {
   testApp: DeployableComponent
   components: Array<DeployableComponent>
-  /**
-   * When true, the test app is deployed concurrently with the component chain.
-   * Only enable this if the test app can start independently of the components.
-   * Defaults to false (sequential: test app starts after all components are ready).
-   */
-  parallelTestApp?: boolean
 }
 
 export class Deployment {

k8s-deployer/src/test-suite-handler.ts

@@ -107,7 +107,7 @@ export const deployGraph = async (config: Config, workspace: string, testSuiteId
   const params = [ testSuiteId ]
 
   let testAppDeployedComponent: DeployedComponent
-  if (graph.parallelTestApp === true) {
+  if (graph.testApp.parallel === true) {
     // Deploy test app concurrently with the component levels (opt-in).
     // Only use this when the test app can start independently of the components.
     const deployTestAppPromise = Deployer.deployComponent(config, workspace, graph.testApp, namespace, params)

k8s-deployer/test/dependency-resolver.spec.ts

@@ -444,7 +444,7 @@ describe("Dependency Resolver", () => {
       expect(logOutput).to.include("  b ──▶ c")
     })
 
-    it("annotates parallel components with ⚡ and prints a legend", () => {
+    it("annotates parallel components with 🔀 and prints a legend", () => {
       const components: Array<Schema.DeployableComponent> = [
         { name: "A", id: "a", location: { type: Schema.LocationType.Local }, deploy: { command: "deploy.sh" }, undeploy: { command: "undeploy.sh" } },
         { name: "B", id: "b", location: { type: Schema.LocationType.Local }, deploy: { command: "deploy.sh" }, undeploy: { command: "undeploy.sh" }, dependsOn: ["a"], parallel: true },
@@ -453,10 +453,10 @@ describe("Dependency Resolver", () => {
       printDependencyGraph(components)
       expect(logOutput[0]).to.equal("Dependency Graph")
       expect(logOutput).to.include("  Stage 1 │  a")
-      expect(logOutput).to.include("  Stage 2 │  b ⚡  c ⚡")
+      expect(logOutput).to.include("  Stage 2 │  b 🔀  c 🔀")
       expect(logOutput).to.include("  a ──▶ b")
       expect(logOutput).to.include("  a ──▶ c")
-      expect(logOutput).to.include("  ⚡ = deployed concurrently within stage")
+      expect(logOutput).to.include("  🔀 = concurrent deployment")
     })
   })
 })

k8s-deployer/test/test-suite-handler.spec.ts

@@ -57,7 +57,6 @@ describe("Deployment happy path", async () => {
     location: { type: LocationType.Local },
     deployment: {
       graph: {
-        parallelTestApp: true,
         testApp: {
           name: "comp-1-test-app-name",
           id: "comp-1-test-app",
@@ -68,7 +67,8 @@ describe("Deployment happy path", async () => {
           },
           logTailing: {
             enabled: true
-          }
+          },
+          parallel: true
         },
         components: [
           {
@@ -499,8 +499,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     const deployStub = sinon.stub().callsFake(async (_cfg, _ws, spec) => `sha-${spec.id}`)
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
-      parallelTestApp: true,
-      testApp: makeSpec("test-app"),
+      testApp: makeSpec("test-app", { parallel: true }),
       components: [makeSpec("comp-a"), makeSpec("comp-b")]
     }
     const result = await SuiteHandler.deployGraph(config, workspace, testSuiteId, graph, namespace)
@@ -522,8 +521,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
-      parallelTestApp: true,
-      testApp: makeSpec("testApp"),
+      testApp: makeSpec("testApp", { parallel: true }),
       components: [makeSpec("B", { parallel: true }), makeSpec("C", { parallel: true })]
     }
     const deployPromise = SuiteHandler.deployGraph(config, workspace, testSuiteId, graph, namespace)
@@ -552,8 +550,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     const SuiteHandler = await loadWithStub(deployStub)
     // B is parallel:true, C has no parallel flag — same dependency level, so neither waits on the other
     const graph = {
-      parallelTestApp: true,
-      testApp: makeSpec("testApp"),
+      testApp: makeSpec("testApp", { parallel: true }),
       components: [makeSpec("B", { parallel: true }), makeSpec("C")]
     }
     const deployPromise = SuiteHandler.deployGraph(config, workspace, testSuiteId, graph, namespace)
@@ -583,8 +580,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
-      parallelTestApp: true,
-      testApp: makeSpec("testApp"),
+      testApp: makeSpec("testApp", { parallel: true }),
       components: [
         makeSpec("A"),
         makeSpec("B", { parallel: true, dependsOn: ["A"] }),
@@ -629,8 +625,7 @@ describe("deployGraph - deployment ordering and concurrency", async () => {
     })
     const SuiteHandler = await loadWithStub(deployStub)
     const graph = {
-      parallelTestApp: true,
-      testApp: makeSpec("testApp"),
+      testApp: makeSpec("testApp", { parallel: true }),
       components: [makeSpec("A")]
     }
     const deployPromise = SuiteHandler.deployGraph(config, workspace, testSuiteId, graph, namespace)