@jmruthers/pace-core 0.6.5 → 0.6.7

package/scripts/install-cursor-rules.cjs

@@ -5,8 +5,16 @@
  * @package @jmruthers/pace-core
  * @module Scripts/install-cursor-rules
  * 
- * Copies cursor rules from pace-core to consuming app's .cursor/rules/ directory.
+ * Sets up cursor rules:
+ * - Copies cursor rules from pace-core to consuming app's .cursor/rules/ directory
+ * 
  * This is an opt-in script - it does NOT run automatically via postinstall.
+ * 
+ * For ESLint setup, use install-eslint-config.cjs separately.
+ * 
+ * Usage:
+ *   node install-cursor-rules.cjs        # Install cursor rules
+ *   node install-cursor-rules.cjs --force # Force update even if already configured
  */
 
 const fs = require('fs');
@@ -171,9 +179,11 @@ function installCursorRules(force = false) {
     const sourcePath = path.join(sourceDir, file);
     const targetPath = path.join(targetDir, file);
     
-    // Check if this is a pace-core rule (00-09) or app rule (50+)
+    // Check if this is a pace-core rule (00-49) or app rule (50+)
+    // pace-core rules: 00-49 (currently 00-12, but leaving room for expansion)
+    // app rules: 50+ (custom rules for consuming apps)
     const ruleNumber = parseInt(file.match(/^(\d+)-/)?.[1] || '99');
-    const isPaceCoreRule = ruleNumber >= 0 && ruleNumber <= 9;
+    const isPaceCoreRule = ruleNumber >= 0 && ruleNumber <= 49;
     
     if (fs.existsSync(targetPath)) {
       if (isPaceCoreRule) {
@@ -208,18 +218,24 @@ function installCursorRules(force = false) {
   console.log(`${colors.cyan}Restart Cursor to load the new rules.${colors.reset}`);
 }
 
+
 // Main execution
 function main() {
   const force = process.argv.includes('--force');
   
   if (force) {
-    console.log(`${colors.yellow}Warning: --force flag is set. This will overwrite existing pace-core rules.${colors.reset}\n`);
+    console.log(`${colors.yellow}Warning: --force flag is set. This will overwrite existing configurations.${colors.reset}\n`);
   }
   
   try {
     installCursorRules(force);
+    
+    console.log(`\n${colors.cyan}Next steps:${colors.reset}`);
+    console.log(`  • Restart Cursor to load the new rules`);
+    console.log(`  • For ESLint setup, run: ${colors.bold}node node_modules/@jmruthers/pace-core/scripts/install-eslint-config.cjs${colors.reset}`);
+    
   } catch (error) {
-    console.error(`${colors.red}Error installing cursor rules:${colors.reset}`);
+    console.error(`${colors.red}Error during setup:${colors.reset}`);
     console.error(error.message);
     if (error.stack) {
       console.error(error.stack);
@@ -233,4 +249,7 @@ if (require.main === module) {
   main();
 }
 
-module.exports = { installCursorRules, getCursorRulesTarget };
+module.exports = { 
+  installCursorRules, 
+  getCursorRulesTarget
+};

package/scripts/install-eslint-config.cjs

@@ -0,0 +1,284 @@
+#!/usr/bin/env node
+
+/**
+ * Installation Script for pace-core ESLint Config
+ * @package @jmruthers/pace-core
+ * @module Scripts/install-eslint-config
+ * 
+ * Sets up pace-core ESLint configuration in consuming apps.
+ * Adds pace-core ESLint config to consuming app's ESLint configuration.
+ * 
+ * This is an opt-in script - it does NOT run automatically via postinstall.
+ * 
+ * Usage:
+ *   node install-eslint-config.cjs        # Setup ESLint config
+ *   node install-eslint-config.cjs --force # Force update even if already configured
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// Check for safety guards
+function checkSafetyGuards() {
+  // Check environment variable
+  if (process.env.PACE_ESLINT_CONFIG_DISABLED === '1') {
+    console.log(`${colors.yellow}Skipping ESLint config installation: PACE_ESLINT_CONFIG_DISABLED=1${colors.reset}`);
+    process.exit(0);
+  }
+  
+  // Check for .git folder (safety guard for CI)
+  const cwd = process.cwd();
+  if (!fs.existsSync(path.join(cwd, '.git'))) {
+    console.log(`${colors.yellow}Skipping ESLint config installation: No .git folder found${colors.reset}`);
+    console.log(`${colors.yellow}This is a safety guard. If you want to install anyway, run with --force${colors.reset}`);
+    if (!process.argv.includes('--force')) {
+      process.exit(0);
+    }
+  }
+}
+
+// ANSI color codes for terminal output
+const colors = {
+  reset: '\x1b[0m',
+  green: '\x1b[32m',
+  yellow: '\x1b[33m',
+  blue: '\x1b[34m',
+  cyan: '\x1b[36m',
+  bold: '\x1b[1m',
+  red: '\x1b[31m'
+};
+
+// Find existing ESLint config file
+function findESLintConfig() {
+  const cwd = process.cwd();
+  const possibleConfigs = [
+    'eslint.config.js',
+    'eslint.config.cjs',
+    'eslint.config.mjs',
+    '.eslintrc.js',
+    '.eslintrc.cjs',
+    '.eslintrc.json',
+    '.eslintrc.yaml',
+    '.eslintrc.yml',
+  ];
+  
+  for (const configFile of possibleConfigs) {
+    const configPath = path.join(cwd, configFile);
+    if (fs.existsSync(configPath)) {
+      return { path: configPath, name: configFile, isESM: configFile.endsWith('.js') || configFile.endsWith('.mjs') };
+    }
+  }
+  
+  return null;
+}
+
+// Check if pace-core config is already included
+function hasPaceCoreConfig(configContent) {
+  // Check for pace-core config import/require
+  const paceCorePatterns = [
+    /@jmruthers\/pace-core\/eslint-config/,
+    /pace-core\/eslint-config/,
+    /paceCoreConfig/,
+    /pace-core-compliance/,
+  ];
+  
+  return paceCorePatterns.some(pattern => pattern.test(configContent));
+}
+
+// Detect if file is ES module
+function isESModule(filePath, content) {
+  if (filePath.endsWith('.mjs')) return true;
+  if (filePath.endsWith('.cjs')) return false;
+  if (filePath.endsWith('.js')) {
+    // Check for ES module indicators
+    return /^import\s+.*from|^export\s+default/.test(content.trim());
+  }
+  return false;
+}
+
+// Backup file before modification
+function backupFile(filePath) {
+  const backupPath = `${filePath}.backup.${Date.now()}`;
+  fs.copyFileSync(filePath, backupPath);
+  return backupPath;
+}
+
+// Setup ESLint configuration
+function setupESLintConfig(force = false) {
+  const cwd = process.cwd();
+  const existingConfig = findESLintConfig();
+  
+  if (existingConfig) {
+    // Read existing config
+    let content = fs.readFileSync(existingConfig.path, 'utf8');
+    const isESM = isESModule(existingConfig.path, content);
+    const format = isESM ? 'ES Module' : 'CommonJS';
+    
+    console.log(`${colors.cyan}Found ESLint config:${colors.reset} ${existingConfig.name} (${format})`);
+    
+    // Check if already configured
+    if (hasPaceCoreConfig(content)) {
+      if (!force) {
+        console.log(`${colors.blue}○${colors.reset} ${existingConfig.name} already includes pace-core rules`);
+        console.log(`${colors.blue}  Use --force to update.${colors.reset}`);
+        return { action: 'skipped', file: existingConfig.name, format };
+      }
+      
+      console.log(`${colors.yellow}Updating existing ESLint config...${colors.reset}`);
+    } else {
+      console.log(`${colors.cyan}Adding pace-core config to ${existingConfig.name}...${colors.reset}`);
+    }
+    
+    // Backup before modification
+    const backupPath = backupFile(existingConfig.path);
+    console.log(`${colors.cyan}  Backed up to ${path.basename(backupPath)}${colors.reset}`);
+    
+    // Add pace-core config
+    if (isESM) {
+      // ES Module format
+      if (!content.includes('import paceCoreConfig')) {
+        // Add import at top (after other imports if they exist)
+        const importLines = content.match(/^(import\s+[^;]+;?\s*\n)+/m);
+        if (importLines) {
+          // Add after existing imports
+          content = content.replace(
+            importLines[0],
+            `${importLines[0]}import paceCoreConfig from '@jmruthers/pace-core/eslint-config';\n`
+          );
+        } else {
+          // Add at the beginning
+          content = `import paceCoreConfig from '@jmruthers/pace-core/eslint-config';\n${content}`;
+        }
+      }
+      
+      // Add to export default array
+      if (content.includes('export default [')) {
+        // Already an array, add paceCoreConfig at the beginning
+        if (!content.includes('...paceCoreConfig')) {
+          content = content.replace(
+            /(export\s+default\s*\[)\s*/,
+            '$1\n  ...paceCoreConfig,'
+          );
+        }
+      } else if (content.includes('export default')) {
+        // Not an array, wrap it
+        const exportMatch = content.match(/(export\s+default\s+)(.+?)(;?\s*$)/s);
+        if (exportMatch) {
+          content = content.replace(
+            exportMatch[0],
+            `${exportMatch[1]}[\n  ...paceCoreConfig,\n  ${exportMatch[2]}\n];`
+          );
+        }
+      }
+    } else {
+      // CommonJS format
+      if (!content.includes('require(\'@jmruthers/pace-core/eslint-config\')')) {
+        // Add require at top (after other requires if they exist)
+        const requireLines = content.match(/^(const\s+\w+\s*=\s*require\([^)]+\);\s*\n)+/m);
+        if (requireLines) {
+          content = content.replace(
+            requireLines[0],
+            `${requireLines[0]}const paceCoreConfig = require('@jmruthers/pace-core/eslint-config');\n`
+          );
+        } else {
+          content = `const paceCoreConfig = require('@jmruthers/pace-core/eslint-config');\n${content}`;
+        }
+      }
+      
+      // Add to module.exports
+      if (content.includes('module.exports = [')) {
+        // Already an array, add paceCoreConfig at the beginning
+        if (!content.includes('...paceCoreConfig')) {
+          content = content.replace(
+            /(module\.exports\s*=\s*\[)\s*/,
+            '$1\n  ...paceCoreConfig,'
+          );
+        }
+      } else if (content.includes('module.exports =')) {
+        // Not an array, wrap it
+        const exportMatch = content.match(/(module\.exports\s*=\s*)(.+?)(;?\s*$)/s);
+        if (exportMatch) {
+          content = content.replace(
+            exportMatch[0],
+            `${exportMatch[1]}[\n  ...paceCoreConfig,\n  ${exportMatch[2]}\n];`
+          );
+        }
+      }
+    }
+    
+    // Write updated config
+    fs.writeFileSync(existingConfig.path, content, 'utf8');
+    console.log(`${colors.green}✓${colors.reset} Updated ${existingConfig.name}`);
+    return { action: 'updated', file: existingConfig.name, backup: backupPath, format };
+  } else {
+    // Create new ESLint config (default to ES modules)
+    const configPath = path.join(cwd, 'eslint.config.js');
+    const configContent = `import paceCoreConfig from '@jmruthers/pace-core/eslint-config';
+
+export default [
+  ...paceCoreConfig,
+  // Your app-specific ESLint configuration
+  {
+    // Add your rules here
+  },
+];
+`;
+    
+    console.log(`${colors.cyan}No ESLint config found. Creating eslint.config.js...${colors.reset}`);
+    fs.writeFileSync(configPath, configContent, 'utf8');
+    console.log(`${colors.green}✓${colors.reset} Created eslint.config.js`);
+    console.log(`${colors.cyan}  Edit eslint.config.js to add your app-specific ESLint rules.${colors.reset}`);
+    return { action: 'created', file: 'eslint.config.js', format: 'ES Module' };
+  }
+}
+
+// Main execution
+function main() {
+  checkSafetyGuards();
+  
+  const force = process.argv.includes('--force');
+  
+  if (force) {
+    console.log(`${colors.yellow}Warning: --force flag is set. This will overwrite existing configurations.${colors.reset}\n`);
+  }
+  
+  try {
+    console.log(`${colors.cyan}Setting up ESLint configuration from pace-core...${colors.reset}\n`);
+    const result = setupESLintConfig(force);
+    
+    // Summary
+    console.log(`\n${colors.bold}Setup Summary:${colors.reset}`);
+    if (result.action === 'created') {
+      console.log(`  ${colors.green}ESLint Config:${colors.reset} Created ${result.file} (${result.format})`);
+    } else if (result.action === 'updated') {
+      console.log(`  ${colors.green}ESLint Config:${colors.reset} Updated ${result.file} (${result.format})`);
+      console.log(`  ${colors.yellow}Backup:${colors.reset} ${path.basename(result.backup)}`);
+    } else {
+      console.log(`  ${colors.blue}ESLint Config:${colors.reset} Already configured (${result.file}, ${result.format})`);
+    }
+    
+    console.log(`\n${colors.cyan}ESLint configuration is now available${colors.reset}`);
+    console.log(`\n${colors.cyan}Next steps:${colors.reset}`);
+    console.log(`  • Run ${colors.bold}npm run lint${colors.reset} to verify ESLint is working`);
+    console.log(`  • Edit your ESLint config to add app-specific rules`);
+    console.log(`  • See ${colors.bold}packages/core/docs/standards/${colors.reset} for complete standards documentation`);
+    
+  } catch (error) {
+    console.error(`${colors.red}Error during setup:${colors.reset}`);
+    console.error(error.message);
+    if (error.stack) {
+      console.error(error.stack);
+    }
+    process.exit(1);
+  }
+}
+
+// Run if called directly
+if (require.main === module) {
+  main();
+}
+
+module.exports = { 
+  setupESLintConfig,
+  findESLintConfig
+};

