ci: add unified success actions for checks (#28563) [follow up] (#28564)

.github/workflows/_base-migration.yml

@@ -15,6 +15,10 @@ on:
         type: number
         default: 20
 jobs:
+  migration-test:
+    name: Migrate
+    runs-on: ubuntu-latest
+    if: ${{ inputs.fake-success != false }}
   migration-test:
     name: Migrate
     runs-on: ubuntu-latest
@@ -121,21 +125,21 @@ jobs:
           cat $f
         done
 
-  migration-test-success:
+  # TIP: Use these for checks, e.g. Server / Migration / Success
+  success:
     name: Success
-    needs: migration-test
-    runs-on: ubuntu-latest
     if: ${{ inputs.fake-success == false }}
-    steps:
-      - name: Passed
-        run: "echo Passed"
-
-  # This satisfies the required checks on pull requests if tests are intentionally skipped
-  migration-test-success-fake:
-    name: Success
+    needs: [migration-test]
     runs-on: ubuntu-latest
-    if: ${{ inputs.fake-success != false }}
     steps:
       - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
+        run: "echo Skipped"
+  success:
+    name: Success
+    if: ${{ inputs.fake-success != false }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
         run: "echo Skipped"
-

.github/workflows/_base-server-tests.yml

@@ -159,21 +159,23 @@ jobs:
             cat $f
           done
 
-  integration-test-success:
-    needs: [unit-test, integration-test]
-    name: Success
-    runs-on: ubuntu-latest
-    if: ${{ inputs.fake-success == false }}
-    steps:
-      - name: Passed
-        run: "echo Passed"
 
-  # This satisfies the required checks on pull requests if tests are intentionally skipped
-  integration-test-success-fake:
+  # TIP: Use these for checks, e.g. Server / Tests / Success
+  success:
     name: Success
+    if: ${{ inputs.fake-success == false }}
+    needs: [unit-test, integration-test]
     runs-on: ubuntu-latest
-    if: ${{ inputs.fake-success != false }}
     steps:
       - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
+        run: "echo Skipped"
+  success:
+    name: Success
+    if: ${{ inputs.fake-success != false }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
         run: "echo Skipped"
 

.github/workflows/_base-ui-tests.yml

@@ -147,20 +147,34 @@ jobs:
             cat $f
           done
 
-  ui-test-success:
-    needs: ui-test
+  success:
     name: Success
+    needs: [ui-test]
     runs-on: ubuntu-latest
-    if: ${{ inputs.fake-success == false }}
     steps:
       - name: Passed
+        if: ${{ inputs.fake-success == false }}
         run: "echo Passed"
-  # This satisfies the required checks on pull requests if tests are intentionally skipped
-  ui-test-success-fake:
-    name: Success
-    runs-on: ubuntu-latest
-    if: ${{ inputs.fake-success != false }}
-    steps:
       - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
+        run: "echo Skipped"
+
+  # TIP: Use these for checks, e.g. UI / Tests (Cypress) / Success
+  success:
+    name: Success
+    if: ${{ inputs.fake-success == false }}
+    needs: [unit-test, integration-test]
+    runs-on: ubuntu-latest
+    steps:
+      - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
+        run: "echo Skipped"
+  success:
+    name: Success
+    if: ${{ inputs.fake-success != false }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Pass skipped tests unconditionally
+        if: ${{ inputs.fake-success != false }}
         run: "echo Skipped"