Addressed comments'

Author: maneesht

packages/data-connect/package.json

@@ -32,8 +32,7 @@
     "build:deps": "lerna run --scope @firebase/'{app,data-connect}' --include-dependencies build",
     "dev": "rollup -c -w",
     "test": "run-p --npm-path npm lint test:emulator",
-    "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:emulator",
-    "test:browser:unit": "karma start --unit",
+    "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all",
     "test:all": "run-p --npm-path npm lint test:browser test:node",
     "test:browser": "karma start",
     "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'test/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js",

scripts/ci-test/testConfig.ts

@@ -54,7 +54,6 @@ export const testConfig: {
       'firebase-namespace-integration-test',
       'firebase-compat-typings-test',
       '@firebase/rules-unit-testing',
-      '@firebase/data-connect',
       '@firebase/auth',
       '@firebase/auth-compat',
       'firebase'

scripts/run_tests_in_ci.js

@@ -27,8 +27,7 @@ const crossBrowserPackages = {
   'packages/auth': 'test:browser:unit',
   'packages/auth-compat': 'test:browser:unit',
   'packages/firestore': 'test:browser:unit',
-  'packages/firestore-compat': 'test:browser',
-  'packages/data-connect': 'test:browser:unit'
+  'packages/firestore-compat': 'test:browser'
 };
 
 function writeLogs(status, name, logText) {