package/src/__tests__/fixtures/supabase.ts

@@ -8,7 +8,7 @@
  * Reduces repeated mock setup across test files.
  */
 
-import { createMockSupabaseClient, createMockQueryBuilder } from '../helpers/supabaseMock';
+import { createMockSupabaseClient } from '../helpers/supabaseMock';
 import type { User, Session } from '@supabase/supabase-js';
 import { TEST_FIXTURES } from './test-data';
 

package/src/__tests__/helpers/__tests__/component-test-utils.test.tsx

@@ -7,7 +7,7 @@
 
 import React from 'react';
 import { render, screen } from '@testing-library/react';
-import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import userEvent from '@testing-library/user-event';
 import {
   componentTestPatterns,
@@ -23,7 +23,7 @@ vi.mock('../test-utils', () => ({
 describe('[helpers] componentTestPatterns', () => {
     describe('testRenders', () => {
     it('creates test that renders component without crashing', () => {
-      const TestComponent = () => <div data-testid="test-component">Test</div>;
+      const TestComponent = () => <p data-testid="test-component">Test</p>;
       const testFn = componentTestPatterns.testRenders(TestComponent);
       
       expect(typeof testFn).toBe('function');
@@ -33,7 +33,7 @@ describe('[helpers] componentTestPatterns', () => {
     });
 
     it('creates test that renders component with props', () => {
-      const TestComponent = ({ name }: { name: string }) => <div data-testid="test-component">{name}</div>;
+      const TestComponent = ({ name }: { name: string }) => <p data-testid="test-component">{name}</p>;
       const testFn = componentTestPatterns.testRenders(TestComponent, { name: 'Test Name' });
       
       expect(typeof testFn).toBe('function');

package/src/__tests__/helpers/__tests__/optimized-test-setup.test.ts

@@ -5,7 +5,7 @@
  * @since 1.0.0
  */
 
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import {
   createFastMock,
   createFastErrorMock,

package/src/__tests__/helpers/__tests__/supabaseMock.test.ts

@@ -5,7 +5,7 @@
  * @since 1.0.0
  */
 
-import { describe, it, expect, vi, beforeEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import {
   createMockQueryBuilder,
   createMockSupabaseClient,

package/src/__tests__/helpers/__tests__/test-providers.test.tsx

@@ -14,11 +14,11 @@ import { TestProviderWrapper, renderWithProviders, createMockUseUnifiedAuth, use
 function TestComponent() {
   const auth = useUnifiedAuth();
   return (
-    <div>
+    <p>
       <span data-testid="user-email">{auth.user?.email}</span>
       <span data-testid="is-authenticated">{auth.isAuthenticated ? 'true' : 'false'}</span>
       <span data-testid="app-name">{auth.appName}</span>
-    </div>
+    </p>
   );
 }
 

package/src/__tests__/helpers/__tests__/test-utils.test.tsx

@@ -7,7 +7,7 @@
 
 import React from 'react';
 import { render, screen } from '@testing-library/react';
-import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+import { describe, it, expect, vi } from 'vitest';
 import { QueryClient } from '@tanstack/react-query';
 import {
   renderWithProviders,
@@ -41,32 +41,32 @@ vi.mock('../fixtures/test-data', () => ({
 
 describe('[helpers] renderWithProviders', () => {
     it('renders component with QueryClient provider by default', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />);
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('renders component without QueryClient when withQueryClient is false', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />, { withQueryClient: false });
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('uses custom QueryClient when provided', () => {
     const customQueryClient = new QueryClient();
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     renderWithProviders(<TestComponent />, { queryClient: customQueryClient });
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 
   it('forwards additional render options', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     const { container } = renderWithProviders(<TestComponent />, {
       container: document.body
@@ -273,7 +273,7 @@ describe('[helpers] createMockSupabaseClient', () => {
 
   it('creates mock query builder with chaining methods', () => {
     const mockClient = createMockSupabaseClient();
-    const queryBuilder = mockClient.from('test_table');
+    const queryBuilder = (mockClient.from as any)('test_table');
     
     expect(queryBuilder).toHaveProperty('select');
     expect(queryBuilder).toHaveProperty('insert');
@@ -302,7 +302,7 @@ describe('[helpers] createMockSupabaseClient', () => {
 
 describe('[helpers] TestWrapper', () => {
     it('renders children with QueryClient provider', () => {
-    const TestComponent = () => <div data-testid="test-component">Test</div>;
+    const TestComponent = () => <p data-testid="test-component">Test</p>;
     
     render(
       <TestWrapper>
@@ -310,7 +310,7 @@ describe('[helpers] TestWrapper', () => {
       </TestWrapper>
     );
     
-    expect(screen.getByTestId('test-component')).toBeInTheDocument();
+    expect(screen.getByTestId('test-component')).toBeDefined();
   });
 });
 
@@ -387,8 +387,8 @@ describe('[helpers] testHelpers', () => {
 
 describe('[helpers] setupTest', () => {
     it('sets up test environment', () => {
-    const mockClearAllMocks = vi.spyOn(vi, 'clearAllMocks');
-    const mockRestoreAllMocks = vi.spyOn(vi, 'restoreAllMocks');
+    vi.spyOn(vi, 'clearAllMocks');
+    vi.spyOn(vi, 'restoreAllMocks');
     
     setupTest();
     

package/src/__tests__/helpers/component-test-utils.tsx

@@ -3,7 +3,7 @@
  * @description Specialized utilities for component testing
  */
 
-import { render, screen, RenderResult, cleanup } from '@testing-library/react';
+import { screen, cleanup } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { vi } from 'vitest';
 import { renderWithProviders } from './test-utils';

package/src/__tests__/helpers/supabaseMock.ts

@@ -28,7 +28,7 @@ export const createMockQueryBuilder = (defaultData: any = { data: [], error: nul
 
   // Create a thenable range function that returns self
   const createRangeFunction = (builder: any) => {
-    return vi.fn().mockImplementation(function(min: number, max: number) {
+    return vi.fn().mockImplementation(function(_min: number, _max: number) {
       // Return builder with thenable interface
       return Object.assign(builder, {
         then: vi.fn().mockImplementation((resolve, reject) => {
@@ -132,7 +132,7 @@ export const createMockQueryBuilderWithData = (data: any, error: any = null) =>
   });
   
   // Ensure range() is available and returns a thenable
-  mockQueryBuilder.range = vi.fn().mockImplementation(function(min: number, max: number) {
+  mockQueryBuilder.range = vi.fn().mockImplementation(function(_min: number, _max: number) {
     // Return a thenable builder for range() that can be awaited
     const rangedBuilder = Object.assign(this, {
       then: vi.fn().mockImplementation((resolve, reject) => {

package/src/__tests__/integration/UserProfile.test.tsx

@@ -37,18 +37,18 @@ const UserProfile = ({ userId }: { userId: string }) => {
     fetchUser();
   }, [userId]);
 
-  if (loading) return <div>Loading...</div>;
-  if (error) return <div>Error: {error}</div>;
-  if (!user) return <div>No user found</div>;
+  if (loading) return <p>Loading...</p>;
+  if (error) return <p>Error: {error}</p>;
+  if (!user) return <p>No user found</p>;
 
   return (
-    <div data-testid="user-profile">
+    <section data-testid="user-profile">
       <h1>{user.name}</h1>
       <p>{user.email}</p>
       <button onClick={() => setUser({ ...user, name: 'Updated Name' })}>
         Update Name
       </button>
-    </div>
+    </section>
   );
 };
 
@@ -59,17 +59,17 @@ describe('User Profile Integration Tests', () => {
       
       // The component loads data synchronously in this mock, so we just check the final state
       await waitFor(() => {
-        expect(screen.getByText('John Doe')).toBeInTheDocument();
+        expect(screen.getByText('John Doe')).toBeDefined();
       });
       
-      expect(screen.getByText('john@example.com')).toBeInTheDocument();
+      expect(screen.getByText('john@example.com')).toBeDefined();
     });
 
     it('handles user not found', async () => {
       renderWithProviders(<UserProfile userId="999" />);
       
       await waitFor(() => {
-        expect(screen.getByText('Error: User not found')).toBeInTheDocument();
+        expect(screen.getByText('Error: User not found')).toBeDefined();
       });
     });
   });
@@ -80,13 +80,13 @@ describe('User Profile Integration Tests', () => {
       renderWithProviders(<UserProfile userId="1" />);
       
       await waitFor(() => {
-        expect(screen.getByText('John Doe')).toBeInTheDocument();
+        expect(screen.getByText('John Doe')).toBeDefined();
       });
       
       await user.click(screen.getByText('Update Name'));
       
-      expect(screen.getByText('Updated Name')).toBeInTheDocument();
-      expect(screen.queryByText('John Doe')).not.toBeInTheDocument();
+      expect(screen.getByText('Updated Name')).toBeDefined();
+      expect(screen.queryByText('John Doe')).toBeNull();
     });
   });
 
@@ -95,12 +95,12 @@ describe('User Profile Integration Tests', () => {
       const { rerender } = renderWithProviders(<UserProfile userId="1" />);
       
       await waitFor(() => {
-        expect(screen.getByText('John Doe')).toBeInTheDocument();
+        expect(screen.getByText('John Doe')).toBeDefined();
       });
       
       rerender(<UserProfile userId="1" />);
       
-      expect(screen.getByText('John Doe')).toBeInTheDocument();
+      expect(screen.getByText('John Doe')).toBeDefined();
     });
   });
 });
@@ -117,7 +117,7 @@ describe('User Profile - Different Users', () => {
       renderWithProviders(<UserProfile userId={userId} />);
       
       await waitFor(() => {
-        expect(screen.getByText(expectedName)).toBeInTheDocument();
+        expect(screen.getByText(expectedName)).toBeDefined();
       });
     });
   });