Fix lint errors in test files

- Add eslint-disable for no-explicit-any in useQueryState tests (test mocks)
- Add eslint-disable for import/named on waitFor import
- Fix type annotations for mock helpers (return types, parameter types)
- Extract MockProfileTypesData type to avoid no-undef-init warning
- Fix toHaveProperty('value', x) to use filter.value assertion

Author: yomete

ui/packages/shared/profile/src/ProfileView/components/ProfileFilters/useProfileFiltersUrlState.test.tsx

@@ -629,7 +629,7 @@ describe('useProfileFiltersUrlState', () => {
       expect(filter).toHaveProperty('type', 'stack');
       expect(filter).toHaveProperty('field', 'function_name');
       expect(filter).toHaveProperty('matchType', 'equal');
-      expect(filter).toHaveProperty('value', 'testFunc');
+      expect(filter.value).toBe('testFunc');
     });
   });
 

ui/packages/shared/profile/src/hooks/useQueryState.test.tsx

@@ -11,10 +11,13 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+/* eslint-disable @typescript-eslint/no-explicit-any */
+
 import {ReactNode, act} from 'react';
 
 // eslint-disable-next-line import/named
 import {QueryClient, QueryClientProvider} from '@tanstack/react-query';
+// eslint-disable-next-line import/named
 import {renderHook, waitFor} from '@testing-library/react';
 import {beforeEach, describe, expect, it, vi} from 'vitest';
 
@@ -78,12 +81,12 @@ vi.mock('../useSumBy', async () => {
   return {
     ...actual,
     useSumBy: (
-      _queryClient: any,
-      _profileType: any,
-      _timeRange: any,
-      _draftProfileType: any,
-      _draftTimeRange: any,
-      defaultValue: any
+      _queryClient: unknown,
+      _profileType: unknown,
+      _timeRange: unknown,
+      _draftProfileType: unknown,
+      _draftTimeRange: unknown,
+      defaultValue: string[] | undefined
     ) => {
       const [draftSumBy, setDraftSumBy] = react.useState<string[] | undefined>(defaultValue);
       const [sumBy, setSumBy] = react.useState<string[] | undefined>(defaultValue);
@@ -100,9 +103,8 @@ vi.mock('../useSumBy', async () => {
   };
 });
 
-// Track profile types loading state for tests
-let mockProfileTypesLoading = false;
-let mockProfileTypesData:
+// Type for mock profile types data
+type MockProfileTypesData =
   | {
       types: Array<{
         name: string;
@@ -113,7 +115,11 @@ let mockProfileTypesData:
         delta: boolean;
       }>;
     }
-  | undefined = undefined;
+  | undefined;
+
+// Track profile types loading state for tests
+let mockProfileTypesLoading = false;
+let mockProfileTypesData: MockProfileTypesData;
 
 // Mock useProfileTypes to control loading state in tests
 vi.mock('../ProfileSelector', async () => {
@@ -129,11 +135,11 @@ vi.mock('../ProfileSelector', async () => {
 });
 
 // Helper to set profile types loading state for tests
-const setProfileTypesLoading = (loading: boolean) => {
+const setProfileTypesLoading = (loading: boolean): void => {
   mockProfileTypesLoading = loading;
 };
 
-const setProfileTypesData = (data: typeof mockProfileTypesData) => {
+const setProfileTypesData = (data: MockProfileTypesData): void => {
   mockProfileTypesData = data;
 };