ensureCrux for test

Change-Id: I94c55d08bad7c58b28a5febccdc126dc0b012e5f

Author: paulirish

src/utils/crux.ts

@@ -183,19 +183,16 @@ export async function populateCruxData(
   parsedTrace: DevTools.TraceEngine.TraceModel.ParsedTrace,
 ): Promise<void> {
   ensureCrUXManager();
-  try {
-    const settings = DevTools.Common.Settings.Settings.instance();
-    const cruxSetting = settings.createSetting<ConfigSetting>(
-      'field-data',
-      {enabled: false},
-      DevTools.Common.Settings.SettingStorageType.GLOBAL,
-    );
-
-    if (!cruxSetting.get().enabled) {
-      return;
-    }
-  } catch {
-    // Fallback if settings are not available
+
+  const settings = DevTools.Common.Settings.Settings.instance();
+  const cruxSetting = settings.createSetting(
+    'field-data',
+    {enabled: true},
+    DevTools.Common.Settings.SettingStorageType.GLOBAL,
+  );
+
+  if (!cruxSetting.get().enabled) {
+    return;
   }
 
   const urls = new Set<string>();

tests/utils/crux.test.ts

@@ -5,7 +5,7 @@
  */
 
 import assert from 'node:assert';
-import {describe, it, afterEach} from 'node:test';
+import {describe, it, afterEach, before} from 'node:test';
 
 import sinon from 'sinon';
 
@@ -20,6 +20,8 @@ import {ensureCrUXManager} from '../../src/utils/crux.js';
 import {loadTraceAsBuffer} from '../trace-processing/fixtures/load.js';
 
 describe('crux util', () => {
+  before(() => ensureCrUXManager());
+
   afterEach(() => {
     sinon.restore();
   });