ci(functions): update tests to use labeled on pull_request trigger (#3748)

* ci(functions): update tests to use labeled on pull_request trigger

* temporary: Add debug job

* revert inspect job

* style: unindent list items

Author: grayside

.github/workflows/functions-concepts.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-concepts.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/concepts/**'
-    - '.github/workflows/functions-concepts.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/concepts/**'
     - '.github/workflows/functions-concepts.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:
@@ -43,13 +42,13 @@ jobs:
     strategy:
       matrix:
         path:
-          - 'functions/concepts/afterResponse'
-          - 'functions/concepts/afterTimeout'
-          - 'functions/concepts/backgroundTermination'
-          - 'functions/concepts/filesystem'
-          - 'functions/concepts/httpTermination'
-          - 'functions/concepts/requests'
-          - 'functions/concepts/stateless'        
+        - 'functions/concepts/afterResponse'
+        - 'functions/concepts/afterTimeout'
+        - 'functions/concepts/backgroundTermination'
+        - 'functions/concepts/filesystem'
+        - 'functions/concepts/httpTermination'
+        - 'functions/concepts/requests'
+        - 'functions/concepts/stateless'
     uses: ./.github/workflows/test.yaml
     with:
       name: 'functions-concepts'

.github/workflows/functions-firebase.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-firebase.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/firebase/**'
-    - '.github/workflows/functions-firebase.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/firebase/**'
     - '.github/workflows/functions-firebase.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:
@@ -43,12 +42,12 @@ jobs:
     strategy:
       matrix:
         path:
-          - 'functions/firebase/helloAnalytics'
-          - 'functions/firebase/helloAuth'
-          - 'functions/firebase/helloFirestore'
-          - 'functions/firebase/helloRemoteConfig'
-          - 'functions/firebase/helloRTDB'
-          - 'functions/firebase/makeUpperCase'
+        - 'functions/firebase/helloAnalytics'
+        - 'functions/firebase/helloAuth'
+        - 'functions/firebase/helloFirestore'
+        - 'functions/firebase/helloRemoteConfig'
+        - 'functions/firebase/helloRTDB'
+        - 'functions/firebase/makeUpperCase'
     uses: ./.github/workflows/test.yaml
     with:
       name: 'functions-firebase'

.github/workflows/functions-helloworld.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-helloworld.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/helloworld/**'
-    - '.github/workflows/functions-helloworld.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/helloworld/**'
     - '.github/workflows/functions-helloworld.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:
@@ -43,11 +42,11 @@ jobs:
     strategy:
       matrix:
         path:
-          - 'functions/helloworld/helloError'
-          - 'functions/helloworld/helloGCS'
-          - 'functions/helloworld/helloPubSub'
-          - 'functions/helloworld/helloworldGet'
-          - 'functions/helloworld/helloworldHttp'
+        - 'functions/helloworld/helloError'
+        - 'functions/helloworld/helloGCS'
+        - 'functions/helloworld/helloPubSub'
+        - 'functions/helloworld/helloworldGet'
+        - 'functions/helloworld/helloworldHttp'
     uses: ./.github/workflows/test.yaml
     with:
       name: 'functions-helloworld'

.github/workflows/functions-http.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-http.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/http/**'
-    - '.github/workflows/functions-http.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/http/**'
     - '.github/workflows/functions-http.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:
@@ -43,10 +42,10 @@ jobs:
     strategy:
       matrix:
         path:
-          - 'functions/http/corsEnabledFunction'
-          - 'functions/http/corsEnabledFunctionAuth'
-          - 'functions/http/httpContent'
-          - 'functions/http/httpMethods'
+        - 'functions/http/corsEnabledFunction'
+        - 'functions/http/corsEnabledFunctionAuth'
+        - 'functions/http/httpContent'
+        - 'functions/http/httpMethods'
     uses: ./.github/workflows/test.yaml
     with:
       name: 'functions-http'

.github/workflows/functions-log-helloWorld.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-log-helloWorld.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/log/helloWorld/**'
-    - '.github/workflows/functions-log-helloWorld.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/log/helloWorld/**'
     - '.github/workflows/functions-log-helloWorld.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:

.github/workflows/functions-log-processEntry.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-log-processEntry.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/log/processEntry/**'
-    - '.github/workflows/functions-log-processEntry.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/log/processEntry/**'
     - '.github/workflows/functions-log-processEntry.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:

.github/workflows/functions-memorystore-redis.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-memorystore-redis.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/memorystore/redis/**'
-    - '.github/workflows/functions-memorystore-redis.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/memorystore/redis/**'
     - '.github/workflows/functions-memorystore-redis.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:

.github/workflows/functions-pubsub.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-pubsub.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/pubsub/**'
-    - '.github/workflows/functions-pubsub.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/pubsub/**'
     - '.github/workflows/functions-pubsub.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:
@@ -43,8 +42,8 @@ jobs:
     strategy:
       matrix:
         path:
-          - 'functions/pubsub/publish'
-          - 'functions/pubsub/subscribe'
+        - 'functions/pubsub/publish'
+        - 'functions/pubsub/subscribe'
     uses: ./.github/workflows/test.yaml
     with:
       name: 'functions-pubsub'

.github/workflows/functions-scheduleinstance.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-scheduleinstance.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/scheduleinstance/**'
-    - '.github/workflows/functions-scheduleinstance.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/scheduleinstance/**'
     - '.github/workflows/functions-scheduleinstance.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions:

.github/workflows/functions-security.yaml

@@ -22,18 +22,17 @@ on:
     - '.github/workflows/functions-security.yaml'
     - '.github/workflows/test.yaml'
   pull_request:
-    paths:
-    - 'functions/security/**'
-    - '.github/workflows/functions-security.yaml'
-    - '.github/workflows/test.yaml'
-  pull_request_target:
-    types: [labeled]
+    types:
+    - opened
+    - reopened
+    - synchronize
+    - labeled
     paths:
     - 'functions/security/**'
     - '.github/workflows/functions-security.yaml'
     - '.github/workflows/test.yaml'
   schedule:
-  - cron:  '0 0 * * 0'
+  - cron: '0 0 * * 0'
 jobs:
   test:
     permissions: