Exclude specs in spec testing. (#9240)

* Exclude specs in spec testing.

Author: granluo

.github/workflows/spectesting.yml

@@ -31,7 +31,9 @@ jobs:
         # The final matrix will be set as an env var in the GHA workflow,
         # This env var, matrix, will be passed as a matrix in the next job of
         # the workflow.
-        ./scripts/health_metrics/get_updated_files.sh -p output.json
+        # If multiple podspecs need to be excluded, escaping space is required
+        # to split podspecs, e.g. get_updated_files.sh ... -e Firebase.podspec\ FirebaseABTesting.podspec
+        ./scripts/health_metrics/get_updated_files.sh -p output.json -e Firebase.podspec
         echo "::set-output name=matrix::{\"include\":$( cat output.json )}"
         # `podspecs` is to help determine if specs_testing job should be run.
         echo "::set-output name=podspecs::$(cat output.json)"

scripts/health_metrics/generate_code_coverage_report/Sources/UpdatedFilesCollector/main.swift

@@ -56,6 +56,10 @@ struct UpdatedFilesCollector: ParsableCommand {
           })
   var outputSDKFileURL: URL?
 
+  /// Exclude pods from spec testings.
+  @Option(parsing: .upToNextOption, help: "Podspecs that will be excluded in the testings.")
+  var excludePodspecs: [String] = []
+
   func run() throws {
     var podspecsWithChangedFiles: [SDKPodspec] = []
     print("=============== list changed files ===============")
@@ -78,7 +82,13 @@ struct UpdatedFilesCollector: ParsableCommand {
             print("=============== paths of changed files ===============")
             print("::set-output name=\(sdkPatterns.sdk)_run_job::true")
             for podspec in sdkPatterns.podspecs {
-              podspecsWithChangedFiles.append(SDKPodspec(podspec: podspec))
+              if !excludePodspecs.contains(podspec) {
+                podspecsWithChangedFiles.append(SDKPodspec(podspec: podspec))
+              } else if let outputPath = outputSDKFileURL {
+                print(
+                  "\(podspec) was excluded and will not be written in \(outputPath.absoluteString) "
+                )
+              }
             }
             print("\(sdkPatterns.sdk): \(changedFilePath) is updated under the pattern, \(pattern)")
             trigger_pod_test_for_coverage_report = true

scripts/health_metrics/get_updated_files.sh

@@ -15,11 +15,12 @@
 set -ex
 # Updated files in paths in file_patterns.json will trigger code coverage workflows.
 # Updates in a pull request will generate a code coverage report in a PR.
-
-while getopts p: flag
+exclude_specs=""
+while getopts "p:e:" flag
 do
     case "${flag}" in
         p) spec_output_file=${OPTARG};;
+        e) exclude_specs=${OPTARG[*]};;
     esac
 done
 
@@ -39,6 +40,7 @@ git diff --name-only remotes/origin/${GITHUB_BASE_REF} ${GITHUB_SHA} > updated_f
 if [ -z $spec_output_file] ; then
   swift run UpdatedFilesCollector --changed-file-paths updated_files.txt --code-coverage-file-patterns ../file_patterns.json
 else
-  swift run UpdatedFilesCollector --changed-file-paths updated_files.txt --code-coverage-file-patterns ../file_patterns.json --output-sdk-file-url "${spec_output_file}"
+  swift run UpdatedFilesCollector --changed-file-paths updated_files.txt --code-coverage-file-patterns ../file_patterns.json --output-sdk-file-url "${spec_output_file}" --exclude-podspecs ${exclude_specs}
+
   mv "${spec_output_file}" "${dir}"
 fi