ignore firebase and firebase-exp

Author: Feiyang1

scripts/ci-test/run_changed.ts

@@ -82,6 +82,12 @@ const specialPaths = {
   'packages/firestore': ['firebase-firestore-integration-test']
 };
 
+/**
+ * firebase and firebase-exp don't have tests and are very expensive to build, so we don't
+ * generate TestTasks for them.
+ */
+const ignoredPackages = ['firebase', 'firebase-exp'];
+
 /**
  * Identify modified packages that require tests.
  */
@@ -97,7 +103,7 @@ export async function getTestTasks(): Promise<TestTask[]> {
   );
   const diff = await git.diff(['--name-only', 'origin/master...HEAD']);
   const changedFiles = diff.split('\n');
-  const testTasks: TestTask[] = [];
+  let testTasks: TestTask[] = [];
   for (const filename of changedFiles) {
     // Files that trigger full test suite.
     if (fullTestTriggerFiles.includes(filename)) {
@@ -156,12 +162,16 @@ export async function getTestTasks(): Promise<TestTask[]> {
       }
     }
   }
+
   if (testTasks.length === 0) {
     console.log(
       chalk`{green No changes detected in any package. No test tasks is created }`
     );
   }
 
+  // remove ignored packages
+  testTasks = testTasks.filter(t => !ignoredPackages.includes(t.pkgName));
+
   return testTasks;
 }
 

scripts/ci-test/run_changed_fcm_integration.ts

@@ -16,13 +16,14 @@
  */
 
 import { getTestTasks, runTests } from './run_changed';
+import { buildForTests } from './build';
 
 const includeOnlyPackages = ['firebase-messaging-integration-test'];
 
 async function run() {
   let testTasks = await getTestTasks();
   testTasks = testTasks.filter(t => includeOnlyPackages.includes(t.pkgName));
-
+  await buildForTests(testTasks);
   runTests(testTasks);
 }
 

scripts/ci-test/run_changed_firestore.ts

@@ -16,6 +16,7 @@
  */
 
 import { getTestTasks, runTests } from './run_changed';
+import { buildForTests } from './build';
 
 const includeOnlyPackages = [
   '@firebase/firestore',
@@ -25,7 +26,7 @@ const includeOnlyPackages = [
 async function run() {
   let testTasks = await getTestTasks();
   testTasks = testTasks.filter(t => includeOnlyPackages.includes(t.pkgName));
-
+  await buildForTests(testTasks);
   runTests(testTasks);
 }